Description of problem: I noticed that the Xsession and xinitrc scripts source *all* executables in xinitrc.d/. This is pretty annoying if you happen to edit one of the files with emacs and then test it. How reproducible: every time Steps to Reproduce: 1. cd /etc/X11/xinit/xinitrc.d 2. cp -p xinput xinput~ (alternatively edit the file with emacs) 3. Start a new X session from gdm say. Actual results: Both xinput and xinput~ get source'd. Expected results: Only xinput to be sourced. Additional info: I think /etc/profile gets this right with: for i in /etc/profile.d/*.sh ; do if [ -r "$i" ]; then . $i fi done I noticed this problem because of the missing "]" in line 25 of xinput. ;)
Also it tries to execute subdirectories too...
Making this block FC2
Another way of inducing backup files, is if one modifies say xinput, and then upgrade xinitrc: then rpm will generate xinput.rpmnew, which will get sources along with (after?) xinput... Mike, do you agree requiring a ".sh" suffix for scripts is a good way to fix this, or is there a better way?
Yeah, I think the .sh method is the proper solution. I don't see how else you could do it foolproof. *.sh makes it explicit that anything needs to be named with a specific extension to get executed, and has the benefit of backup files, etc. getting ignored. It has the drawback that any other packages which drop files in there, which aren't already named *.sh, will get ignored too. Perhaps we should make the loop: for file in /etc/X11/xinit/xinitrc.d/* ; do if echo $file | grep -q "\.sh$" ; then . $file else echo "warning: $file does not end in .sh extension, ignoring" fi done Perhaps the warning should go to stderr. What do you think of the above?
The only other way, would be to ignore specific backup extensions, but that's not foolproof, as there is no standard.
Yeah, I like your solution in comment 4. I think that is fine.
Fixed in xinitrc-3.40-1
Note that you also have to change the other packages so they contain .sh scripts instead of saying: warning: /etc/X11/xinit/xinitrc.d/xinput does not end in .sh extension, ignoring