Bug 3163

Summary: Beta - bitbake error message doesn't tell me what's wrong with my layer
Product: [Build System, Metadata & Runtime] BitBake Reporter: Dave Stewart <david.c.stewart>
Component: bitbakeAssignee: Paul Eggleton <bluelightning>
Status: VERIFIED FIXED QA Contact:
Severity: major    
Priority: Medium+ CC: alexandrux.palalau, bluelightning, corneliux.stoicescu, jessica.zhang, poky.bs.watcher, poky.watcher
Version: 1.3   
Target Milestone: 1.3 M5   
Hardware: x86   
OS: Multiple   
Whiteboard:
OS type for building Yocto: --- Type of Regression: ---
Verified: Documentation change: ---

Description Dave Stewart 2012-09-24 21:22:22 UTC 
You might want to grab the meta-dlna right away before it changes...

To reproduce:

1. Set up the 1.3 beta
2. clone the meta-dlna repository
3. Edit bblayers.conf as the README in the layer instructs, adding meta-guacamayo and meta-kvm
4. Execute bitbake, see the transcript below:

[davest@davest-f17 yp-beta-build]$ bitbake guacamayo-image-mediaserver
DEBUG: Removed the following variables from the environment: PSEUDO_BINDIR, LD_LIBRARY_PATH, HTTP_PROXY, PSEUDO_LOCALSTATEDIR, NO_PROXY, GJS_DEBUG_OUTPUT, FTP_PROXY, LESSOPEN, QT_IM_MODULE, PSEUDO_LIBDIR, XDG_VTNR, GNOME_KEYRING_CONTROL, WINDOWPATH, XDG_RUNTIME_DIR, LANG, HISTCONTROL, SHLVL, HISTSIZE, WINDOWID, _, IMSETTINGS_INTEGRATE_DESKTOP, BBFETCH2, USERNAME, IMSETTINGS_MODULE, CCACHE_HASHDIR, GDMSESSION, XMODIFIERS, QT_GRAPHICSSYSTEM_CHECKED, XDG_SESSION_ID, BUILDDIR, PSEUDO_OPTS, HOSTNAME, PSEUDO_PREFIX, HTTPS_PROXY, CVS_RSH, LD_PRELOAD, MAIL, GNOME_DESKTOP_SESSION_ID, LS_COLORS, GJS_DEBUG_TOPICS, XDG_SEAT
DEBUG: UI 'knotty' does not require extra cache!
DEBUG: Found bblayers.conf (/home/davest/poky-1.3-beta/yp-beta-build/conf/bblayers.conf)
DEBUG: LOAD /home/davest/poky-1.3-beta/yp-beta-build/conf/bblayers.conf
ERROR: Unable to parse /home/davest/poky-1.3-beta/yp-beta-build/conf/bblayers.conf: ParseError at /home/davest/poky-1.3-beta/yp-beta-build/conf/bblayers.conf:13: unparsed line: 'BBLAYERS ?= " /home/davest/poky-1.3-beta/yp-beta/meta /home/davest/poky-1.3-beta/yp-beta/meta-yocto /home/davest/poky-1.3-beta/meta-dlna/meta-guacamayo /home/davest/poky-1.3-beta/meta-dlna/meta-kvm '
[davest@davest-f17 yp-beta-build]$
Comment 1 Dave Stewart 2012-09-25 04:55:21 UTC 
In another bug report, Paul says the parse error might be due to the tabs-> spaces change. Do you think you could make the error message more explicit that this is the problem?
Comment 2 Richard Purdie 2012-09-25 08:00:03 UTC 
Can you paste exactly what this bblayers.conf contains. In this case I think bitbake is right about the error, I think there was a typo in bblayers.conf. Perhaps a missing " character at the end?

If I put the correct thing in bblayers.conf, I see:

WARNING: Variable populate_packages contains tabs, please remove these (/media/data1/build1/poky/meta-dlna/meta-guacamayo/recipes-devtools/orc/orc_0.4.14.bb)                                      | ETA:  --:--:--
NOTE: Error expanding variable populate_packages
NOTE: Error during finalise of /media/data1/build1/poky/meta-dlna/meta-guacamayo/recipes-devtools/orc/orc_0.4.14.bb
ERROR: Unable to parse /media/data1/build1/poky/meta-dlna/meta-guacamayo/recipes-devtools/orc/orc_0.4.14.bb
ERROR: Command execution failed: Exited with 1

which is a bit clearer about the fact it is a tabs vs. spaces issue.
Comment 3 Dave Stewart 2012-09-25 15:45:52 UTC 
Yeah, could be. I was trying to follow the README on the layer.

[davest@davest-f17 yp-beta-build]$ more conf/bblayers.conf
# LAYER_CONF_VERSION is increased each time build/conf/bblayers.conf
# changes incompatibly
LCONF_VERSION = "5"

BBPATH = "${TOPDIR}"
BBFILES ?= ""

BBLAYERS ?= " \
  /home/davest/poky-1.3-beta/yp-beta/meta \
  /home/davest/poky-1.3-beta/yp-beta/meta-yocto \
  /home/davest/poky-1.3-beta/meta-dlna/meta-guacamayo \
  /home/davest/poky-1.3-beta/meta-dlna/meta-kvm \

  "
Comment 4 Paul Eggleton 2012-09-26 09:05:05 UTC 
If that's a verbatim paste, I think the problem is the extra blank line before the closing quote.
Comment 5 Dave Stewart 2012-09-26 17:44:34 UTC 
Ah ... again, just following the README for the layer too closely!

Yes, removing the extra newline causes a much better error message about tabs. :-) Thanks for the tip
Comment 6 Paul Eggleton 2012-09-26 17:49:26 UTC 
Right, it is broken in the README in meta-dlna as you say - we should definitely fix that.
Comment 7 Paul Eggleton 2012-09-28 08:43:59 UTC 
Saul has fixed the README in meta-dlna to remove the extra blank line; I think we can call this one fixed.
Comment 8 Alexandru Palalau 2012-11-01 15:01:53 UTC 
Verified on master branch commit ID: 074016d1692d88ed5a4e245c242c2ffd901a490d