iris-edu / yasmine-stationxml-editor

GNU General Public License v3.0
11 stars 3 forks source link

Files with useless execution bit set #7

Closed fabienengels closed 2 years ago

fabienengels commented 2 years ago

Hi,

Not a big issue but I saw that a lot of file have the execution bit set :

yasmine-stationxml-editor on  main (fb58f7f9) 
>> fd -t x | wc -l
6762

A quick fix would be to run the following command :

fd -E '*.py' -t f -x chmod -x {}

Letting only python files with the execution bit (even I think most of them won't need it as they are lib files).

Have a good day Fabien

fabienengels commented 2 years ago

@jmsaurel fd (https://github.com/sharkdp/fd) is a modern alternative to the good old find command, it can replace gnu parallel too.

You can use find instead instead :

find . -type f -executable -print -exec chmod -x {} \;

Another solution would be to remove all the execution bit and set it only where is required.

jmsaurel commented 2 years ago

Thanks @fabienengels for the tip on find and fd. This should be solved in PR #20 .

Can you double-check and close the issue if solved ?