madaidan via Whonix Forum:
Merged.
xsession_d_file_list_executable=( "20torbrowser" "20uwt" ... ) for file_name in "/etc/X11/Xsession.d/${xsession_d_file_list_executable[@]}" ; do
I don’t think this code would work. Tested. Would expand to:
/etc/X11/Xsession.d/20torbrowser
20uwt
String /etc/X11/Xsession.d/
could be inside the for
loop. That would
work.
But I don’t see much advantage in that. Trading a bit shorter contents
inside the variable assignment at the cost of higher complexity inside
the for
loop. Using the full file names such as
/etc/X11/Xsession.d/20uwt
makes it easier to grep (the whole Whonix)
source code. As currently implemented, code is easier to understand.
Currently one can copy/paste one string, the file name, and then use cat
or open in an editor to view the file. By making it
/etc/X11/Xsession.d/
+ variable it gets harder. Then one has to
understand this a bit better, copy/paste /etc/X11/Xsession.d/
(or
manually write) + the variable name (such as 20uwt
).