Bug 6398 - libexecdir should be set to follow FHS recommendation
Summary: libexecdir should be set to follow FHS recommendation
Status: VERIFIED FIXED
Alias: None
Product: OE-Core
Classification: Build System, Metadata & Runtime
Component: core (show other bugs)
Version: 1.6
Hardware: All Multiple
: Medium+ enhancement
Target Milestone: 2.1 M2
Assignee: Ross Burton
QA Contact: Costin
URL:
Whiteboard: 2 Feb 2016: IN PROGRESS REVIEW
Depends on:
Blocks:
 
Reported: 2014-06-04 11:02 UTC by Ross Burton
Modified: 2016-02-05 11:12 UTC (History)
5 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: New (Never tested)
Verified:
Documentation change: Done (doc changes complete)


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ross Burton 2014-06-04 11:02:53 UTC
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.
Comment 1 Ross Burton 2014-11-17 18:24:14 UTC
I have a patch and am testing now.
Comment 2 Scott Rifenbark 2015-06-01 23:04:28 UTC
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
Comment 3 Ross Burton 2015-06-02 07:52:37 UTC
Scott: yes, that's probably the only place (and the release notes) in the docs that's relevant.
Comment 4 Ross Burton 2016-01-26 14:21:26 UTC
Merged in oe-core e7270e331560546d3805cd66ed14afcbc96b6d89.
Comment 5 Scott Rifenbark 2016-01-27 23:41:58 UTC
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
Comment 6 Ross Burton 2016-01-28 11:13:33 UTC
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.
Comment 7 Scott Rifenbark 2016-02-03 02:00:40 UTC
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
Comment 8 Ross Burton 2016-02-03 17:30:50 UTC
The revision looks good, marking as done.  Thanks Scott!
Comment 9 Costin 2016-02-05 11:12:40 UTC
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