search

CuarzoSoftware / SRM

Simple Rendering Manager
MIT License
51 stars 5 forks source link

Clean up meson.build (Fixed PR against main branch) #3

Closed godalming123 closed 11 months ago

godalming123 commented 11 months ago

This makes a few changes to the meson.build file that fix one warning, and make it slightly more readable.

godalming123 commented 11 months ago

(I can make this into one commit if you want)

ehopperdietzel commented 11 months ago

Hey, thanks for your feedback. I've tested it on Debian, and it compiles smoothly without any issues. However, I did run into problems with library dependencies on Red Hat distros in the past, which is why I included the library paths array. I plan to retest it on Red Hat distros, and if everything checks out, I'll proceed with the merge.

Which distro are you using btw?

ehopperdietzel commented 11 months ago

Tested it and everything is working ;)

godalming123 commented 11 months ago

I am using fedora, and am finding that in order to run the example (found at this projects readme), I have to set the LD_LIBRARY_PATH enviroment variable first.

ehopperdietzel commented 11 months ago

Yes, I had previously hardcoded the /usr/lib path for library installation, but I've now removed it. Meson should now install the library in the default distribution library directory. On Fedora, running sudo ldconfig after installing the library is necessary to update the shared library cache.