Re: current directory (...) contains an invalid character (+,>;=\t\n)

[Larry Jones]

Rolf Krahl writes:
> 
> Hmm, if i'm guessing right that the _editors entry if present always
> contains the three subfields date, host, and cwd, then i think one
> could drop the limitation at least for the '+' character in the cwd (i
> think its the one that will most likely appear in actual file names)
> and still parse this file correctly.  Maybe i'll have a look in the
> CVS sources if i find some spare hours to spent.  Do you think this
> was worth the effort to try and code a fix?  (E.g. do you think such a
> patch would have a chance to be accepted?)

I'm not sure I'd want to have the rules for _editors be different than
the rules for _watchers since they're very closely related.  It may be
possible to enhance the code so that it parses entries correctly with
fewer restrictions, but I warn you that the code is convoluted and not
well documented, which means it's not only going to be hard to fix, it's
going to be hard to convince me that it really is fixed.  But I'm
willing to accept a patch if you can convince me -- the current
restrictions really are a bit draconian.

> Thanks for the tip.  Indeed i just checked out thats the current
> working directory while invoking the edit command that counts, not the
> work directory where the file in question resides.

That's interesting -- I wonder if that's a bug?

-Larry Jones

Santa's gonna skip this block for years. -- Calvin