[PATCH 0/6] meta-yocto: cleanup of local.conf.sample

[PATCH 0/6] meta-yocto: cleanup of local.conf.sample

[Paul Eggleton]

A number of changes aimed at cleaning up local.conf.sample so it is easier
to deal with for new users. These are not necessarily final, happy to take
comments/suggestions.

The following changes since commit eea4bd731ef286516ee1fb934fe2222a09c06e70:

  mpc8315e-rdb: Set TARGET_FPU correct (2011-07-22 09:02:58 -0700)

are available in the git repository at:
  git://git.pokylinux.org/poky-contrib paule/local-conf-cleanup
  http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=paule/local-conf-cleanup

Paul Eggleton (6):
  meta-yocto/local.conf.sample.extended: update DISTRO_FEATURES
  meta-yocto/local.conf.sample: remove oprofileui-native related
    settings
  meta-yocto/conf/local.conf.sample: move advanced settings to extended
    sample file
  meta-yocto/conf/local.conf.sample: improve comments/layout
  meta-yocto/conf/local.conf.sample: remove emenlow from machine list
  meta-yocto/conf/local.conf.sample: add rm_work example

 meta-yocto/conf/local.conf.sample          |  130 ++++++----------------------
 meta-yocto/conf/local.conf.sample.extended |   92 +++++++++++++++++++-
 2 files changed, 118 insertions(+), 104 deletions(-)

-- 
1.7.4.1

[PATCH 1/6] meta-yocto/local.conf.sample.extended: update DISTRO_FEATURES

[Paul Eggleton]

Update to match recent changes to this variable in the
default-distrovars.inc file.

Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
---
 meta-yocto/conf/local.conf.sample.extended |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta-yocto/conf/local.conf.sample.extended b/meta-yocto/conf/local.conf.sample.extended
index 277ee2a..775d6d3 100644
--- a/meta-yocto/conf/local.conf.sample.extended
+++ b/meta-yocto/conf/local.conf.sample.extended
@@ -2,11 +2,11 @@
 # the all supported eglibc options are listed in DISTRO_FEATURES_LIBC
 # and disabled by default. Uncomment and copy the DISTRO_FEATURES_LIBC
 # and DISTRO_FEATURES definitions to local.conf to enable the options.
-#DISTRO_FEATURES_LIBC = "ipv6 libc-backtrace libc-big-macros libc-bsd libc-cxx-tests libc-catgets libc-charsets libc-crypt \
+#DISTRO_FEATURES_LIBC = "ipv4 ipv6 libc-backtrace libc-big-macros libc-bsd libc-cxx-tests libc-catgets libc-charsets libc-crypt \
 #               libc-crypt-ufc libc-db-aliases libc-envz libc-fcvt libc-fmtmsg libc-fstab libc-ftraverse \
 #               libc-getlogin libc-idn libc-inet libc-inet-anl libc-libm libc-libm-big libc-locales libc-locale-code \
 #               libc-memusage libc-nis libc-nsswitch libc-rcmd libc-rtld-debug libc-spawn libc-streams libc-sunrpc \
 #               libc-utmp libc-utmpx libc-wordexp libc-posix-clang-wchar libc-posix-regexp libc-posix-regexp-glibc \
 #               libc-posix-wchar-io"
+#DISTRO_FEATURES = "alsa argp bluetooth ext2 irda largefile pcmcia usbgadget usbhost wifi xattr nfs zeroconf pci 3g ${DISTRO_FEATURES_LIBC}"
 
-#DISTRO_FEATURES = "alsa bluetooth ext2 irda pcmcia usbgadget usbhost wifi nfs zeroconf pci ${DISTRO_FEATURES_LIBC}"
-- 
1.7.4.1

[PATCH 2/6] meta-yocto/local.conf.sample: remove oprofileui-native related settings

[Paul Eggleton]

oprofileui-nativesdk would be a better way to handle these kinds of
requirements with the modern system, and it is not something that most
users will need to configure anyway (nor was it supported.)

Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
---
 meta-yocto/conf/local.conf.sample |    6 ------
 1 files changed, 0 insertions(+), 6 deletions(-)

diff --git a/meta-yocto/conf/local.conf.sample b/meta-yocto/conf/local.conf.sample
index 499c48e..6ba92af 100644
--- a/meta-yocto/conf/local.conf.sample
+++ b/meta-yocto/conf/local.conf.sample
@@ -142,12 +142,6 @@ PACKAGE_DEBUG_SPLIT_STYLE = '.debug'
 # Uncomment this if you want BitBake to emit the log if a build fails.
 BBINCLUDELOGS = "yes"
 
-# Set this if you wish to make pkgconfig libraries from your system available 
-# for native builds. Combined with extra ASSUME_PROVIDEDs this can allow 
-# native builds of applications like oprofileui-native (unsupported feature).
-#EXTRA_NATIVE_PKGCONFIG_PATH = ":/usr/lib/pkgconfig"
-#ASSUME_PROVIDED += "gtk+-native libglade-native"
-
 ENABLE_BINARY_LOCALE_GENERATION = "1"
 
 # The architecture to build SDK items for, by setting this you can build SDK
-- 
1.7.4.1

[PATCH 3/6] meta-yocto/conf/local.conf.sample: move advanced settings to extended sample file

[Paul Eggleton]

These settings are very unlikely to be needed by new users. Also add a
comment pointing to the extended file as well as the Poky Reference
Manual.

