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

object-file: pass filename to fsync_or_die #1194

Closed

Conversation

neerajsi-msft
Copy link

If we die while trying to fsync a loose object file, pass the actual
filename we're trying to sync. This is likely to be more helpful for a
user trying to diagnose the cause of the failure than the former
'loose object file' string. It also sidesteps any concerns about
translating the die message differently for loose objects versus
something else that has a real path.

Reported-by: Ævar Arnfjörð Bjarmason avarab@gmail.com
Signed-off-by: Neeraj Singh neerajsi@microsoft.com

This is in response to:
https://lore.kernel.org/git/220321.861qyv9rjr.gmgdl@evledraar.gmail.com/

Unrelated nit: this API is producing sentence lego unfriendly to translators.

If we die while trying to fsync a loose object file, pass the actual
filename we're trying to sync. This is likely to be more helpful for a
user trying to diagnose the cause of the failure than the former
'loose object file' string. It also sidesteps any concerns about
translating the die message differently for loose objects versus
something else that has a real path.

Reported-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Neeraj Singh <neerajsi@microsoft.com>
@neerajsi-msft
Copy link
Author

/submit

@gitgitgadget
Copy link

gitgitgadget bot commented Mar 30, 2022

Submitted as pull.1194.git.1648664055986.gitgitgadget@gmail.com

To fetch this version into FETCH_HEAD:

git fetch https://github.com/gitgitgadget/git/ pr-1194/neerajsi-msft/ns/fsync-die-filename-v1

To fetch this version to local tag pr-1194/neerajsi-msft/ns/fsync-die-filename-v1:

git fetch --no-tags https://github.com/gitgitgadget/git/ tag pr-1194/neerajsi-msft/ns/fsync-die-filename-v1

@gitgitgadget
Copy link

gitgitgadget bot commented Mar 31, 2022

This branch is now known as ns/fsync-or-die-message-fix.

@gitgitgadget
Copy link

gitgitgadget bot commented Mar 31, 2022

This patch series was integrated into seen via git@1c2d173.

@gitgitgadget gitgitgadget bot added the seen label Mar 31, 2022
@gitgitgadget
Copy link

gitgitgadget bot commented Mar 31, 2022

There was a status update in the "New Topics" section about the branch ns/fsync-or-die-message-fix on the Git mailing list:

When creating a loose object file, we didn't report the exact
filename of the file we failed to fsync, even though the
information was readily available, which has been corrected.

Will merge to 'next'.
source: <pull.1194.git.1648664055986.gitgitgadget@gmail.com>

@gitgitgadget
Copy link

gitgitgadget bot commented Mar 31, 2022

This patch series was integrated into seen via git@26d1013.

@gitgitgadget
Copy link

gitgitgadget bot commented Mar 31, 2022

This patch series was integrated into next via git@9de1909.

@gitgitgadget gitgitgadget bot added the next label Mar 31, 2022
@gitgitgadget
Copy link

gitgitgadget bot commented Apr 1, 2022

This patch series was integrated into seen via git@656a3cf.

@gitgitgadget
Copy link

gitgitgadget bot commented Apr 3, 2022

This patch series was integrated into seen via git@7f77b4d.

@gitgitgadget
Copy link

gitgitgadget bot commented Apr 4, 2022

This patch series was integrated into seen via git@98f6a3a.

@gitgitgadget
Copy link

gitgitgadget bot commented Apr 4, 2022

This patch series was integrated into master via git@98f6a3a.

@gitgitgadget
Copy link

gitgitgadget bot commented Apr 4, 2022

This patch series was integrated into next via git@98f6a3a.

@gitgitgadget gitgitgadget bot added the master label Apr 4, 2022
@gitgitgadget gitgitgadget bot closed this Apr 4, 2022
@gitgitgadget
Copy link

gitgitgadget bot commented Apr 4, 2022

Closed via 98f6a3a.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant