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

Remove executable bit from source files #1

Merged
merged 1 commit into from Feb 1, 2015

Conversation

Projects
None yet
2 participants
@paultcochrane
Contributor

paultcochrane commented Feb 1, 2015

None of the source files need to be run stand-alone and none of the
documentation or configuration-related files need to be able to be run.
Thus this change removes the executable bit from files not needing this to
be set.

Paul Cochrane
Remove executable bit from source files
None of the source files need to be run stand-alone and none of the
documentation or configuration-related files need to be able to be run.
Thus this change removes the executable bit from files not needing this to
be set.

hoehrmann added a commit that referenced this pull request Feb 1, 2015

Merge pull request #1 from paultcochrane/pr/remove_exec_bit
Remove executable bit from source files

@hoehrmann hoehrmann merged commit c523897 into hoehrmann:master Feb 1, 2015

@hoehrmann

This comment has been minimized.

Owner

hoehrmann commented Feb 1, 2015

Thanks for the patch. Since I do most development on Windows, I guess all the repositories look like this. Is there some way to let git automatically correct file permissions? I do let Makefile.PL do this for make dist.

@paultcochrane

This comment has been minimized.

Contributor

paultcochrane commented Feb 1, 2015

Wow! That was quick! :-)

As far as I can tell, one needs to do something like this: git update-index --chmod=-x file.pl.
To explicitly make a file executable you need this: git update-index --chmod=+x script.sh.
There is also an option via git config to ignore file permissions changes: git config core.filemode false (see the docs for git config). Hope that helps!

@hoehrmann

This comment has been minimized.

Owner

hoehrmann commented Feb 1, 2015

Hmm, okay. I'll see what I can find out when I have the time. Thanks again!

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