Thanks to visit codestin.com
Credit goes to GitHub.com

Skip to content

Conversation

@cloobTech
Copy link
Contributor

Thanks for taking the time to contribute to Git! Please be advised that the
Git community does not use github.com for their contributions. Instead, we use
a mailing list ([email protected]) for code submissions, code reviews, and
bug reports. Nevertheless, you can use GitGitGadget (https://gitgitgadget.github.io/)
to conveniently send your Pull Requests commits to our mailing list.

For a single-commit pull request, please leave the pull request description
empty
: your commit message itself should describe your changes.

Please read the "guidelines for contributing" linked above!

In other to reduce the usage of the global `trust_executable_bit`,
add a parameter to the function `ce_mode_from_stat()`, that will
get passed a repository variable.
This commit uses `UNUSED` as the next commit will modify the function
to pass the repository parameter.

Signed-off-by: Olamide Caleb Bello <[email protected]>
The `core.filemode` setting is stored in a global variable
and populated via `repo_cfg_bool()`. This may cause issues
when handling more than one repository in a single process
with diffrent values for that config key.

Refactor the code to store the value in `struct repo-settings`.
This help the path towards the libification of Git

Signed-off-by: Olamide Caleb Bello <[email protected]>
@dscho
Copy link
Member

dscho commented Jan 5, 2026

Thanks for taking the time to contribute to Git! Please be advised that the
Git community does not use github.com for their contributions. Instead, we use
a mailing list ([email protected]) for code submissions, code reviews, and
bug reports. Nevertheless, you can use GitGitGadget (https://gitgitgadget.github.io/)
to conveniently send your Pull Requests commits to our mailing list.

For a single-commit pull request, please leave the pull request description
empty
: your commit message itself should describe your changes.

Please read the "guidelines for contributing" linked above!

You will want to delete this before /submiting ..

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