Tracker: Patches

5 Add support of filterdiff - ID: 1755623
Last Update: Comment added ( cattelan )

Many time patches come from various source (email for example)

This is a quick patch to run patches through fiterdiff such that
the rest of the CS patch parsing engine can handle the patch.

Russell Cattelan ( cattelan ) - 2007-07-17 10:54:34 PDT

5

Open

None

Nobody/Anonymous

None

None

Public


Comments ( 2 )

Date: 2007-08-30 08:15:54 PDT
Sender: cattelan


I understand installing filterdiff may be bit of a pain.

An all perl solutions might be nice.
maybe we should look at packages like Text-Diff-Parser
I have not looked to see how well it will parse up a patch that has extra
cruft in it.

If you want to test this simply email a patch to yourself save the email
and then
try importing that email message into CS.

You might ask why not save the patch and then import that.
Since CS will save the whole file as the topic text any useful info that
might be in the email is then archived as part of the CS topic.

Its also easier for us lazy people :-)

Date: 2007-08-25 01:42:47 PDT
Sender: sitsProject Admin


I like the idea, but this is hard to execute since we need to also support
win32. A lot of sites would then need to install filterdiff, or at least
make it an option.

Ideally, I'd like to make Codestriker's parser handle these "email diffs" -
can you provide some examples? Thanks.

Attached File ( 1 )

Filename Description Download
parser.patch Download

Change ( 1 )

Field Old Value Date By
File Added 237376: parser.patch 2007-07-17 10:54:34 PDT cattelan