bugConcurrent Versions System - Bugs: bug #35267, cvs should use diff --label, not...

 
 

bug #35267: cvs should use diff --label, not diff -L

Submitted by:  Paul Eggert <eggert>
Submitted on:  Sun 08 Jan 2012 08:25:00 PM UTC  
 
Category:  Bug Fix (patch attached) Severity:  3 - Normal
Item Group:  None Status:  None
Privacy:  Public Assigned to:  None
Open/Closed:  Open Release: 
Fixed Release:  None Fixed Feature Release:  None

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Thu 03 May 2018 02:15:50 PM UTC, comment #1:

I spotted more places where -L option is passed to diff. Attached patch covers them all and it passed all tests for me.

(file #44075)

Petr Pisar <petrp>
Sun 08 Jan 2012 08:25:00 PM UTC, original submission:

Currently CVS uses GNU diff's -L option.
That option has been undocumented since diffutils 2.8
(released in March 2002) and the option is intended to
be replaced sometime soon with a different meaning.

Can you please fix CVS so that it uses "diff --label"
instead of "diff -L"? That will make CVS immune to this
planned change. Thanks.

A proposed (but untested) patch is attached.

Paul Eggert <eggert>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #44075:  cvs-1.11.23-Use-diff-label.patch added by petrp (3KiB - application/octet-stream - More extensive fix)
file #24768:  cvs-patch.txt added by eggert (2KiB - text/plain - proposed patch for -L -> --label)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by petrp (Updated the item)
  • -email is unavailable- added by eggert (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

     

     

    Follow 2 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2018-05-03 petrp Attached File- => Added cvs-1.11.23-Use-diff-label.patch, #44075
    2012-01-08 eggert Attached File- => Added cvs-patch.txt, #24768

    Back to the top


    Powered by Savane 3.3