public inbox for patchutils-list@sourceware.org
 help / color / mirror / Atom feed
* Re: Support for index data by splitdiff command
@ 2009-10-10 19:10 SF Markus Elfring
  2009-10-12 22:27 ` Sergio Monteiro Basto
  2010-11-26  3:02 ` Sergio Monteiro Basto
  0 siblings, 2 replies; 5+ messages in thread
From: SF Markus Elfring @ 2009-10-10 19:10 UTC (permalink / raw)
  To: Sergio Monteiro Basto; +Cc: patchutils-list

> filterdiff cleans all things that don't belongs to patch and that
> includes Index ...

I would prefer to keep all informations intact. Is still any fine-tuning needed for the deletion of unwanted source lines?

The command "cvs diff -u" generates also this data together with the field "RCS file:" which will be lost, too. Would you like to consider such details as supported extensions for the patch data format?

Regards,
Markus

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Support for index data by splitdiff command
  2009-10-10 19:10 Support for index data by splitdiff command SF Markus Elfring
@ 2009-10-12 22:27 ` Sergio Monteiro Basto
  2010-11-26  3:02 ` Sergio Monteiro Basto
  1 sibling, 0 replies; 5+ messages in thread
From: Sergio Monteiro Basto @ 2009-10-12 22:27 UTC (permalink / raw)
  To: SF Markus Elfring; +Cc: patchutils-list

