|Summary:||Formatted diff has problems with minus characters|
|Product:||Gentoo Infrastructure||Reporter:||Martin von Gagern <Martin.vGagern>|
|Component:||Bugzilla||Assignee:||Gentoo Linux bug wranglers <bug-wranglers>|
|Package list:||Runtime testing required:||---|
Description Martin von Gagern 2008-09-10 11:30:52 UTC
See this attachment here: http://bugs.gentoo.org/attachment.cgi?id=165098&action=diff http://bugs.gentoo.org/attachment.cgi?id=165098&action=diff&format=raw http://bugs.gentoo.org/attachment.cgi?id=165098 As you can see, both the coloured and the processed unified patch have problems with the lines starting in "-----" in the original file. I guess this is due to some confusion with the "---" introducing the fist input file of an unified patch. If that is the case, maybe you should rather check for "--- ", i.e. make the space character part of the search pattern.
Comment 1 Jeremy Olexa (darkside) (RETIRED) 2008-09-10 17:27:35 UTC
I don't see a problem tbh, don't use the "diff" feature. Anyway, if it can be fixed, it needs to be fixed upstream anyway. You should file a bug at bugzilla.org. Keep in mind that we use version 2.22 and 3.2 is now released. bug #213782
Comment 2 Martin von Gagern 2008-09-10 19:03:45 UTC
Issue is known upstream: https://bugzilla.mozilla.org/show_bug.cgi?id=233695 Still present in latest CVS version. Sorry I didn't look upstream first. Seems to come from a bug in PatchReader perl module. I can't reach the perl issue tracker atm, so I don't know what details might be available on http://rt.cpan.org/Public/Bug/Display.html?id=35362 Would this issue here be better off closed as UPSTREAM?