[Libre-soc-bugs] [Bug 696] [Coroliosis 2] Library libMut Not Found During Alliance Install

[bugzilla-daemon at libre-soc.org]

https://bugs.libre-soc.org/show_bug.cgi?id=696

--- Comment #11 from andrey at technepisteme.xyz ---
After looking at Luke's git merges for "coriolis_install" script (only came
back to it today), noticed again the issue with "coriolisenv" file containing
"//" in the PATHs (missing the username contained in $SUDO_USER variable of
whoever runs the script).

For example:
LD_LIBRARY_PATH="/home/rohdo/coriolis-2.x/Linux.x86_64/Release.Shared/install/lib64:/home//alliance/install/lib64:/home//alliance/install/lib:/home//alliance/install/lib64:/home//alliance/install/lib:";

This is caused, not by an issue in coriolisEnv.py, but because the PATH
variable already had the $SUDO_USER missing. This actually happens earlier in
"coriolis_install" script.

The "echo -e" statement (see line 28 of the script) that writes into
.bash_profile was using single quotes ('') and thus was not expanding the
$SUDO_USER (which is defined at the script level, and doesn't exist at the
shell level where the echo statement is executed).

I'm working on a fix atm, and will have it done tomorrow. Then I'll give the
coriolis installation another go, and this time (hopefully) alliance should see
all the needed libraries for compilation.


The reason we didn't see such an issue before, is probably because scripts were
always run as root via "sudo bash" which always adds the $SUDO_USER environment
variable. 

Running a script as a normal user means $SUDO_USER doesn't exist, and so has to
be defined locally. Variable expansion would then be done within the script
(using double quotes "") before the command is run in the shell.

Otherwise we could define our own variable with a different name, as manually
defining $SUDO_USER outside a script is probably a bad idea.
The linux environment doesn't seem to define an environment variable for the
current user (there is a var for user id though).

-- 
You are receiving this mail because:
You are on the CC list for the bug.