Note that this removes the setting of PACKAGE_DEBUG_SPLIT_STYLE; however
we check for the value that is not the default so this should not make
any difference to the default behaviour.

Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
---
 meta-yocto/conf/local.conf.sample          |   98 +++-------------------------
 meta-yocto/conf/local.conf.sample.extended |   88 +++++++++++++++++++++++++
 2 files changed, 97 insertions(+), 89 deletions(-)

diff --git a/meta-yocto/conf/local.conf.sample b/meta-yocto/conf/local.conf.sample
index 6ba92af..523a737 100644
--- a/meta-yocto/conf/local.conf.sample
+++ b/meta-yocto/conf/local.conf.sample
@@ -1,3 +1,12 @@
+# For more information on these configuration options, please see the
+# Poky Reference Manual at:
+#
+#  http://www.yoctoproject.org/docs/poky-ref-manual/poky-ref-manual.html
+#
+# Note: there are additional more advanced configuration option examples
+# (e.g. for debugging) in local.conf.sample.extended which can be found
+# in meta-yocto/conf/
+
 # CONF_VERSION is increased each time build/conf/ changes incompatibly
 CONF_VERSION = "1"
 
@@ -64,15 +73,6 @@ EXTRA_IMAGE_FEATURES = "debug-tweaks"
 #PACKAGE_CLASSES ?= "package_rpm package_deb package_ipk"
 PACKAGE_CLASSES ?= "package_rpm package_ipk"
 
-# mklibs library size optimization is more useful to smaller images,
-# and less useful for bigger images. Also mklibs library optimization
-# can break the ABI compatibility, so should not be applied to the
-# images which are to be extended or upgraded later.
-#This enabled mklibs library size optimization just for the specified image.
-#MKLIBS_OPTIMIZED_IMAGES ?= "core-image-minimal"
-#This enable mklibs library size optimization will be for all the images.
-#MKLIBS_OPTIMIZED_IMAGES ?= "all"
-
 # A list of additional classes to use when building the system
 # include 'image-mklibs' to reduce shared library files size for an image
 # include 'image-prelink' in order to prelink the filesystem image
@@ -80,65 +80,6 @@ PACKAGE_CLASSES ?= "package_rpm package_ipk"
 # NOTE: if listing mklibs & prelink both, then make sure mklibs is before prelink
 USER_CLASSES ?= "image-mklibs image-prelink"
 
-# POKYMODE controls the characteristics of the generated packages/images by
-# telling poky which type of toolchain to use.
-#
-# Options include several different EABI combinations and a compatibility 
-# mode for the OABI mode poky previously used. 
-#
-# The default is "eabi"
-# Use "oabi" for machines with kernels < 2.6.18 on ARM for example.
-# Use "external-MODE" to use the precompiled external toolchains where MODE
-# is the type of external toolchain to use e.g. eabi.
-# POKYMODE = "external-eabi"
-
-# Uncomment this to specify where BitBake should create its temporary files.
-# Note that a full build of everything in OpenEmbedded will take GigaBytes of hard
-# disk space, so make sure to free enough space. The default TMPDIR is
-# <build directory>/tmp
-#TMPDIR = "${COREBASE}/build/tmp"
-
-# The following are used to control options related to debugging.
-#
-# Uncomment this to change the optimization to make debugging easer, at the 
-# possible cost of performance.
-# DEBUG_BUILD = "1"
-#
-# Uncomment this to disable the stripping of the installed binaries
-# INHIBIT_PACKAGE_STRIP = "1"
-#
-# Uncomment this to disable the split of the debug information into -dbg files
-# INHIBIT_PACKAGE_DEBUG_SPLIT = "1"
-#
-# When splitting debug information, the following controls the results of the
-# file splitting.
-#
-#  .debug (default):
-#    When splitting the debug information will be placed into
-#    a .debug directory in the same dirname of the binary produced:
-#      /bin/foo -> /bin/.debug/foo
-#
-#  debug-file-directory:
-#    When splitting the debug information will be placed into
-#    a central debug-file-directory, /usr/lib/debug:
-#      /bin/foo -> /usr/lib/debug/bin/foo.debug
-#
-#    Any source code referenced in the debug symbols will be copied
-#    and made available within the /usr/src/debug directory
-#
-PACKAGE_DEBUG_SPLIT_STYLE = '.debug'
-# PACKAGE_DEBUG_SPLIT_STYLE = 'debug-file-directory'
-
-# Uncomment these to build a package such that you can use gprof to profile it.
-# NOTE: This will only work with 'linux' targets, not
-# 'linux-uclibc', as uClibc doesn't provide the necessary
-# object files.  Also, don't build glibc itself with these
-# flags, or it'll fail to build.
-#
-# PROFILE_OPTIMIZATION = "-pg"
-# SELECTED_OPTIMIZATION = "${PROFILE_OPTIMIZATION}"
-# LDFLAGS =+ "-pg"
-
 # Uncomment this if you want BitBake to emit the log if a build fails.
 BBINCLUDELOGS = "yes"
 
@@ -159,23 +100,6 @@ ENABLE_BINARY_LOCALE_GENERATION = "1"
 #file://.* http://someserver.tld/share/sstate/ \n \
 #file://.* file:///some/local/dir/sstate/"
 
