package_mrtrix updated: stripped release, scripts-->bin. #988
Conversation
|
|
||
| echo - collating dependent system libraries... | ||
| LIBS=$(for n in release/bin/*; do ldd $n; done | sed 's/(.*)$//g' | sed 's/^.*=>//g' | sort | uniq | grep -v 'mrtrix\|vdso\|libGL.so\|libEGL.so\|nvidia\|fglrx' | xargs) | ||
| LIBS=$(for n in bin/*; do ldd $n; done | sed 's/(.*)$//g' | sed 's/^.*=>//g' | sort | uniq | grep -v 'mrtrix\|vdso\|libGL.so\|libEGL.so\|nvidia\|fglrx' | xargs) |
There was a problem hiding this comment.
Not sure about this line. Not exactly intuitive...
There was a problem hiding this comment.
🤣 agreed...
What it does is list all dynamic dependencies (libraries) used by our executables (ldd), makes sure they're only listed once (sort | uniq), and strips out libraries that we shouldn't just blindly add (mostly all OpenGL/graphics-related libraries).
Not sure it would still work anyway since bin/ now also contains scripts. Not sure whether ldd would produce output that might interfere in this case...
There was a problem hiding this comment.
For scripts, ldd produces the output "not a dynamic executable" which triggers the warning "not all executables have dynamic libraries. Possibly skipping dependent libraries for some commands. This is expected if you built with -static"
I'd just get rid of the warning. Alternatively, we could use file to filter out scripts and non-dynamically linked binaries:
file bin/* | awk -F: '/dynamically linked/ {print $1}' | xargs -d'\n' -r echo -u
No description provided.