I want to learn to patch

Dotan Cohen dotancohen at gmail.com
Sat Jun 6 15:51:27 BST 2009

> First I admire your enthusiasm.  Do not take offense at my rather direct
> engineer's analysis.

Quite the opposite, I appreciate the harsh truth. I have thick skin!

> I have to warn your that there have been major arguments about this
> issue and I have rather strong opinions about it.  We won't worry about
> my opinions but expect to run into some resistance.

Really? Well that's good for me then! It means that the bug should
still be around by the time I get my head wrapped around the
development base. I'm not overly concerned with the bug in question, I
find it to be only a minor issue, but it seems to be a good bug to
practice on.

> This is probably part of the Konqueror code:
>        KDEBase:Apps:Konqueror
> in SVN.  You can find the relevant files using "Find File" looking for
> "*.*c*" with a string in: "Context::Containing Text" or use Grep
> (command line) in  Konsole.

The web interface will not allow asterisks in searches, so local it must be!

>> 2) How does one make a diff file?
> First, you need to checkout and build the KDEBase module from SVN:
> svn co svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase
> Well actually, you need to install subversion, the GNU C/C++ compiler,
> and CMake.  I would suggest reading the relevant (user/client) parts of:
>        http://svnbook.red-bean.com/
> Now, having found what needs to be changed:
> 1.  Change it.
> 2.  Compile and install.
> 3.  Test to see that it works like you want (if not go back to #1)
> 4.  Execute: "svn status" to see that all is well
> 5.  Execute: "svn up" to see that there is no conflict
> 6.  Execute: "svn diff" to see your patch.
> 7.  If you want a patch file, Execute: "svn diff > file.diff
> 8.  Post your patch to the "kde-devel" (subscribe to it) mailing list
>     with the Subject starting with "[PATCH]" or attach the diff file to
>     the bug and label the attachment as a Patch.
> So, that is a basic outline.  I expect that you might have further
> questions.

Thank you! I will have more questions as I progress, and I won't be
shy about asking. Thanks!

Dotan Cohen

This message is from the kde mailing list.
Account management:  https://mail.kde.org/mailman/listinfo/kde.
Archives: http://lists.kde.org/.
More info: http://www.kde.org/faq.html.

More information about the kde mailing list