-# Set IMAGETEST to qemu if you want to build testcases and start 
-# testing in qemu after do_rootfs.
-#IMAGETEST = "qemu"
-
-# By default test cases in sanity suite will be ran. If you want to run other 
-# test suite or specific test case(e.g. bat or boot test case under sanity suite), 
-# list them like following.
-#TEST_SCEN = "sanity bat sanity:boot toolchain"
-
-#Because of the QEMU booting slowness issue(see bug #646 and #618), autobuilder
-#may suffer a timeout issue when running sanity test. We introduce variable
-#TEST_SERIALIZE here to reduce the time on sanity test. It is by default set 
-#to 1. Poky will start image and run cases in the same image without reboot 
-#or kill. If it is set to 0, the image will be copied and tested for each 
-#case, which will take much time.
-#TEST_SERIALIZE = "1"
-
 # Set GLIBC_GENERATE_LOCALES to the locales you wish to generate should you not
 # wish to perform the time-consuming step of generating all LIBC locales.
 # NOTE: If removing en_US.UTF-8 you will also need to uncomment, and set
@@ -207,9 +131,5 @@ NO32LIBS = "1"
 #TERMCMD = "${KONSOLE_TERMCMD}"
 #TERMCMDRUN = "${KONSOLE_TERMCMDRUN}"
 
-# The network based PR service host and port
-#PRSERV_HOST = "localhost"
-#PRSERV_PORT = "8585"
-
 # Uncomment this if your host distribution provides the help2man tool.
 #ASSUME_PROVIDED += "help2man-native"
diff --git a/meta-yocto/conf/local.conf.sample.extended b/meta-yocto/conf/local.conf.sample.extended
index 775d6d3..82e9ea9 100644
--- a/meta-yocto/conf/local.conf.sample.extended
+++ b/meta-yocto/conf/local.conf.sample.extended
@@ -10,3 +10,91 @@
 #               libc-posix-wchar-io"
 #DISTRO_FEATURES = "alsa argp bluetooth ext2 irda largefile pcmcia usbgadget usbhost wifi xattr nfs zeroconf pci 3g ${DISTRO_FEATURES_LIBC}"
 
+# mklibs library size optimization is more useful to smaller images,
+# and less useful for bigger images. Also mklibs library optimization
+# can break the ABI compatibility, so should not be applied to the
+# images which are to be extended or upgraded later.
+#This enabled mklibs library size optimization just for the specified image.
+#MKLIBS_OPTIMIZED_IMAGES ?= "core-image-minimal"
+#This enable mklibs library size optimization will be for all the images.
+#MKLIBS_OPTIMIZED_IMAGES ?= "all"
+
+# POKYMODE controls the characteristics of the generated packages/images by
+# telling poky which type of toolchain to use.
+#
+# Options include several different EABI combinations and a compatibility 
+# mode for the OABI mode poky previously used. 
+#
+# The default is "eabi"
+# Use "oabi" for machines with kernels < 2.6.18 on ARM for example.
+# Use "external-MODE" to use the precompiled external toolchains where MODE
+# is the type of external toolchain to use e.g. eabi.
+# POKYMODE = "external-eabi"
+
+# Uncomment this to specify where BitBake should create its temporary files.
+# Note that a full build of everything in OpenEmbedded will take GigaBytes of hard
+# disk space, so make sure to free enough space. The default TMPDIR is
+# <build directory>/tmp
+#TMPDIR = "${COREBASE}/build/tmp"
+
+# The following are used to control options related to debugging.
+#
+# Uncomment this to change the optimization to make debugging easer, at the 
+# possible cost of performance.
+# DEBUG_BUILD = "1"
+#
+# Uncomment this to disable the stripping of the installed binaries
+# INHIBIT_PACKAGE_STRIP = "1"
+#
+# Uncomment this to disable the split of the debug information into -dbg files
+# INHIBIT_PACKAGE_DEBUG_SPLIT = "1"
+#
+# When splitting debug information, the following controls the results of the
+# file splitting.
+#
+#  .debug (default):
+#    When splitting the debug information will be placed into
+#    a .debug directory in the same dirname of the binary produced:
+#      /bin/foo -> /bin/.debug/foo
+#
+#  debug-file-directory:
+#    When splitting the debug information will be placed into
+#    a central debug-file-directory, /usr/lib/debug:
+#      /bin/foo -> /usr/lib/debug/bin/foo.debug
+#
+#    Any source code referenced in the debug symbols will be copied
+#    and made available within the /usr/src/debug directory
+#
+# PACKAGE_DEBUG_SPLIT_STYLE = 'debug-file-directory'
+
+# Uncomment these to build a package such that you can use gprof to profile it.
+# NOTE: This will only work with 'linux' targets, not
+# 'linux-uclibc', as uClibc doesn't provide the necessary
+# object files.  Also, don't build glibc itself with these
+# flags, or it'll fail to build.
+#
+# PROFILE_OPTIMIZATION = "-pg"
+# SELECTED_OPTIMIZATION = "${PROFILE_OPTIMIZATION}"
+# LDFLAGS =+ "-pg"
+
+# Set IMAGETEST to qemu if you want to build testcases and start 
+# testing in qemu after do_rootfs.
+#IMAGETEST = "qemu"
+
+# By default test cases in sanity suite will be ran. If you want to run other 
+# test suite or specific test case(e.g. bat or boot test case under sanity suite), 
+# list them like following.
+#TEST_SCEN = "sanity bat sanity:boot toolchain"
+
+#Because of the QEMU booting slowness issue(see bug #646 and #618), autobuilder
+#may suffer a timeout issue when running sanity test. We introduce variable
+#TEST_SERIALIZE here to reduce the time on sanity test. It is by default set 
+#to 1. Poky will start image and run cases in the same image without reboot 
+#or kill. If it is set to 0, the image will be copied and tested for each 
+#case, which will take much time.
+#TEST_SERIALIZE = "1"
+
+# The network based PR service host and port
+#PRSERV_HOST = "localhost"
+#PRSERV_PORT = "8585"
+
-- 
1.7.4.1

[PATCH 4/6] meta-yocto/conf/local.conf.sample: improve comments/layout

[Paul Eggleton]

Some additions and grammar fixes to the comments, as well as moving
the locale options to the same place.

Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
---
 meta-yocto/conf/local.conf.sample |   21 ++++++++++++++-------
 1 files changed, 14 insertions(+), 7 deletions(-)

diff --git a/meta-yocto/conf/local.conf.sample b/meta-yocto/conf/local.conf.sample
index 523a737..524a256 100644
--- a/meta-yocto/conf/local.conf.sample
+++ b/meta-yocto/conf/local.conf.sample
@@ -11,10 +11,10 @@
 CONF_VERSION = "1"
 
 # Uncomment and change to cache the files Poky downloads in an alternative
-# location, default it ${TOPDIR}/downloads
+# location, default is ${TOPDIR}/downloads
 #DL_DIR ?= "${TOPDIR}/downloads"
 # Uncomment and change to cache Poky's built staging output in an alternative
-# location, default ${TOPDIR}/sstate-cache
+# location, default is ${TOPDIR}/sstate-cache
 #SSTATE_DIR ?= "${TOPDIR}/sstate-cache"
 
 # Uncomment and set to allow bitbake to execute multiple tasks at once.
@@ -42,10 +42,13 @@ MACHINE ??= "qemux86"
 #MACHINE ?= "mpc8315e-rdb"
 #MACHINE ?= "routerstationpro"
 
+# Target distribution to set (from conf/distro/)
 DISTRO ?= "poky"
 # For bleeding edge / experimental / unstable package versions
 # DISTRO ?= "poky-bleeding"
 
+# BBMASK is a regular expression that can be used to tell BitBake to ignore
+# certain recipes.
 BBMASK = ""
 
 # EXTRA_IMAGE_FEATURES allows extra packages to be added to the generated images 
@@ -80,11 +83,9 @@ PACKAGE_CLASSES ?= "package_rpm package_ipk"
 # NOTE: if listing mklibs & prelink both, then make sure mklibs is before prelink
 USER_CLASSES ?= "image-mklibs image-prelink"
 
-# Uncomment this if you want BitBake to emit the log if a build fails.
+# Comment this out if you do not want BitBake to emit the log if a build fails.
 BBINCLUDELOGS = "yes"
 
-ENABLE_BINARY_LOCALE_GENERATION = "1"
-
 # The architecture to build SDK items for, by setting this you can build SDK
 # packages for architectures other than the host i.e. building i686 packages
 # on an x86_64 host.
@@ -100,6 +101,12 @@ ENABLE_BINARY_LOCALE_GENERATION = "1"
 #file://.* http://someserver.tld/share/sstate/ \n \
 #file://.* file:///some/local/dir/sstate/"
 
+# ENABLE_BINARY_LOCALE_GENERATION controls the generation of binary locale
+# packages at build time using qemu-native. Disabling it (by setting it to 0)
+# will save some build time at the expense of breaking i18n on devices with
+# less than 128MB RAM.
+ENABLE_BINARY_LOCALE_GENERATION = "1"
+
 # Set GLIBC_GENERATE_LOCALES to the locales you wish to generate should you not
 # wish to perform the time-consuming step of generating all LIBC locales.
 # NOTE: If removing en_US.UTF-8 you will also need to uncomment, and set
@@ -110,10 +117,10 @@ ENABLE_BINARY_LOCALE_GENERATION = "1"
 #IMAGE_LINGUAS ?= "en-gb"
 #LIMIT_BUILT_LOCALES ?= "POSIX en_GB"
 
-# This value is currently used by PSEUDO to determine if the recipe should
+# This value is currently used by pseudo to determine if the recipe should
 # build both the 32-bit and 64-bit wrapper libraries on a 64-bit build system.
 #
-# PSEUDO will attempt to determine if a 32-bit wrapper is necessary, but 
+# Pseudo will attempt to determine if a 32-bit wrapper is necessary, but
 # it doesn't always guess properly.  If you have 32-bit executables on 
 # your 64-bit build system, you likely want to set this to "0", 
 # otherwise you could end up with incorrect file attributes on the 
-- 
1.7.4.1

[PATCH 5/6] meta-yocto/conf/local.conf.sample: remove emenlow from machine list

[Paul Eggleton]

emenlow is not supported in meta-yocto (need meta-emenlow from meta-intel
for that).

Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
---
 meta-yocto/conf/local.conf.sample |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/meta-yocto/conf/local.conf.sample b/meta-yocto/conf/local.conf.sample
index 524a256..6498bb4 100644
--- a/meta-yocto/conf/local.conf.sample
+++ b/meta-yocto/conf/local.conf.sample
@@ -38,7 +38,6 @@ MACHINE ??= "qemux86"
 # Supported target hardware for demonstration purposes
 #MACHINE ?= "atom-pc"
 #MACHINE ?= "beagleboard"
-#MACHINE ?= "emenlow"
 #MACHINE ?= "mpc8315e-rdb"
 #MACHINE ?= "routerstationpro"
 
-- 
1.7.4.1

[PATCH 6/6] meta-yocto/conf/local.conf.sample: add rm_work example

