Junio C Hamano [off-list ref] writes:
It is probably unlikely to see a SP in the pathname, but I do not think it
is reasonable to introduce a regression to forbid '=' in the pathname to
the repository, which we have been supporting since August 2009, when we
know the patch as-is will regress the use case, and especially when we
already know a way to code not to regress is not too complex.
Even though I don't deeply care about what CVSNT does, but I am somewhat
curious why this "change SP to =" was done when CVSNT writes out its
$HOME/.cvs/cvspass file.
Does anybody know why? Only to make things incompatible, perhaps? ;-)
http://www.selenic.com/pipermail/mercurial/2009-April/025095.html
seems to indicate that somebody next door had a similar experience on
exactly the same issue.
On Tue, Feb 22, 2011 at 6:08 PM, Junio C Hamano [off-list ref] wrote:
Even though I don't deeply care about what CVSNT does...
...
Does anybody know why? Only to make things incompatible, perhaps? ;-)
A brief googling around shows that it also stores it in the Windows registry.
Should we support that too...? ;-)
m
--
martin@laptop.org -- Software Architect - OLPC
- ask interesting questions
- don't get distracted with shiny stuff - working code first
- http://wiki.laptop.org/go/User:Martinlanghoff
On 2/22/2011 6:50 PM, Martin Langhoff wrote:
On Tue, Feb 22, 2011 at 6:08 PM, Junio C Hamano[off-list ref] wrote:
quoted
Even though I don't deeply care about what CVSNT does...
..
quoted
Does anybody know why? Only to make things incompatible, perhaps? ;-)
A brief googling around shows that it also stores it in the Windows registry.
Should we support that too...? ;-)
One thing at a time, Martin :). After I get this patch through, I want
to start working on getting the rest of the Perl script to run under
Windows. I was almost there; the biggest issue is that Perl
implementations (ActiveState, Strawberry) for Windows don't support the
list form of open. I converted most of them successfully, but got stuck
on one so decided to submit this patch first. Thank goodness I did this
separately :).
To answer Junio's question, I'm looking at the CVSNT code now
(GlobalSettings.cpp, if anyone is interested.) The password is stored
in a general fashion like any other user-specified value. So, the
authors elected to use a properties file format of key=value. That is
as valid a format as any other.
--
Guy Rouillier