GIT_ONE_FILESYSTEM inclusion?

Hi all,

Just wondering if anyone has any further comments or suggestions for the GIT_ONE_FILESYSTEM patches I posted. See here for context:

    http://thread.gmane.org/gmane.comp.version-control.git/142436

If it looks good I'd love to have it considered for inclusion.

Thanks! -lars

-- lars r. damerow :: button pusher :: pixar animation studios

On Wed, Mar 24, 2010 at 00:20, Lars Damerow <lars@pixar.com> wrote:

Have considered making it an option to git wrapper as suggested (-x/--one-filesystem)? And while at it, how about making it a config option? (i.e. core.oneFilesystem?)

Am 3/24/2010 10:29, schrieb Alex Riesen:

It's difficult to have this as a config option when you are looking for a repository... ;)

-- Hannes

On Wed, Mar 24, 2010 at 10:35, Johannes Sixt <j.sixt@viscovery.net> wrote:

Isn't it a little pointless to look for repository when you can't access its config?

If you're doing the trendy thing of putting repository information in your shell prompt, then your prompt is looking for a repository every time you change a directory.

As for making this a command-line option: that's certainly doable, but it makes it harder to enforce the option's use across an organization. It's far easier for me to set an environment variable centrally on a thousand workstations than to convince a potentially large number of users to set a new shell alias for git.

thanks, -lars

-- lars r. damerow :: button pusher :: pixar animation studios

On Wed, Mar 24, 2010 at 14:40, Lars Damerow <lars@pixar.com> wrote:

Right. OTOH, if you know that your .git is accessible (on the right filesystem), but want to avoid scans in the directories below the top- level working directory (submodules?), you may want to have it configurable per-repository. The environment variable will affect scan in all your repositories you visit in this session, which maybe what you want, and maybe not. And yes, the config option is also in addition to, not instead of the environment variable.

Oh, I wasn't clear. I didn't mean "instead of". A command-line option _in_addition_ to the environment variable. For one-off uses.

Alex Riesen <raa.lkml@gmail.com> writes:

I may have been the one who mentioned "-x to git potty" as a possibility, but I agree with Lars---I think it would be too cumbersome to use in practice.

When would you invoke this feature as "one-off"?

On Wed, Mar 24, 2010 at 17:33, Junio C Hamano <gitster@pobox.com> wrote:

Maybe when I want to make sure git status does not go all over an NFS mounted submodule.

But frankly, I think it will be the most obscure feature of Git ever, with a command-line option or without. I just wanted to suggest (and actually, just should have, stupid me) to keep it in line with other similar configurations, which are present in all three forms (work tree comes to mind... Come to think of it, only the work tree).

Alex Riesen <raa.lkml@gmail.com> writes:

But then what would you do after finding it out that it might? You stop working on that project? Doesn't --ignore-submodules do the same thing?

On Wed, Mar 24, 2010 at 21:04, Junio C Hamano <gitster@pobox.com> wrote:

Postpone whatever you were about to do, maybe.

It does. I said it is obscure.

Wow. Would I win a prize or something? :)

-- lars r. damerow :: button pusher :: pixar animation studios

Alex Riesen <raa.lkml@gmail.com> writes:

I guess Johannes's point is that the config option can be in $GIT_DIR/config, which you can't access when looking for $GIT_DIR. OTOH, looking at ~/.gitconfig to find a core.oneFilesystem makes sense (but is probably painfull to implement since the current code to parse config file will look for $GIT_DIR first).

On Wed, Mar 24, 2010 at 02:43:26PM +0100, Matthieu Moy wrote:

Right.

As this thread showed, we can't have a config option for this feature. Were there any other objections to the patch? It looks good to me.

Peff also agreed that it was an overall improvement. http://thread.gmane.org/gmane.comp.version-control.git/142262

Thread with the latest (v3) version of the patch: http://thread.gmane.org/gmane.comp.version-control.git/142436

We could use this feature at Disney Animation as well. We maintain a list of $GIT_CEILING_DIRECTORIES and could do without having to maintain it if we had this feature.

Plus, Lars is one of the git users that doesn't even flinch when the Perforce servers at Pixar get a hiccup, so we should try and help our fellow "renegade" git users ;-)

On Sat, Mar 27, 2010 at 03:27:32PM -0700, David Aguilar wrote:

Yep. I had been meaning to review v3, and I just did. With the exception of a few minor nits (which I posted in that thread), I think it looks good.

-Peff