Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

nanosecond file times for v2.5.3 #443

Merged
merged 2 commits into from
Sep 28, 2015

Conversation

kblees
Copy link

@kblees kblees commented Sep 26, 2015

Here are the nanosecond file time patches, rebased to v2.5.3.

While testing, I found a few nasty issues when using git variants with different file time resolutions on the same repository (e.g. Git for Windows and JGit/EGit) [1]. This is why the second patch does not currently enable USE_NSEC in the Makefile (use make USE_NSEC=1 to build with nanosecond support).

[1] http://article.gmane.org/gmane.comp.version-control.git/278683

fstat() is the only stat-related CRT function for which we don't have a
full replacement yet (and thus the only reason to stick with MSVCRT's
'struct stat' definition).

Fully implement fstat(), in preparation of implementing a POSIX 2013
compatible 'struct stat' with nanosecond-precision file times.

Signed-off-by: Karsten Blees <[email protected]>
We no longer use any of MSVCRT's stat-functions, so there's no need to
stick to a CRT-compatible 'struct stat' either.

Define and use our own POSIX-2013-compatible 'struct stat' with nanosecond-
precision file times.

Note: Due to performance issues when using git variants with different file
time resolutions, this patch does *not* yet enable nanosecond precision in
the Makefile (use 'make USE_NSEC=1').

Signed-off-by: Karsten Blees <[email protected]>
@dscho
Copy link
Member

dscho commented Sep 26, 2015

Nice! (Of course, it is not so nice that there are still racy things going on...).

I am thinking about merging this and play it by ear, depending on the discussion on the mailing list. @kblees are you okay with that?

@kblees
Copy link
Author

kblees commented Sep 28, 2015

I am thinking about merging this and play it by ear, depending on the discussion on the mailing list.
@kblees are you okay with that?

Of course, go ahead.

dscho added a commit that referenced this pull request Sep 28, 2015
@dscho dscho merged commit 805e2c2 into git-for-windows:master Sep 28, 2015
@dscho
Copy link
Member

dscho commented Sep 28, 2015

Thanks!

dscho added a commit that referenced this pull request Sep 29, 2015
dscho added a commit that referenced this pull request Oct 5, 2015
dscho added a commit that referenced this pull request Oct 19, 2015
dscho added a commit that referenced this pull request Nov 9, 2015
dscho added a commit that referenced this pull request Dec 11, 2015
dscho added a commit that referenced this pull request Jan 5, 2016
dscho added a commit that referenced this pull request Jan 28, 2016
dscho added a commit that referenced this pull request Feb 6, 2016
dscho added a commit that referenced this pull request Feb 23, 2016
dscho added a commit that referenced this pull request Mar 15, 2016
dscho added a commit that referenced this pull request Mar 18, 2016
dscho added a commit that referenced this pull request Mar 29, 2016
dscho added a commit that referenced this pull request Apr 4, 2016
dscho added a commit that referenced this pull request Apr 30, 2016
dscho added a commit that referenced this pull request May 20, 2016
dscho added a commit that referenced this pull request Jun 7, 2016
dscho added a commit that referenced this pull request Jun 9, 2016
dscho added a commit that referenced this pull request Jun 9, 2016
dscho added a commit that referenced this pull request Jun 9, 2016
dscho added a commit that referenced this pull request Jun 9, 2016
dscho added a commit that referenced this pull request Jun 14, 2016
jamill pushed a commit to jamill/git that referenced this pull request Sep 5, 2018
…-times-v2.5.3

nanosecond file times for v2.5.3
git-for-windows-ci pushed a commit that referenced this pull request Sep 10, 2018
jamill pushed a commit to jamill/git that referenced this pull request Sep 11, 2018
…-times-v2.5.3

nanosecond file times for v2.5.3
git-for-windows-ci pushed a commit that referenced this pull request Sep 24, 2018
dscho added a commit that referenced this pull request Oct 10, 2018
dscho added a commit to dscho/git that referenced this pull request Oct 12, 2018
…-times-v2.5.3

nanosecond file times for v2.5.3
dscho added a commit that referenced this pull request Oct 12, 2018
dscho added a commit to dscho/git that referenced this pull request Oct 19, 2018
…-times-v2.5.3

nanosecond file times for v2.5.3
dscho added a commit to dscho/git that referenced this pull request Oct 19, 2018
…-times-v2.5.3

nanosecond file times for v2.5.3
dscho added a commit to dscho/git that referenced this pull request Oct 30, 2018
…-times-v2.5.3

nanosecond file times for v2.5.3
dscho added a commit to dscho/git that referenced this pull request Oct 30, 2018
…-times-v2.5.3

nanosecond file times for v2.5.3
dscho added a commit to dscho/git that referenced this pull request Nov 19, 2018
…-times-v2.5.3

nanosecond file times for v2.5.3
dscho added a commit that referenced this pull request Nov 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants