Re: [PATCH] core.filemode may need manual action

From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Oct 16, 2014, 19:47 UTC
Message-ID: <alpine.DEB.1.00.1410162146140.990@s15462909.onlinehome-server.info>
In-Reply-To: <54401F38.7080101@virtuell-zuhause.de>

Hi,

On Thu, 16 Oct 2014, Thomas Braun wrote:

Show 55 quoted lines

> Am 16.10.2014 um 21:29 schrieb Torsten Bögershausen:
> > core.filemode is set automatically when a repo is created.
> > But when a repo is exported via CIFS or cygwin is mixed with Git for Windows
> > core.filemode may better be set manually to false.
> > Update and improve the documentation.
> > 
> > Helped-by: Junio C Hamano <gitster@pobox.com>
> > Signed-off-by: Torsten Bögershausen <tboegi@web.de>
> > ---
> > 
> > Does this reflect the discussion via email ?
> > Or is more tweaking needed ?
> > 
> > 
> >  Documentation/config.txt | 19 +++++++++++++++++--
> >  1 file changed, 17 insertions(+), 2 deletions(-)
> > 
> > diff --git a/Documentation/config.txt b/Documentation/config.txt
> > index 4333636..b4fea43 100644
> > --- a/Documentation/config.txt
> > +++ b/Documentation/config.txt
> > @@ -204,8 +204,23 @@ advice.*::
> >  --
> >  
> >  core.fileMode::
> > -	If false, the executable bit differences between the index and
> > -	the working tree are ignored; useful on broken filesystems like FAT.
> > +	Tells Git if the executable bit of files in the working tree
> > +	is to be honored.
> > +
> > +	Some filesystems lose the executable bit when a file that is
> > +	marked as executable is checked out, or checks out an
> > +	non-executable file with executable bit on.  "git init" and
> > +	"git clone" probe the filesystem to see if it records
> > +	executable bit correctly when they create a new repository
> > +	and this variable is automatically set as necessary.
> > +
> > +	A repository, however, may be on a filesystem that records
> > +	the filemode correctly, and this variable is set to 'true'
> > +	when created, but later may be made accessible from another
> > +	environment that loses the filemode (e.g. exporting ext4 via
> > +	CIFS mount, visiting a Cygwin managed repository with
> > +	MsysGit).  In such a case, it may be necessary to set this
> > +	variable to 'false'.
> >  	See linkgit:git-update-index[1].
> >  +
> >  The default is true, except linkgit:git-clone[1] or linkgit:git-init[1]
> > 
> 
> [CC'ing msysgit aka git-for-windows/sdk for input]
> 
> I'm not really happy with the term MsysGit here.
> Would it be too bold to say
> [... ] visiting a Cygwin managed repository with Git for Windows.
> ?

I agree that msysGit is the wrong term. Not only is it about to be replaced by the Git for Windows SDK, it is *actively* wrong because msysGit is just the *development environment* to build Git for Windows. Most users do *not* need msysGit at all.

Ciao, Dscho

-- 
-- 
*** Please reply-to-all at all times ***
*** (do not pretend to know who is subscribed and who is not) ***
*** Please avoid top-posting. ***
The msysGit Wiki is here: https://github.com/msysgit/msysgit/wiki - Github accounts are free.

You received this message because you are subscribed to the Google
Groups "msysGit" group.
To post to this group, send email to msysgit@googlegroups.com
To unsubscribe from this group, send email to
msysgit+unsubscribe@googlegroups.com
For more options, and view previous threads, visit this group at
http://groups.google.com/group/msysgit?hl=en_US?hl=en

--- 
You received this message because you are subscribed to the Google Groups "Git for Windows" group.
To unsubscribe from this group and stop receiving emails from it, send an email to msysgit+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.