Re: [gentoo-alt] bad news: doxygen

[Ben Peddell]

Fabian Groffen wrote:
> On 13-07-2007 22:50:31 +1000, Ben Peddell wrote:
>> Fabian Groffen wrote:
>>> On 12-07-2007 21:01:28 +0200, Elias Pipping wrote:
>>>> Mail.app tends to integrate text attachments into the body of the mail,
>>>> so you no longer know where the patch begins and where it ends. Might
>>>> become ugly with tabs/spaces, too.
>>> It horks the spaces/tabs.
>>>
>> If it weren't for the spaces/tabs issue, all would be fine. As long as:
>> 1. The patch is in unified format,
>> 2. The MUA doesn't gobble any spaces, and
>> 3. You don't start any of your lines with +, - or @,
>> the entire message body can be the patch.
>> patch has no problem with stripping any leading / trailing lines from
>> the patch.
>
> Of course I tried it, and the patch failed to apply because of spaces
> mismatches.
>
Hmm. No problem with spaces here.
====================================================================
~/src/doxygen-1.5.2 $ bzcat ~/Documents/doxygen-1.5.2-libiconv.patch.bz2
| patch --dry-run -p0 --verbose
Hmm...  Looks like a unified diff to me...
The text leading up to this was:
--------------------------
|--- src/portable.cpp~  2007-07-12 20:41:27 +0200
|+++ src/portable.cpp   2007-05-06 14:30:18 +0200
--------------------------
Patching file src/portable.cpp using Plan A...
Hunk #1 succeeded at 373.
done
====================================================================
Is there a patch that patches the same area of
doxygen-1.5.2/src/portable.cpp? I'll find out when I've finished
fetching the latest alt-prefix overlay.

I don't know what the "Inappropriate ioctl for device" error would have
been caused by. Perhaps bzcat expected to be able to seek in the stream.
-- 
gentoo-alt@g.o mailing list