[Paul Eggleton]

rm-work can save a lot of disk space and will be useful to a lot of
users, so add a comment explaining what it is for and why it's off by
default.

Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
---
 meta-yocto/conf/local.conf.sample |    6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)

diff --git a/meta-yocto/conf/local.conf.sample b/meta-yocto/conf/local.conf.sample
index 6498bb4..1977e07 100644
--- a/meta-yocto/conf/local.conf.sample
+++ b/meta-yocto/conf/local.conf.sample
@@ -139,3 +139,9 @@ NO32LIBS = "1"
 
 # Uncomment this if your host distribution provides the help2man tool.
 #ASSUME_PROVIDED += "help2man-native"
+
+# Uncomment this to delete work files as the build progresses rather than
+# keeping them around, which saves a lot of disk space. However, if any
+# problems arise it can be useful to have the work files to examine, which
+# is why it is off by default.
+#INHERIT += "rm_work"
-- 
1.7.4.1

Re: [PATCH 6/6] meta-yocto/conf/local.conf.sample: add rm_work example

[Koen Kooi]

Op 25 jul. 2011, om 16:28 heeft Paul Eggleton het volgende geschreven:

> rm-work can save a lot of disk space and will be useful to a lot of
> users, so add a comment explaining what it is for and why it's off by
> default.
> 
> Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
> ---
> meta-yocto/conf/local.conf.sample |    6 ++++++
> 1 files changed, 6 insertions(+), 0 deletions(-)
> 
> diff --git a/meta-yocto/conf/local.conf.sample b/meta-yocto/conf/local.conf.sample
> index 6498bb4..1977e07 100644
> --- a/meta-yocto/conf/local.conf.sample
> +++ b/meta-yocto/conf/local.conf.sample
> @@ -139,3 +139,9 @@ NO32LIBS = "1"
> 
> # Uncomment this if your host distribution provides the help2man tool.
> #ASSUME_PROVIDED += "help2man-native"
> +
> +# Uncomment this to delete work files as the build progresses rather than
> +# keeping them around, which saves a lot of disk space. However, if any
> +# problems arise it can be useful to have the work files to examine, which
> +# is why it is off by default.
> +#INHERIT += "rm_work"

That implies that you expect problems to happen by default :)

Re: [PATCH 6/6] meta-yocto/conf/local.conf.sample: add rm_work example

[Paul Eggleton]

On Monday 25 July 2011 15:44:32 Koen Kooi wrote:
> Op 25 jul. 2011, om 16:28 heeft Paul Eggleton het volgende geschreven:
> > +# Uncomment this to delete work files as the build progresses rather
> > than +# keeping them around, which saves a lot of disk space. However,
> > if any +# problems arise it can be useful to have the work files to
> > examine, which +# is why it is off by default.
> > +#INHERIT += "rm_work"
> 
> That implies that you expect problems to happen by default :)

It does say "if" ;)  I'm happy to take suggestions on an alternative 
comment...

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre

Re: [PATCH 6/6] meta-yocto/conf/local.conf.sample: add rm_work example

[Koen Kooi]

Op 25 jul. 2011, om 16:49 heeft Paul Eggleton het volgende geschreven:

> On Monday 25 July 2011 15:44:32 Koen Kooi wrote:
>> Op 25 jul. 2011, om 16:28 heeft Paul Eggleton het volgende geschreven:
>>> +# Uncomment this to delete work files as the build progresses rather
>>> than +# keeping them around, which saves a lot of disk space. However,
>>> if any +# problems arise it can be useful to have the work files to
>>> examine, which +# is why it is off by default.
>>> +#INHERIT += "rm_work"
>> 
>> That implies that you expect problems to happen by default :)
> 
> It does say "if" ;)  I'm happy to take suggestions on an alternative 
> comment…

I'd argue that people can turn it off if they encounter issues. And if the issue is a build failure, rm_work hasn't run on the f(l)ailing recipe anyway.

Re: [PATCH 6/6] meta-yocto/conf/local.conf.sample: add rm_work example

[Paul Eggleton]

On Monday 25 July 2011 15:54:18 Koen Kooi wrote:
> I'd argue that people can turn it off if they encounter issues. And if the
> issue is a build failure, rm_work hasn't run on the f(l)ailing recipe
> anyway.

Well, I only wanted to ensure it was available for users to enable in the 
default config file; actually enabling rm_work by default should really be a 
wider (different?) discussion.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre

Re: [PATCH 3/6] meta-yocto/conf/local.conf.sample: move advanced settings to extended sample file

[Saul Wold]

