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

 
 

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

Submitter:  Paul Eggert <eggert>
Submitted:  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
* Mandatory Fields

Add a New Comment Rich Markup
   

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)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    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.13-4448.
    Corresponding source code