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 May 9 17:25:35 2004  
Category: NoneSeverity: 3 - Normal
Priority: 5 - NormalStatus: None
Privacy: PublicAssigned to: Vincent Caron <zerodeux>
Originator Email: -unavailable-Open/Closed: Open

Thu Feb 15 14:05:10 2007, 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 Jun 11 09:37:45 2004, 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 May 9 17:25:35 2004, 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).



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.


    Error: not logged in



    Follow 2 latest changes.

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

    Back to the top

    Powered by Savane 3.1-cleanup