On 07/25/2011 07:28 AM, Paul Eggleton wrote:
> These settings are very unlikely to be needed by new users. Also add a
> comment pointing to the extended file as well as the Poky Reference
> Manual.
>
> Note that this removes the setting of PACKAGE_DEBUG_SPLIT_STYLE; however
> we check for the value that is not the default so this should not make
> any difference to the default behaviour.
>
> Signed-off-by: Paul Eggleton<paul.eggleton@linux.intel.com>
> ---
>   meta-yocto/conf/local.conf.sample          |   98 +++-------------------------
>   meta-yocto/conf/local.conf.sample.extended |   88 +++++++++++++++++++++++++
>   2 files changed, 97 insertions(+), 89 deletions(-)
>
> diff --git a/meta-yocto/conf/local.conf.sample b/meta-yocto/conf/local.conf.sample
> index 6ba92af..523a737 100644
> --- a/meta-yocto/conf/local.conf.sample
> +++ b/meta-yocto/conf/local.conf.sample
> @@ -1,3 +1,12 @@
> +# For more information on these configuration options, please see the
> +# Poky Reference Manual at:
> +#
> +#  http://www.yoctoproject.org/docs/poky-ref-manual/poky-ref-manual.html
> +#
> +# Note: there are additional more advanced configuration option examples
> +# (e.g. for debugging) in local.conf.sample.extended which can be found
> +# in meta-yocto/conf/
> +
>   # CONF_VERSION is increased each time build/conf/ changes incompatibly
>   CONF_VERSION = "1"
>
> @@ -64,15 +73,6 @@ EXTRA_IMAGE_FEATURES = "debug-tweaks"
>   #PACKAGE_CLASSES ?= "package_rpm package_deb package_ipk"
>   PACKAGE_CLASSES ?= "package_rpm package_ipk"
>
> -# mklibs library size optimization is more useful to smaller images,
> -# and less useful for bigger images. Also mklibs library optimization
> -# can break the ABI compatibility, so should not be applied to the
> -# images which are to be extended or upgraded later.
> -#This enabled mklibs library size optimization just for the specified image.
> -#MKLIBS_OPTIMIZED_IMAGES ?= "core-image-minimal"
> -#This enable mklibs library size optimization will be for all the images.
> -#MKLIBS_OPTIMIZED_IMAGES ?= "all"
> -
>   # A list of additional classes to use when building the system
>   # include 'image-mklibs' to reduce shared library files size for an image
>   # include 'image-prelink' in order to prelink the filesystem image
> @@ -80,65 +80,6 @@ PACKAGE_CLASSES ?= "package_rpm package_ipk"
>   # NOTE: if listing mklibs&  prelink both, then make sure mklibs is before prelink
>   USER_CLASSES ?= "image-mklibs image-prelink"
>
> -# POKYMODE controls the characteristics of the generated packages/images by
> -# telling poky which type of toolchain to use.
> -#
> -# Options include several different EABI combinations and a compatibility
> -# mode for the OABI mode poky previously used.
> -#
> -# The default is "eabi"
> -# Use "oabi" for machines with kernels<  2.6.18 on ARM for example.
> -# Use "external-MODE" to use the precompiled external toolchains where MODE
> -# is the type of external toolchain to use e.g. eabi.
> -# POKYMODE = "external-eabi"
> -
> -# Uncomment this to specify where BitBake should create its temporary files.
> -# Note that a full build of everything in OpenEmbedded will take GigaBytes of hard
> -# disk space, so make sure to free enough space. The default TMPDIR is
> -#<build directory>/tmp
> -#TMPDIR = "${COREBASE}/build/tmp"
> -
> -# The following are used to control options related to debugging.
> -#
> -# Uncomment this to change the optimization to make debugging easer, at the
> -# possible cost of performance.
> -# DEBUG_BUILD = "1"
> -#
> -# Uncomment this to disable the stripping of the installed binaries
> -# INHIBIT_PACKAGE_STRIP = "1"
> -#
> -# Uncomment this to disable the split of the debug information into -dbg files
> -# INHIBIT_PACKAGE_DEBUG_SPLIT = "1"
> -#
> -# When splitting debug information, the following controls the results of the
> -# file splitting.
> -#
> -#  .debug (default):
> -#    When splitting the debug information will be placed into
> -#    a .debug directory in the same dirname of the binary produced:
> -#      /bin/foo ->  /bin/.debug/foo
> -#
> -#  debug-file-directory:
> -#    When splitting the debug information will be placed into
> -#    a central debug-file-directory, /usr/lib/debug:
> -#      /bin/foo ->  /usr/lib/debug/bin/foo.debug
> -#
> -#    Any source code referenced in the debug symbols will be copied
> -#    and made available within the /usr/src/debug directory
> -#
> -PACKAGE_DEBUG_SPLIT_STYLE = '.debug'
> -# PACKAGE_DEBUG_SPLIT_STYLE = 'debug-file-directory'
> -
> -# Uncomment these to build a package such that you can use gprof to profile it.
> -# NOTE: This will only work with 'linux' targets, not
> -# 'linux-uclibc', as uClibc doesn't provide the necessary
> -# object files.  Also, don't build glibc itself with these
> -# flags, or it'll fail to build.
> -#
> -# PROFILE_OPTIMIZATION = "-pg"
> -# SELECTED_OPTIMIZATION = "${PROFILE_OPTIMIZATION}"
> -# LDFLAGS =+ "-pg"
> -
>   # Uncomment this if you want BitBake to emit the log if a build fails.
>   BBINCLUDELOGS = "yes"
>
> @@ -159,23 +100,6 @@ ENABLE_BINARY_LOCALE_GENERATION = "1"
>   #file://.* http://someserver.tld/share/sstate/ \n \
>   #file://.* file:///some/local/dir/sstate/"
>
> -# Set IMAGETEST to qemu if you want to build testcases and start
> -# testing in qemu after do_rootfs.
> -#IMAGETEST = "qemu"
> -
> -# By default test cases in sanity suite will be ran. If you want to run other
> -# test suite or specific test case(e.g. bat or boot test case under sanity suite),
> -# list them like following.
> -#TEST_SCEN = "sanity bat sanity:boot toolchain"
> -
> -#Because of the QEMU booting slowness issue(see bug #646 and #618), autobuilder
> -#may suffer a timeout issue when running sanity test. We introduce variable
> -#TEST_SERIALIZE here to reduce the time on sanity test. It is by default set
> -#to 1. Poky will start image and run cases in the same image without reboot
> -#or kill. If it is set to 0, the image will be copied and tested for each
> -#case, which will take much time.
> -#TEST_SERIALIZE = "1"
> -
>   # Set GLIBC_GENERATE_LOCALES to the locales you wish to generate should you not
>   # wish to perform the time-consuming step of generating all LIBC locales.
>   # NOTE: If removing en_US.UTF-8 you will also need to uncomment, and set
> @@ -207,9 +131,5 @@ NO32LIBS = "1"
>   #TERMCMD = "${KONSOLE_TERMCMD}"
>   #TERMCMDRUN = "${KONSOLE_TERMCMDRUN}"
>
Paul,

