All of lore.kernel.org
 help / color / mirror / Atom feed
From: Junio C Hamano <gitster@pobox.com>
To: "Torsten Bögershausen" <tboegi@web.de>
Cc: Hilco Wijbenga <hilco.wijbenga@gmail.com>,
	Git Users <git@vger.kernel.org>
Subject: Re: [PATCH] init - Honour the global core.filemode setting
Date: Thu, 02 Oct 2014 10:02:02 -0700	[thread overview]
Message-ID: <xmqqzjdeo16d.fsf@gitster.dls.corp.google.com> (raw)
In-Reply-To: <542D33E1.6080709@web.de> ("Torsten Bögershausen"'s message of "Thu, 02 Oct 2014 13:15:45 +0200")

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

> On 2014-10-01 19.10, Junio C Hamano wrote:
>> Hilco Wijbenga <hilco.wijbenga@gmail.com> writes:
>> 
>>> Perhaps I completely misunderstand the meaning of core.filemode but I
>>> thought it determined whether Git cared about changes in file
>>> properties?
>> 
>> By setting it to "false", you tell Git that the filesystem you
>> placed the repository does not correctly represent the filemode
>> (especially the executable bit).
>> 
>> "core.fileMode" in "git config --help" reads:
>> 
>>        core.fileMode
>>            If false, the executable bit differences between the
>>            index and the working tree are ignored; useful on broken
>>            filesystems like FAT. See git-update- index(1).
>
> Out of my head: Could the following be a starting point:
>
>         core.fileMode
>             If false, the executable bit differences between the
>             index and the working tree are ignored.
>             This may be usefull when visiting a cygwin repo with a non-cygwin
>             Git client. (should we mention msysgit ? should we mention JGit/EGit ?)

Between these two sentences, there may still be the same cognitive
gap that may have lead to the original confusion.

The first sentence says what happens, as it should.

But it is not directly clear what makes the executable bit differ
and when it is a useful thing to ignore the differences, so the
second sentence that says "This may be useful" does not give the
reader very much.

Here is my attempt.

	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
        to 'false'.

  reply	other threads:[~2014-10-02 17:02 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-28  0:37 [PATCH] init - Honour the global core.filemode setting Hilco Wijbenga
2014-09-28 11:52 ` Torsten Bögershausen
2014-10-01  1:55   ` Hilco Wijbenga
2014-10-01 17:10     ` Junio C Hamano
2014-10-02 11:15       ` Torsten Bögershausen
2014-10-02 17:02         ` Junio C Hamano [this message]
2014-10-03 16:54           ` Torsten Bögershausen
2014-10-03 17:07             ` Junio C Hamano

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=xmqqzjdeo16d.fsf@gitster.dls.corp.google.com \
    --to=gitster@pobox.com \
    --cc=git@vger.kernel.org \
    --cc=hilco.wijbenga@gmail.com \
    --cc=tboegi@web.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.