[PATCH V2B] core.filemode may need manual action

[PATCH V2B] core.filemode may need manual action

[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
or Eclipse 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 makes more sence ?

 Documentation/config.txt | 23 ++++++++++++++++++-----
 1 file changed, 18 insertions(+), 5 deletions(-)

diff --git a/Documentation/config.txt b/Documentation/config.txt
index 04a1e2f..3127e5d 100644
--- a/Documentation/config.txt
+++ b/Documentation/config.txt
@@ -204,13 +204,26 @@ 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.
+	linkgit:git-clone[1] or linkgit:git-init[1] probe the filesystem
+	to see if it handles the executable bit correctly
+	and this variable is automatically set as necessary.
++
+	A repository, however, may be on a filesystem that handles
+	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 created repository with
+	Git for Windows or Eclipse).
+	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]
-will probe and set core.fileMode false if appropriate when the
-repository is created.
+The default is true (when core.filemode is not specified in the config file).
 
 core.ignorecase::
 	If true, this option enables various workarounds to enable
-- 
2.1.0.rc2.210.g636bceb

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

[Junio C Hamano]

Torsten Bögershausen <tboegi@web.de> writes:

> Does this makes more sence ?

Giving the links in-line in the text and avoiding to repeat the same
information at the end like this version does probably makes sense.
Good thinking.

Thanks.

>
>  Documentation/config.txt | 23 ++++++++++++++++++-----
>  1 file changed, 18 insertions(+), 5 deletions(-)
>
> diff --git a/Documentation/config.txt b/Documentation/config.txt
> index 04a1e2f..3127e5d 100644
> --- a/Documentation/config.txt
> +++ b/Documentation/config.txt
> @@ -204,13 +204,26 @@ 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.
> +	linkgit:git-clone[1] or linkgit:git-init[1] probe the filesystem
> +	to see if it handles the executable bit correctly
> +	and this variable is automatically set as necessary.
> ++
> +	A repository, however, may be on a filesystem that handles
> +	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 created repository with
> +	Git for Windows or Eclipse).
> +	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]
> -will probe and set core.fileMode false if appropriate when the
> -repository is created.
> +The default is true (when core.filemode is not specified in the config file).
>  
>  core.ignorecase::
>  	If true, this option enables various workarounds to enable