I noticed the TERMCMD* here and realized that maybe we need to update 
this default with XTERM_* or SCREEN_*, rather than KONSOLE, I realize 
that this is an example.

Sau!

> -# The network based PR service host and port
> -#PRSERV_HOST = "localhost"
> -#PRSERV_PORT = "8585"
> -
>   # Uncomment this if your host distribution provides the help2man tool.
>   #ASSUME_PROVIDED += "help2man-native"
> diff --git a/meta-yocto/conf/local.conf.sample.extended b/meta-yocto/conf/local.conf.sample.extended
> index 775d6d3..82e9ea9 100644
> --- a/meta-yocto/conf/local.conf.sample.extended
> +++ b/meta-yocto/conf/local.conf.sample.extended
> @@ -10,3 +10,91 @@
>   #               libc-posix-wchar-io"
>   #DISTRO_FEATURES = "alsa argp bluetooth ext2 irda largefile pcmcia usbgadget usbhost wifi xattr nfs zeroconf pci 3g ${DISTRO_FEATURES_LIBC}"
>
> +# mklibs library size optimization is more useful to smaller images,
> +# and less useful for bigger images. Also mklibs library optimization
> +# can break the ABI compatibility, so should not be applied to the
> +# images which are to be extended or upgraded later.
> +#This enabled mklibs library size optimization just for the specified image.
> +#MKLIBS_OPTIMIZED_IMAGES ?= "core-image-minimal"
> +#This enable mklibs library size optimization will be for all the images.
> +#MKLIBS_OPTIMIZED_IMAGES ?= "all"
> +
> +# POKYMODE controls the characteristics of the generated packages/images by
> +# telling poky which type of toolchain to use.
> +#
> +# Options include several different EABI combinations and a compatibility
> +# mode for the OABI mode poky previously used.
> +#
> +# The default is "eabi"
> +# Use "oabi" for machines with kernels<  2.6.18 on ARM for example.
> +# Use "external-MODE" to use the precompiled external toolchains where MODE
> +# is the type of external toolchain to use e.g. eabi.
> +# POKYMODE = "external-eabi"
> +
> +# Uncomment this to specify where BitBake should create its temporary files.
> +# Note that a full build of everything in OpenEmbedded will take GigaBytes of hard
> +# disk space, so make sure to free enough space. The default TMPDIR is
> +#<build directory>/tmp
> +#TMPDIR = "${COREBASE}/build/tmp"
> +
> +# The following are used to control options related to debugging.
> +#
> +# Uncomment this to change the optimization to make debugging easer, at the
> +# possible cost of performance.
> +# DEBUG_BUILD = "1"
> +#
> +# Uncomment this to disable the stripping of the installed binaries
> +# INHIBIT_PACKAGE_STRIP = "1"
> +#
> +# Uncomment this to disable the split of the debug information into -dbg files
> +# INHIBIT_PACKAGE_DEBUG_SPLIT = "1"
> +#
> +# When splitting debug information, the following controls the results of the
> +# file splitting.
> +#
> +#  .debug (default):
> +#    When splitting the debug information will be placed into
> +#    a .debug directory in the same dirname of the binary produced:
> +#      /bin/foo ->  /bin/.debug/foo
> +#
> +#  debug-file-directory:
> +#    When splitting the debug information will be placed into
> +#    a central debug-file-directory, /usr/lib/debug:
> +#      /bin/foo ->  /usr/lib/debug/bin/foo.debug
> +#
> +#    Any source code referenced in the debug symbols will be copied
> +#    and made available within the /usr/src/debug directory
> +#
> +# PACKAGE_DEBUG_SPLIT_STYLE = 'debug-file-directory'
> +
> +# Uncomment these to build a package such that you can use gprof to profile it.
> +# NOTE: This will only work with 'linux' targets, not
> +# 'linux-uclibc', as uClibc doesn't provide the necessary
> +# object files.  Also, don't build glibc itself with these
> +# flags, or it'll fail to build.
> +#
> +# PROFILE_OPTIMIZATION = "-pg"
> +# SELECTED_OPTIMIZATION = "${PROFILE_OPTIMIZATION}"
> +# LDFLAGS =+ "-pg"
> +
> +# Set IMAGETEST to qemu if you want to build testcases and start
> +# testing in qemu after do_rootfs.
> +#IMAGETEST = "qemu"
> +
> +# By default test cases in sanity suite will be ran. If you want to run other
> +# test suite or specific test case(e.g. bat or boot test case under sanity suite),
> +# list them like following.
> +#TEST_SCEN = "sanity bat sanity:boot toolchain"
> +
> +#Because of the QEMU booting slowness issue(see bug #646 and #618), autobuilder
> +#may suffer a timeout issue when running sanity test. We introduce variable
> +#TEST_SERIALIZE here to reduce the time on sanity test. It is by default set
> +#to 1. Poky will start image and run cases in the same image without reboot
> +#or kill. If it is set to 0, the image will be copied and tested for each
> +#case, which will take much time.
> +#TEST_SERIALIZE = "1"
> +
> +# The network based PR service host and port
> +#PRSERV_HOST = "localhost"
> +#PRSERV_PORT = "8585"
> +

