From: "Le" <le_wen@distributel.ca>
To: "Git Mailing List" <git@vger.kernel.org>
Subject: Bugs in sub argsfromdir of git-cvsserver
Date: Mon, 22 Dec 2008 15:33:45 -0500 [thread overview]
Message-ID: <494FF9A9.9060706@distributel.ca> (raw)
Hi, all,
I found a bug in git-cvsserver.
sub argsfromdir
{
if ( scalar(@{$state->{args}}) == 1 )
{
my $arg = $state->{args}[0];
#$arg .= $state->{prependdir} if ( defined (
$state->{prependdir} ) );
$arg = $state->{prependdir} . $arg if ( defined (
$state->{prependdir} ) );
$log->info("Only one arg specified, checking for directory
expansion on '$arg'");
...
}
...
}
It makes not sense of the above remarked out code.
But when I use the code followed remarked out, I have problem to use cvs
diff or cvs log in a sub directory of the root (may other commands as
well). Eg.:
I have a project called test1 and in test1 there is a test2 subdirectory
and a file test2/test3.txt.
I have no problem to cvs diff and cvs log in test1 directory. But if I
enter test1/test2 to check these commands then they are not working.
I tried to fixed the problem to remarked out the code:
#$filename = filecleanup($filename);
in req_log and req_diff, then both cvs diff and cvs log work.
In
sub filecleanup
{
...
$filename = $state->{prependdir} . $filename;
return $filename;
}
inserts $state->{prependdir} into the $filename.
But I don't know if what I did brings incompatibilities.
Can anybody check this issue?
Thanks!
Le Wen
next reply other threads:[~2008-12-22 20:35 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-12-22 20:33 Le [this message]
2008-12-22 21:42 ` Bugs in sub argsfromdir of git-cvsserver Le
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=494FF9A9.9060706@distributel.ca \
--to=le_wen@distributel.ca \
--cc=git@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.