[OE-core] [PATCH 00/30] cleanup: use variables instead of hardcoded paths

Javier Martinez Canillas javier at dowhile0.org
Sun Aug 5 15:53:47 UTC 2012


The OpenEmbedded User Manual list the variables that should be used to
control the directories into which files are installed.

It says that is a poor practice to specify hardcoded paths instead of
using these variables, yet there are many recipes that don't use it.

This is a big patch-set that does a cleanup and replace the hardcoded
paths used on these recipes with the build system installation variables.

I tried to be as careful as possible to do the proper replacement but
since I could introduce regressions I split the changes in 30 different
patches so it could be git bisectable in case of messing a recipe.

Also, I didn't increment the PR on the recipes since the modifications
don't introduce a semantic change on them.

The patch-set consist of the following patches:

[PATCH 01/30] xinetd: use ${sbindir} instead of /usr/sbin for
[PATCH 02/30] alsa-state: use ${sbindir} instead of /usr/sbin for
[PATCH 03/30] lsbsetup: use ${bindir} instead of /usr/bin for
[PATCH 04/30] sudo: use ${bindir} and ${sysconfdir} instead of
[PATCH 05/30] lsbtest: use ${bindir} instead of /usr/bin for
[PATCH 06/30] cronie: use variables instead of hardcoded paths
[PATCH 07/30] useradd-example: use ${datadir} instead of /usr/share
[PATCH 08/30] ubootchart: use variables instead of hardcoded paths
[PATCH 09/30] xkeyboard-config: use ${datadir} instead of /usr/share
[PATCH 10/30] systemtap: use ${datadir} instead of /usr/share for
[PATCH 11/30] lsb: use ${base_bindir} instead of /bin for packaging
[PATCH 12/30] mingetty: use ${base_sbindir} instead of /sbin for
[PATCH 13/30] external-sourcery: use ${prefix} and ${libdir} instead
[PATCH 14/30] rpm: use ${localstatedir} and ${libdir} instead of
[PATCH 15/30] at: use ${base_sbindir} instead of /sbin for packaging
[PATCH 16/30] kernel.bbclass: use ${base_libdir} and ${sysconfdir}
[PATCH 17/30] linux-firware: use ${base_libdir} instead of /lib for
[PATCH 18/30] openssh: use ${localstatedir} instead of /var for
[PATCH 19/30] libpam: use ${localstatedir} and ${sysconfdir} instead
[PATCH 20/30] x11-common: use ${sysconfdir} instead of /etc for
[PATCH 21/30] lsb: use ${sysconfdir} instead of /etc for packaging
[PATCH 22/30] builder: use ${sysconfdir} instead of /etc for
[PATCH 23/30] xserver-nodm-init: use ${sysconfdir} instead of /etc
[PATCH 24/30] lsbinitscripts: use ${sysconfdir} instead of /etc for
[PATCH 25/30] xinetd: use ${sysconfdir} instead of /etc for
[PATCH 26/30] usbinit: use ${sysconfdir} instead of /etc for
[PATCH 27/30] qemu-config: use ${sysconfdir} instead of /etc for
[PATCH 28/30] rsync: use ${sysconfdir} instead of /etc for packaging
[PATCH 29/30] chkconfig: use ${sysconfdir} instead of /etc for
[PATCH 30/30] man: use ${sysconfdir} instead of /etc for packaging

Best regards,
Javier




More information about the Openembedded-core mailing list