Re: [PATCH 0/6] meta-yocto: cleanup of local.conf.sample

[Saul Wold]

On 07/25/2011 07:28 AM, Paul Eggleton wrote:
> A number of changes aimed at cleaning up local.conf.sample so it is easier
> to deal with for new users. These are not necessarily final, happy to take
> comments/suggestions.
>
How is this going to affect the oe-core local.conf.sample?

Sau!


> The following changes since commit eea4bd731ef286516ee1fb934fe2222a09c06e70:
>
>    mpc8315e-rdb: Set TARGET_FPU correct (2011-07-22 09:02:58 -0700)
>
> are available in the git repository at:
>    git://git.pokylinux.org/poky-contrib paule/local-conf-cleanup
>    http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=paule/local-conf-cleanup
>
> Paul Eggleton (6):
>    meta-yocto/local.conf.sample.extended: update DISTRO_FEATURES
>    meta-yocto/local.conf.sample: remove oprofileui-native related
>      settings
>    meta-yocto/conf/local.conf.sample: move advanced settings to extended
>      sample file
>    meta-yocto/conf/local.conf.sample: improve comments/layout
>    meta-yocto/conf/local.conf.sample: remove emenlow from machine list
>    meta-yocto/conf/local.conf.sample: add rm_work example
>
>   meta-yocto/conf/local.conf.sample          |  130 ++++++----------------------
>   meta-yocto/conf/local.conf.sample.extended |   92 +++++++++++++++++++-
>   2 files changed, 118 insertions(+), 104 deletions(-)
>

Re: [PATCH 0/6] meta-yocto: cleanup of local.conf.sample

[Paul Eggleton]

On Monday 25 July 2011 16:31:40 Saul Wold wrote:
> On 07/25/2011 07:28 AM, Paul Eggleton wrote:
> > A number of changes aimed at cleaning up local.conf.sample so it is
> > easier to deal with for new users. These are not necessarily final,
> > happy to take comments/suggestions.
> 
> How is this going to affect the oe-core local.conf.sample?

That's a good question - it is of course a separate file. I guess it depends on 
how people using oe-core feel about moving these options out into a separate 
file. I was intending to tackle the oe-core file as a separate exercise.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre

Re: [PATCH 3/6] meta-yocto/conf/local.conf.sample: move advanced settings to extended sample file

[Paul Eggleton]

On Monday 25 July 2011 16:31:09 Saul Wold wrote:
> On 07/25/2011 07:28 AM, Paul Eggleton wrote:
> >   #TERMCMD = "${KONSOLE_TERMCMD}"
> >   #TERMCMDRUN = "${KONSOLE_TERMCMDRUN}"
> 
> I noticed the TERMCMD* here and realized that maybe we need to update
> this default with XTERM_* or SCREEN_*, rather than KONSOLE, I realize
> that this is an example.

You're probably right, and actually this reminds me, I meant to add a comment 
that KONSOLE_TERMCMD will only work with Konsole from KDE 3.x as well.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre

Re: [PATCH 0/6] meta-yocto: cleanup of local.conf.sample

[Paul Eggleton]

On Monday 25 July 2011 16:37:08 Paul Eggleton wrote:
> On Monday 25 July 2011 16:31:40 Saul Wold wrote:
> > On 07/25/2011 07:28 AM, Paul Eggleton wrote:
> > > A number of changes aimed at cleaning up local.conf.sample so it is
> > > easier to deal with for new users. These are not necessarily final,
> > > happy to take comments/suggestions.
> > 
> > How is this going to affect the oe-core local.conf.sample?
> 
> That's a good question - it is of course a separate file. I guess it
> depends on how people using oe-core feel about moving these options out
> into a separate file. I was intending to tackle the oe-core file as a
> separate exercise.

I've posted patches to update the OE-core version of this file first (sans the 
splitting). We can bring over those patches and then decide what we want to do 
for everything else; so consider this patchset superseded for now.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre

Re: [PATCH 6/6] meta-yocto/conf/local.conf.sample: add rm_work example

[Darren Hart]

On 07/25/2011 07:49 AM, Paul Eggleton wrote:
> On Monday 25 July 2011 15:44:32 Koen Kooi wrote:
>> Op 25 jul. 2011, om 16:28 heeft Paul Eggleton het volgende geschreven:
>>> +# Uncomment this to delete work files as the build progresses rather
>>> than +# keeping them around, which saves a lot of disk space. However,

This reads as keeping them around save a lot of disk space to me. You
can add a comma after progresses, or reword to group the modifier with
the subject.

>>> if any +# problems arise it can be useful to have the work files to
>>> examine, which +# is why it is off by default.
>>> +#INHERIT += "rm_work"
>>
>> That implies that you expect problems to happen by default :)
> 
> It does say "if" ;)  I'm happy to take suggestions on an alternative 
> comment...

If we assume people will be adding and developing recipes as part of the
typical usage model, then yes, I would expect problems to happen by
default - it is development afterall. :)


-- 
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel