Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
Use also ubuntu-app-platform's lib/$ARCH dir for LD_LIBRARY_PATH (LP:… #40
Conversation
didrocks
requested changes
Jan 30, 2017
I guess this is fixed with the runtime refactor I just did merge now that I'm back (I did ping you on IRC about it), do you mind confirming?
tjyrinki
commented
Feb 2, 2017
|
Asking oSoMoN to retest since he had the original bug (if he can back off his changes to workaround the bug). Looking at the desktop-helpers repository, I don't immediately see where the platform snap's $RUNTIME/lib/ would be added, only eg export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:$RUNTIME/usr/lib/$ARCH |
tjyrinki
commented
Feb 2, 2017
|
Updated the pull request to reflect what might be needed. |
|
I don't think this fix is needed. I would just ask oSoMoN to rest trunk first (you don't have any ubuntu-app-platform/lib inclusion that's needed AFAIK). |
|
I can confirm that https://launchpad.net/bugs/1642900 is fixed without this change. |
|
Let's close that PR then. |
tjyrinki commentedJan 24, 2017
•
Edited 1 time
-
tjyrinki
Jan 24, 2017
… #1642900)