From: "Torsten Bögershausen" <tboegi@web.de>
To: "Junio C Hamano" <gitster@pobox.com>,
"Ævar Arnfjörð Bjarmason" <avarab@gmail.com>
Cc: Duy Nguyen <pclouds@gmail.com>,
Christian Couder <christian.couder@gmail.com>,
Git <git@vger.kernel.org>, Jeff King <peff@peff.net>,
David Turner <dturner@twopensource.com>,
Christian Couder <chriscool@tuxfamily.org>
Subject: Re: [RFC/PATCH] config: add core.trustmtime
Date: Tue, 01 Dec 2015 06:57:05 +0100 [thread overview]
Message-ID: <565D36B1.9050104@web.de> (raw)
In-Reply-To: <xmqq610j8fzt.fsf@gitster.mtv.corp.google.com>
On 11/30/2015 08:05 PM, Junio C Hamano wrote:
> Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:
>
>> Maybe I'm missing some really common breakage with st_mtime on some
>> system, but having a feature the user explicitly enables turn itself
>> off and doing FS-testing that takes 10 seconds when it's enabled seems
>> like the wrong default to me.
>>
>> We don't do it with core.fileMode, core.ignorecase or core.trustctime
>> or core.symlinks. Do we really need to be treating this differently?
> I share the exact thought. I was looking the other way when
> untracked-cache was done originally ;-), and I would also want to
> know the answers to the above questions.
Maybe somewhat off topic, but:
Create a repo under Linux:
Linux will probe the FS and will set
core.filemode true
core.ignorecase false.
Export it using SAMBA to Windows or Mac OS X.
In some magical way the mounted repo becomes case-insensitive under
both Mac and Windows, even if core.ignorecase is true in the config file.
Same for filemode:
(Git for Windows doesn't see the execute bit at all, but relies on
core.filemode anyway)
Depending how the repo is mounted on the Mac, the execute bits may work,
be always 0 or always 1.
Relying on this kind of config files is not ideal in a networked
environment.
It is not ideal when different implementations of Git access the same repo,
(Git for Windows vs cygwin vs Egit/Jgit)
So I think that the original patch could make Git unreliable under some
circumstances.
And some day I may send a patch which does a quick auto-probe for
filemode and ignorecase,
to adjust Git to what the underlying "local OS - network - remote OS -
remote FS" really achieves.
next prev parent reply other threads:[~2015-12-01 5:58 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-25 6:35 [RFC/PATCH] config: add core.trustmtime Christian Couder
2015-11-25 9:00 ` Ævar Arnfjörð Bjarmason
2015-11-25 19:51 ` Duy Nguyen
2015-11-26 5:21 ` Christian Couder
2015-11-26 17:53 ` Duy Nguyen
2015-11-27 1:35 ` Ævar Arnfjörð Bjarmason
2015-11-30 19:05 ` Junio C Hamano
2015-11-30 19:12 ` Duy Nguyen
2015-12-01 5:57 ` Torsten Bögershausen [this message]
2015-12-02 19:28 ` Duy Nguyen
2015-12-07 5:42 ` Christian Couder
2015-11-25 10:25 ` Johannes Schindelin
2015-11-25 10:39 ` Christian Couder
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=565D36B1.9050104@web.de \
--to=tboegi@web.de \
--cc=avarab@gmail.com \
--cc=chriscool@tuxfamily.org \
--cc=christian.couder@gmail.com \
--cc=dturner@twopensource.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=pclouds@gmail.com \
--cc=peff@peff.net \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).