[-- Attachment #1: Type: text/plain, Size: 660 bytes --]

On Sat, 2009-10-10 at 21:10 +0200, SF Markus Elfring wrote: 
> > filterdiff cleans all things that don't belongs to patch and that
> > includes Index ...
> 
> I would prefer to keep all informations intact. 

> Is still any fine-tuning needed for the deletion of unwanted source lines?

I don't understand the question ... , filterdiff ?! 

> The command "cvs diff -u" generates also this data together with the field "RCS file:" which will be lost, too. 

> Would you like to consider such details as supported extensions for the patch data format?

I can't answer on that, I am not the maintainer of sources. 

Regards, 
-- 
Sérgio M. B.

[-- Attachment #2: smime.p7s --]
[-- Type: application/x-pkcs7-signature, Size: 2192 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Support for index data by splitdiff command
  2009-10-10 19:10 Support for index data by splitdiff command SF Markus Elfring
  2009-10-12 22:27 ` Sergio Monteiro Basto
@ 2010-11-26  3:02 ` Sergio Monteiro Basto
  1 sibling, 0 replies; 5+ messages in thread
From: Sergio Monteiro Basto @ 2010-11-26  3:02 UTC (permalink / raw)
  To: SF Markus Elfring; +Cc: patchutils-list

[-- Attachment #1: Type: text/plain, Size: 654 bytes --]

On Sat, 2009-10-10 at 21:10 +0200, SF Markus Elfring wrote:
> > filterdiff cleans all things that don't belongs to patch and that
> > includes Index ...
> 
> I would prefer to keep all informations intact. Is still any fine-tuning needed for the deletion of unwanted source lines?
> 
> The command "cvs diff -u" generates also this data together with the field "RCS file:" which will be lost, too. Would you like to consider such details as supported extensions for the patch data format?
> 
> Regards,
> Markus


http://cyberelk.net/tim/software/patchutils/
Fixcvsdiff is for correcting the output of ‘cvs diff’.

-- 
Sérgio M. B.

[-- Attachment #2: smime.p7s --]
[-- Type: application/x-pkcs7-signature, Size: 3309 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Support for index data by splitdiff command
       [not found] <1254885773.10925.ezmlm@sourceware.org>
@ 2009-10-07  3:57 ` Sergio Monteiro Basto
  0 siblings, 0 replies; 5+ messages in thread
From: Sergio Monteiro Basto @ 2009-10-07  3:57 UTC (permalink / raw)
  To: patchutils-list; +Cc: Markus Elfring

[-- Attachment #1: Type: text/plain, Size: 2219 bytes --]

Hi sorry for the SPAM , my fault , elmz is too ... , last try ( I promise :)) 

Hi, 
Don't know if it help, I use svn and I use filterdiff.
filterdiff cleans all things that don't belongs to patch and that
includes Index ...
If you just want a clean splitdiff, you may use : 

svn diff | filterdiff  > txt; splitdiff -a txt


On Wed, 2009-10-07 at 03:22 +0000, patchutils-list-help@sourceware.org
wrote:
>         Hello,
>         
>         I would like to point out an open issue for the tool
>         "splitdiff" from the software package "patchutils 0.3.0-1.52".
>         
>         A special header "Index:" is mentioned in the article
>         "http://en.wikipedia.org/wiki/Diff". Tools like Subversion
>         write it into difference files. The current command does not
>         handle this data correctly.
>         
>         The following input ...
>         
>         Index: abc.h
>         ===================================================================
>         --- abc.h       (Revision 123)
>         +++ abc.h       (work copy)
>         @@ -1,1 +1,1 @@
>         -check
>         +out
>         Index: xyz.h
>         ===================================================================
>         --- xyz.h       (Revision 987)
>         +++ xyz.h       (work copy)
>         @@ -1,1 +1,1 @@
>         -test
>         +this
>         
>         ... is converted into the following arrangement by "splitdiff
>         -a".
>         
>         First source file:
>         --- abc.h       (Revision 123)
>         +++ abc.h       (work copy)
>         @@ -1,1 +1,1 @@
>         -check
>         +out
>         Index: xyz.h
>         ===================================================================
>         
>         Second source file:
>         --- xyz.h       (Revision 987)
>         +++ xyz.h       (work copy)
>         @@ -1,1 +1,1 @@
>         -test
>         +this
>         
>         
>         I would appreciate if the first file will keep its own header
>         and will not additionally get informations from the second
>         file.
>         
>         Regards,
>         Markus
-- 
Sérgio M. B.

[-- Attachment #2: smime.p7s --]
[-- Type: application/x-pkcs7-signature, Size: 2192 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Support for index data by splitdiff command
@ 2009-09-30 17:11 SF Markus Elfring
  0 siblings, 0 replies; 5+ messages in thread
From: SF Markus Elfring @ 2009-09-30 17:11 UTC (permalink / raw)
  To: patchutils-list

Hello,

I would like to point out an open issue for the tool "splitdiff" from the software package "patchutils 0.3.0-1.52".

A special header "Index:" is mentioned in the article "http://en.wikipedia.org/wiki/Diff". Tools like Subversion write it into difference files. The current command does not handle this data correctly.

The following input ...

Index: abc.h
===================================================================
--- abc.h	(Revision 123)
+++ abc.h	(work copy)
@@ -1,1 +1,1 @@
-check
+out
Index: xyz.h
===================================================================
--- xyz.h	(Revision 987)
+++ xyz.h	(work copy)
@@ -1,1 +1,1 @@
-test
+this

... is converted into the following arrangement by "splitdiff -a".

First source file:
--- abc.h	(Revision 123)
+++ abc.h	(work copy)
@@ -1,1 +1,1 @@
-check
+out
Index: xyz.h
===================================================================

Second source file:
--- xyz.h	(Revision 987)
+++ xyz.h	(work copy)
@@ -1,1 +1,1 @@
-test
+this


I would appreciate if the first file will keep its own header and will not additionally get informations from the second file.

Regards,
Markus

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2010-11-26  3:02 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-10-10 19:10 Support for index data by splitdiff command SF Markus Elfring
2009-10-12 22:27 ` Sergio Monteiro Basto
2010-11-26  3:02 ` Sergio Monteiro Basto
     [not found] <1254885773.10925.ezmlm@sourceware.org>
2009-10-07  3:57 ` Sergio Monteiro Basto
  -- strict thread matches above, loose matches on Subject: below --
2009-09-30 17:11 SF Markus Elfring

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).