[yocto] distro.conf and LOCALCONF_VERSION, LAYER_CONF_VERSION, OELAYOUT_ABI

[Rudolf Streif]

The Poky distro configuration distro.conf includes assignments to
LOCALCONF_VERSION, LAYER_CONF_VERSION, OELAYOUT_ABI to set the expected
version numbers for local.conf, bblayers.conf and the directory layout of
${TMPDIR}.

>From a user perspective this is somewhat confusing as these variables
really have nothing to do with the distro policy but only with the
compatibility between the build system and a build environment. For Poky it
may somewhat make sense to put them there. However if I write my own
distro.conf and put it in a layer I would want to use this layer with
different version of the build system. I think it would make more sense to
assign these variables in poky-sanity.bbclass.

The second little gripe is about the inconsistent naming of these variables
and the ones they relate to:

LOCALCONF_VERSION (expected value) <-> CONF_VERSION (actual value)
LAYER_CONF_VERSION (expected value) <-> LCONF_VERSION (actual value)
OELAYOUT_ABI (expected value) <-> actual value retrieved from
tmp/abi_version

Adding to the confusion is LAYERVERSION which refers to the version of the
conf/layer.conf file of a layer. That is in contrast to LAYER_CONF_VERSION
/ LCONF_VERSION which are for conf/bblayers.conf in the build enviroment.

Changing stuff that has been established for years is of course a pain but
it has been done and can be done in a backward compatible way.

My suggestions:

   1. Rename LOCALCONF_VERSION to LOCAL_CONF_VERSION_SET
   2. Rename LAYER_CONF_VERSION to BBLAYERS_CONF_VERSION_SET
   3. Rename CONF_VERSION to LOCAL_CONF_VERSION
   4. Rename LCONF_VERSION to BBLAYERS_CONF_VERSION
   5. Rename OELAYOUT_ABI to OELAYOUT_VERSION_SET
   6. Provide values for LOCAL_CONF_VERSION_SET, BBLAYERS_CONF_VERSION_SET,
   OELAYOUT_VERSION_SET in poky-sanity.bbclass and remove the settings from
   the distro configuration.

You would also want to change LAYERVERSION to LAYER_CONF_VERSION to be
consistent. But that cannot be done backwards compatible as it conflicts
with the currently used version variable for bblayers.conf.

These are small things that increase usability and help people to
understand the Yocto Project better.

Cheers,
Rudi
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.yoctoproject.org/pipermail/yocto/attachments/20140812/a0f782d1/attachment.html>