[cvs] commit: hordedoc/hordeadmin/work Deploying IMP.rtf

[Jon Parise]

On Thu, Dec 19, 2002 at 02:10:20PM -0600, Eric Rostetter wrote:

> >   Chora Links:
> >   http://cvs.horde.org/diff.php/?r1=.-1&r2=No&ty=u
> >   http://cvs.horde.org/diff.php/?r1=.-1&r2=No&ty=u
> 
> Obviously those chora links are bogus.  I've seen this before, and while I've
> not checked it appears to happen with the file names have "strange" characters
> or whitespace in them.  Anyone care to look at the cvs scripts and see if
> they can maybe fix this (maybe quote the filenames or something)?

The whitespace is the main culprit.  I've know about this problem for
a while, but it comes up so infrequently, it never became a priority.
 
Coincidentally, I started rewriting the CVSROOT scripts in Python the
other day (sort of as an exercise).  I haven't gotten to the
log_accum.pl script yet, but when I do, I'll be sure to give this
problem some attention.

I've looked at it before, by the way, and there wasn't an "easy" fix.
Essentially, filenames are passed around in the log_accum.pl script as
a space-separated list, so some serious changes are in order.

-- 
Jon Parise (jon@horde.org) :: The Horde Project (http://horde.org/)