-
-
Notifications
You must be signed in to change notification settings - Fork 59
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
cargo-c: improve install path conventions #317
Comments
The normal libdir IS See autotools docs. If meson has some heuristic it is not exactly clear in that document, but if it works well enough there we can match it as long somebody provides a patch. I'd avoid distro-specific logic though. |
That seems like a question for @nirbheek |
meson detects the operating system, and for Debian-like systems it executes Unlike autotools, the rough heuristic is "we try to do what's actually correct on your OS, if we do not know then we can always just default to /usr/local/lib instead, but let's try to actually be correct if we can". It's not precisely a moving target, but it's definitely one where the code evolves. We got Haiku patches recently, for example. |
Sounds like a fair amount of code who's willing to look into porting it? I wonder if would make sense making a separate crate for it. |
@eli-schwartz can you link the meson heuristics? Probably it could be distilled in a stand alone crate so other tools may use it for the same purpose. |
https://github.com/mesonbuild/meson/blob/master/mesonbuild/utils/universal.py#L1021-L1094 It's really quite simple, not worth putting it into a crate yet, IMO. Maybe you'd like to keep it as an internal module till it grows enough and then it can be split out? |
If you have a timeslice to convert to a patch and send it would be great since I have plans to make a release soon due cbindgen ^^; |
When installing the
gst-plugin-rtp
plugin from https://gitlab.freedesktop.org/gstreamer/gst-plugins-rs, I noticed that with:cargo cinstall -p gst-plugin-rtp --prefix=/usr/local
the rust plugins (the shared libs) install to
/usr/local/lib/gstreamer-1.0/libgstrsrtp.so
whereas other gstreamer packages install to e.g.,
/usr/local/lib/x86_64-linux-gnu/gstreamer-1.0/
For now I can workaround this by explicitly adding
--libdir=${PREFIX}/lib/x86_64-linux-gnu
but it'd be preferable for this to follow themeson
+autotools
conventions (at least as I understand them).@sdroege's comment:
The text was updated successfully, but these errors were encountered: