bugcvsreport - Bugs: bug #374, Filenames with spaces are not...

 
 
Show feedback again

You are not allowed to post comments on this tracker with your current authentification level.

bug #374: Filenames with spaces are not handled correctly

Submitted by:  None
Submitted on:  Sun 09 May 2004 05:25:35 PM UTC  
 
Category: NoneSeverity: 3 - Normal
Priority: 5 - NormalStatus: None
Privacy: PublicAssigned to: Vincent Caron <zerodeux>
Originator Email: -unavailable-Open/Closed: Open

Thu 15 Feb 2007 02:05:10 PM UTC, comment #2:

I have implemented a fix for this problem in another way, still relying on the output from cvs.

Instead of splitting the output lines from cvs history, simply based on spaces, I use two passes to find the exact location of where to split the line.

The first pass, simply finds all the columns where we have a space in every line of output from history.

The second pass, we try to split the file/directory names based on these columns. Every column where we cannot find a filename that matches are removed.

This way we end up with a list of columns that we can use to split the history output after.

I have attached the patch.

(file #1936)

Klaus S. Madsen <hjernemadsen>
Fri 11 Jun 2004 09:37:45 AM UTC, comment #1:

This is a know problem, 'cvs history' returns file information in such a way that we can't parse it properly when there are spaces in its name, eg:

M 2004-02-10 17:45 +0000 zerodeux 1.56 ChangeLog cvsreport == <remote>
M 2004-02-10 17:45 +0000 zerodeux 1.3 MAINTAINER cvsreport == <remote>
M 2004-02-10 17:45 +0000 zerodeux 1.53 cvsreport cvsreport == <remote>

A possible fix (although not trivial to implement) is to fallback to parse CVSROOT/history when cvsreport is run on the server side (most common case). In this case it will only fails if you have '|' (pipes) in your file names, which is less bad.

Vincent Caron <zerodeux>
Project AdministratorIn charge of this item.
Sun 09 May 2004 05:25:35 PM UTC, original submission:

I have some files with spaces in their names in my repository and they are not handled correctly by cvsreport. The file name appears truncated in the report and cvsreport generates an error message about not finding the rcs file (although it's there and works fine as far as cvs itself is concerned).

Anonymous

 

Attached Files

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by hjernemadsen (Updated the item)
  • -unavailable- added by zerodeux (Updated the item)
  •  

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

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 2 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Thu 15 Feb 2007 02:05:10 PM UTChjernemadsenAttached File-=>Added space_in_filename_fix.patch, #1936
    Fri 11 Jun 2004 09:37:45 AM UTCzerodeuxAssigned toNone=>zerodeux
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup