From mboxrd@z Thu Jan  1 00:00:00 1970
From: cdrice at pobox.com (Charles Dee Rice)
Date: Sat, 13 Dec 2014 19:28:23 +0000 (UTC)
Subject: [PATCH v2] Remove trailing slash after remove-suffix
In-Reply-To: <1418470251-7909-1-git-send-email-cgit@cryptocrack.de>
References: <1418470251-7909-1-git-send-email-cgit@cryptocrack.de>
Message-ID: <1089416287.277235.1418498903540.JavaMail.yahoo@jws10633.mail.bf1.yahoo.com>

Thank you so much for the quick reply!

> The previous version had a flaw (bogus pointer if there is no ".git" 
> suffix). This one is much more readable add should be applied on top of 
> the Git 2.2.0 patch (which adds strip_suffix() and strip_suffix_mem()). 

Version 1 would definitely cause some issues.  My brief testing confirmed it could cut off the last letter of the repot name for a bare repo -- "blah.git" became "bla".  It did work for non-bare repos, though -- "blah/.git" became "blah" and worked!  ;) 


Applying the latest version of Christian's git 2.2.0 patch from November + your version 2 looks like it works perfectly! 

I now see both bare and non-bare repos working with scanning and remove-suffix enabled.  All repo features (refs, log, tree, etc) work correctly in each case.


This really had me scratching my head -- I'm glad to see I wasn't doing something silly after all!   Thanks again! 


- Chuck