git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] cvsserver: fix "cvs diff" in a subdirectory
@ 2006-10-10 22:33 Johannes Schindelin
  2006-10-11  1:01 ` Junio C Hamano
  0 siblings, 1 reply; 4+ messages in thread
From: Johannes Schindelin @ 2006-10-10 22:33 UTC (permalink / raw)
  To: Martyn Smith, Martin Langhoff; +Cc: git


Signed-off-by: Johannes Schindelin <Johannes.Schindelin@gmx.de>
---
 git-cvsserver.perl |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/git-cvsserver.perl b/git-cvsserver.perl
index 7260f32..da00432 100755
--- a/git-cvsserver.perl
+++ b/git-cvsserver.perl
@@ -277,7 +277,7 @@ sub req_Directory
     $state->{directory} = "" if ( $state->{directory} eq "." );
     $state->{directory} .= "/" if ( $state->{directory} =~ /\S/ );
 
-    if ( not defined($state->{prependdir}) and $state->{localdir} eq "." and $state->{path} =~ /\S/ )
+    if ( (not defined($state->{prependdir}) or $state->{prependdir} eq '') and $state->{localdir} eq "." and $state->{path} =~ /\S/ )
     {
         $log->info("Setting prepend to '$state->{path}'");
         $state->{prependdir} = $state->{path};
-- 
1.4.3.rc2.g55b2-dirty

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] cvsserver: fix "cvs diff" in a subdirectory
  2006-10-10 22:33 [PATCH] cvsserver: fix "cvs diff" in a subdirectory Johannes Schindelin
@ 2006-10-11  1:01 ` Junio C Hamano
  2006-10-11  8:17   ` Johannes Schindelin
  2006-10-23  6:11   ` Martin Langhoff (CatalystIT)
  0 siblings, 2 replies; 4+ messages in thread
From: Junio C Hamano @ 2006-10-11  1:01 UTC (permalink / raw)
  To: Johannes Schindelin; +Cc: git, Martyn Smith, Martin Langhoff

Will park this and the "U for update, M for modified" patch in
"pu", waiting for acks from actual git-cvsserver users.  It
feels it is safe enough change, so with an Ack from Mart[yi]n
it may be woth having it in 1.4.3

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] cvsserver: fix "cvs diff" in a subdirectory
  2006-10-11  1:01 ` Junio C Hamano
@ 2006-10-11  8:17   ` Johannes Schindelin
  2006-10-23  6:11   ` Martin Langhoff (CatalystIT)
  1 sibling, 0 replies; 4+ messages in thread
From: Johannes Schindelin @ 2006-10-11  8:17 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: git, Martyn Smith, Martin Langhoff

Hi,

On Tue, 10 Oct 2006, Junio C Hamano wrote:

> Will park this and the "U for update, M for modified" patch in
> "pu", waiting for acks from actual git-cvsserver users.  It
> feels it is safe enough change, so with an Ack from Mart[yi]n
> it may be woth having it in 1.4.3

Thanks.

Ciao,
Dscho

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] cvsserver: fix "cvs diff" in a subdirectory
  2006-10-11  1:01 ` Junio C Hamano
  2006-10-11  8:17   ` Johannes Schindelin
@ 2006-10-23  6:11   ` Martin Langhoff (CatalystIT)
  1 sibling, 0 replies; 4+ messages in thread
From: Martin Langhoff (CatalystIT) @ 2006-10-23  6:11 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: Johannes Schindelin, git, Martyn Smith

Junio C Hamano wrote:

> Will park this and the "U for update, M for modified" patch in
> "pu", waiting for acks from actual git-cvsserver users.  It
> feels it is safe enough change, so with an Ack from Mart[yi]n
> it may be woth having it in 1.4.3

Late late Ack. Sorry about the delay. I see it's in master already ;-)

cheers,


martin
-- 
-----------------------------------------------------------------------
Martin @ Catalyst .Net .NZ  Ltd, PO Box 11-053, Manners St,  Wellington
WEB: http://catalyst.net.nz/           PHYS: Level 2, 150-154 Willis St
OFFICE: +64(4)916-7224                              MOB: +64(21)364-017
       Make things as simple as possible, but no simpler - Einstein
-----------------------------------------------------------------------

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2006-10-23  6:11 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-10-10 22:33 [PATCH] cvsserver: fix "cvs diff" in a subdirectory Johannes Schindelin
2006-10-11  1:01 ` Junio C Hamano
2006-10-11  8:17   ` Johannes Schindelin
2006-10-23  6:11   ` Martin Langhoff (CatalystIT)

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).