This is a follow-up for 7d91908d.
Details
Details
- Reviewers
yousra - Commits
- rOPSe819c47a7673: Simplify autochmod-git
Tested on Complector on /opt/salt/nasqueron-operations
Diff Detail
Diff Detail
- Repository
- rOPS Nasqueron Operations
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Comment Actions
About shellcheck linter issues.
As documented in the commit message, we trade whitespace security to straightforward convenience knowing we don't use space in file names.
As such, we want to use space for word splitting in the output of find command, space is a valid separator between files in our repository.
Comment Actions
I've discovered X option in chmod that solves the issue nicely, by setting executable bit only where needed (already executable files and directories), so we can keep the find approach and stay safe for whitespaces.