| Summary: | libexecdir should be set to follow FHS recommendation | ||
|---|---|---|---|
| Product: | [Build System, Metadata & Runtime] OE-Core | Reporter: | Ross Burton <ross.burton> |
| Component: | core | Assignee: | Ross Burton <ross.burton> |
| Status: | VERIFIED FIXED | QA Contact: | Costin <costin.c.constantin> |
| Severity: | enhancement | ||
| Priority: | Medium+ | CC: | alexandru.c.georgescu, bluelightning, meta.mr.watcher, meta.watcher, srifenbark |
| Version: | 1.6 | ||
| Target Milestone: | 2.1 M2 | ||
| Hardware: | All | ||
| OS: | Multiple | ||
| Whiteboard: | 2 Feb 2016: IN PROGRESS REVIEW | ||
| OS type for building Yocto: | --- | Type of Regression: | New (Never tested) |
| Verified: | Documentation change: | Done (doc changes complete) | |
I have a patch and am testing now. Hi, Right now I have a section http://www.yoctoproject.org/docs/1.9/ref-manual/ref-manual.html#qa-errors-and-warnings that states: By default, libexecdir is set to "${libdir}/${BPN}" rather than to "/usr/libexec". I am assuming that any solution to this bug will change that? I think that is the only area in the YP doc set affected. Scott Scott: yes, that's probably the only place (and the release notes) in the docs that's relevant. Merged in oe-core e7270e331560546d3805cd66ed14afcbc96b6d89. Ross, What is the fallout for that error message? How does it change. See the link below to see what that error message says in the doc right now. I don't know how to change it regarding your fix. http://www.yoctoproject.org/docs/2.1/ref-manual/ref-manual.html#qa-errors-and-warnings Thanks, Scott That section needs to be something more like:
The specified package contains files in /usr/libexec when the distro configuration uses a different path for <libexecdir>. By default libexecdir is $prefix/libexec but this can be changed, for example to ${libdir}.
We also need the change of $libexecdir from $libdir/$BPN to $prefix/libexec added to the release notes.
Ross, See http://www.yoctoproject.org/docs/2.1/ref-manual/ref-manual.html#qa-errors-and-warnings and tell me if we are good on this description. Regarding the release notes... Probably need to notify Paul Eggleton about this. He is good at keeping a list of such things that get passed to Beth when the release notes are put down. Once we get the doc changes approved, I can set the doc flag to "done"... or you can. Scott The revision looks good, marking as done. Thanks Scott! Hello, Verified on latest master: b33e440cc6fbd703e8045d94b806790343e72eb6 To verify: 1) bitbake mc 2) check in the resulting .rpm file for /usr/libexec dir. rpm2cpio mc-4.8.15-r0.core2_64.rpm | cpio -iv | grep libexec |
We currently set libexecdir to ${libdir}/${BPN}, but we're alone in using this directory structure. The Debian world sets libexecdir to ${exec_prefix}/lib, and the Red Hat world uses ${exec_prefix}/libexec. The FHS says either are fine. Nobody puts the package name into the definition of libexecdir. I've a WIP branch that solves this but there will be fallout so we should merge it sooner rather than later.