On Fri, Nov 25, 2016 at 11:28 AM, Albert Astals Cid <albert.astals at canonical.com> wrote: > On Thu, Nov 24, 2016 at 9:51 PM, Tanu Kaskinen <tanuk at iki.fi> wrote: >> On Wed, 2016-11-23 at 17:10 +0100, Albert Astals Cid wrote: >>> Helps making pulseaudio relocatable >> >> Could you provide more information in the commit message about what use >> case requires pulseaudio to be relocatable? > > In this particular case is making pulseaudio part of a snap file [1] > > But it seems somehow the problem we were having has been solved in a > different way by a colleague, i need to find out what he did and if we > still want to pursue this patch approach or not. Ok, so no there was no solution, just that i was using the tooling wrong and it was picking up a library i had built somewhere else with a hardcoded path in it ^_^ So i've updated the patch to be just one environment variable named PULSE_DATA_PATH What do you think? Cheers, Albert > > Cheers, > Albert > > [1] http://snapcraft.io/ > >> >> The existing convention for environment variable names is to use >> "PULSE_" as the prefix, and I think this patch should use the same >> convention. >> >> I think it would be better to add an environment variable for the data >> directory (/usr/share/pulseaudio). If and when we install more things >> to the data directory in the future, we would have to create more >> environment variables just to keep the relocation support working if >> each thing in the data directory needs a separate environment variable. >> >> -- >> Tanu >> >> https://www.patreon.com/tanuk >> _______________________________________________ >> pulseaudio-discuss mailing list >> pulseaudio-discuss at lists.freedesktop.org >> https://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss -------------- next part -------------- A non-text attachment was scrubbed... Name: pulse_data_path.patch Type: text/x-patch Size: 2576 bytes Desc: not available URL: <https://lists.freedesktop.org/archives/pulseaudio-discuss/attachments/20161201/67267d8f/attachment.bin>
