Skip to content

Conversation

@McSinyx
Copy link
Contributor

@McSinyx McSinyx commented Nov 13, 2024

For real files, git only distinguish between executable or not, and the mode of a checked-out file is either 644 or 755.

For real files, git only distinguish between executable or not,
and the mode of a checked-out file is either 644 or 755.
@McSinyx
Copy link
Contributor Author

McSinyx commented Dec 10, 2024

Gentle ping!

@GJDuck
Copy link
Owner

GJDuck commented Dec 10, 2024

This seems to depend on the system, and the patch will fail when I try it. Perhaps the mode can be set explicitly in the Makefile?

PS: the entire regtest suite will pass on my system, but can fail on other systems. This should be fixed eventually.

@McSinyx
Copy link
Contributor Author

McSinyx commented Dec 10, 2024

Perhaps the mode can be set explicitly in the Makefile?

Ideally, yes, then reverted to the original mode (or to be run in some isolated environment).

Does the suite in a fresh checkout pass on your system though? From the Git index format documentation:

9-bit unix permission. Only 0755 and 0644 are valid for regular files.
Symbolic links and gitlinks have value 0 in this field.

Git wouldn't mandate these two modes for existing file (e.g. mode change from 644 to 664 is ignored), but new files would be created with mode 755 or 644.

@McSinyx
Copy link
Contributor Author

McSinyx commented Jan 9, 2025

May I have this merged? If the documentation is not convincing enough, here are the modes of added files in a formatted patch.

@GJDuck GJDuck closed this in e888f38 Jan 13, 2025
@GJDuck
Copy link
Owner

GJDuck commented Jan 13, 2025

The Makefile now sets the mode explicitly, so the test should now pass for any system.

@McSinyx
Copy link
Contributor Author

McSinyx commented Jan 13, 2025

Thanks!

@McSinyx McSinyx deleted the mode branch January 13, 2025 02:11
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