fix: don't set LD_LIBRARY_PATH in the snap

snapcrafters / alacritty

A community-maintained package to easily install Alacritty on Linux

MIT License

9 stars 12 forks source link

fix: don't set LD_LIBRARY_PATH in the snap #14

Closed jnsgruk closed 9 months ago

jnsgruk commented 9 months ago

This snap currently sets LD_LIBRARY_PATH, which is incorrect for a classic snap. This PR stops that behaviour and uses patchelf instead. Tested locally and seems to function correctly.

jnsgruk commented 9 months ago

@compuguy @xiyuzhai WDYT?

xiyuzhai commented 9 months ago

Thanks a lot!

compuguy commented 9 months ago

Built the changes locally and installed it. LGTM @jnsgruk!