Re: [PATCH v3 0/8] Fix GIT_CEILING_DIRECTORIES that contain symlinks

2013-02-20 Thread Anders Kaseorg
On Tue, 19 Feb 2013, Junio C Hamano wrote: > Assuming that this says "yes": > > D=/afs/athena.mit.edu/user/a/n/andersk/my/dir > cd "$D" > test "$(/bin/pwd)" = "$D" && echo yes Correct. > Perhaps existing of an empty element in the list would do? E.g. > > GIT_CEILING

Re: [PATCH v3 0/8] Fix GIT_CEILING_DIRECTORIES that contain symlinks

2013-02-19 Thread Junio C Hamano
Anders Kaseorg writes: > On 10/29/2012 01:10 AM, Michael Haggerty wrote: >> How do you use GIT_CEILING_DIRECTORIES that the proposed changes cause a >> slowdown? > > Sorry to bring up this old thread again, but I just realized why my > computer has been acting so slow when I’m not connected to th

Re: [PATCH v3 0/8] Fix GIT_CEILING_DIRECTORIES that contain symlinks

2013-02-19 Thread Anders Kaseorg
On 10/29/2012 01:10 AM, Michael Haggerty wrote: How do you use GIT_CEILING_DIRECTORIES that the proposed changes cause a slowdown? Sorry to bring up this old thread again, but I just realized why my computer has been acting so slow when I’m not connected to the network. I put various network

Re: [PATCH v3 0/8] Fix GIT_CEILING_DIRECTORIES that contain symlinks

2012-11-15 Thread Michael Haggerty
On 11/13/2012 09:50 PM, David Aguilar wrote: > On Mon, Nov 12, 2012 at 9:47 AM, Junio C Hamano wrote: >> Michael Haggerty writes: >> >>> The log message of the original commit (0454dd93bf) described the >>> following scenario: a /home partition under which user home directories >>> are automounte

Re: [PATCH v3 0/8] Fix GIT_CEILING_DIRECTORIES that contain symlinks

2012-11-13 Thread David Aguilar
On Mon, Nov 12, 2012 at 9:47 AM, Junio C Hamano wrote: > Michael Haggerty writes: > >> The log message of the original commit (0454dd93bf) described the >> following scenario: a /home partition under which user home directories >> are automounted, and setting GIT_CEILING_DIRECTORIES=/home to avoi

Re: [PATCH v3 0/8] Fix GIT_CEILING_DIRECTORIES that contain symlinks

2012-11-12 Thread Junio C Hamano
Michael Haggerty writes: > The log message of the original commit (0454dd93bf) described the > following scenario: a /home partition under which user home directories > are automounted, and setting GIT_CEILING_DIRECTORIES=/home to avoid > hitting /home/.git, /home/.git/objects, and /home/objects

Re: [PATCH v3 0/8] Fix GIT_CEILING_DIRECTORIES that contain symlinks

2012-10-28 Thread Lars Damerow
>From David Aguilar , Sun, Oct 28, 2012 at 05:15:29PM -0700: > > In 8030e44215fe8f34edd57d711a35f2f0f97a0423 Lars added > GIT_ONE_FILESYSTEM to fix a related issue. > Do you guys have GIT_CEILING_DIRECTORIES set too? Once GIT_DISCOVERY_ACROSS_FILESYSTEM (the eventual name for GIT_ONE_FILESYSTEM)

Re: [PATCH v3 0/8] Fix GIT_CEILING_DIRECTORIES that contain symlinks

2012-10-28 Thread Michael Haggerty
On 10/29/2012 01:15 AM, David Aguilar wrote: > On Sat, Oct 20, 2012 at 11:51 PM, Junio C Hamano wrote: >> Michael Haggerty writes: >> >>> This patch series has the side effect that all of the directories >>> listed in GIT_CEILING_DIRECTORIES are accessed *unconditionally* to >>> resolve any symli

Re: [PATCH v3 0/8] Fix GIT_CEILING_DIRECTORIES that contain symlinks

2012-10-28 Thread Junio C Hamano
David Aguilar wrote: >Is there another way to accomplish this without the performance hit? Perhaps not canonicalize elements on the CEILING list ourselves? If we make it a user error to put symlinked alias in the variable, and document it clearly, wouldn't it suffice? -- To unsubscribe from

Re: [PATCH v3 0/8] Fix GIT_CEILING_DIRECTORIES that contain symlinks

2012-10-28 Thread David Aguilar
On Sat, Oct 20, 2012 at 11:51 PM, Junio C Hamano wrote: > Michael Haggerty writes: > >> This patch series has the side effect that all of the directories >> listed in GIT_CEILING_DIRECTORIES are accessed *unconditionally* to >> resolve any symlinks that are present in their paths. It is >> admit

Re: [PATCH v3 0/8] Fix GIT_CEILING_DIRECTORIES that contain symlinks

2012-10-22 Thread Michael Haggerty
On 10/21/2012 08:51 AM, Junio C Hamano wrote: > Michael Haggerty writes: > >> This patch series has the side effect that all of the directories >> listed in GIT_CEILING_DIRECTORIES are accessed *unconditionally* to >> resolve any symlinks that are present in their paths. It is >> admittedly odd

Re: [PATCH v3 0/8] Fix GIT_CEILING_DIRECTORIES that contain symlinks

2012-10-20 Thread Junio C Hamano
Michael Haggerty writes: > This patch series has the side effect that all of the directories > listed in GIT_CEILING_DIRECTORIES are accessed *unconditionally* to > resolve any symlinks that are present in their paths. It is > admittedly odd that a feature intended to avoid accessing expensive >