[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kmail-devel
Subject:    Re: [PATCH] kioslaves/imap4/imap.cc
From:       Ingo =?iso-8859-15?q?Kl=F6cker?= <kloecker () kde ! org>
Date:       2006-05-01 0:10:59
Message-ID: 200605010211.05432 () erwin ! ingo-kloecker ! de
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Wednesday 26 April 2006 19:35, Allen Winter wrote:
> Howdy KMail Gurus,
>
> Christoph Bartoschek's "Suspicious Code" led me to this patch for
> imap.cc in branches/KDE/3.5. I'm not sure if I'm doing the right
> thing so please review.
>
> Regards,
> Allen
>
> PS: sorry for all the whitespace diffs... not sure how to tell 'svn
> diff' to ignore whitespace

You have to redefine the diff command (~/.subversion/config; look for 
diff-cmd, uncomment it and set it to the wrapper script) and add a 
wrapper script like the following:
=====
#! /bin/sh
exec /usr/bin/diff -u -b -p "$@"
=====

I can't say anything about the patch because I'm unable to see whether 
you changed anything apart from adding "cmd &&" in the if-statement.

Regards,
Ingo

[Attachment #5 (application/pgp-signature)]

_______________________________________________
KMail developers mailing list
KMail-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmail-devel


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic