Bug 1279 - If parse fails Hob just spins forever
Summary: If parse fails Hob just spins forever
Status: VERIFIED FIXED
Alias: None
Product: Hob
Classification: Build System, Metadata & Runtime
Component: hob (show other bugs)
Version: unspecified
Hardware: x86 Multiple
: High major
Target Milestone: 1.1
Assignee: Joshua Lock - Disabled
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2011-07-22 16:40 UTC by Joshua Lock - Disabled
Modified: 2011-08-02 16:12 UTC (History)
6 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Joshua Lock - Disabled 2011-07-22 16:40:54 UTC
I'd enabled a layer in my testing which included bbappends for non-existent files and caused a parse error. The GUI just kept spinning regardless...

If I run the knotty with the same bblayers.conf:

joshual@scimitar:~/Projects/Yocto/oe-core/build [master]
$ bitbake core-image-minimal -n
Loading cache: 100% |###########################################| ETA:  00:00:00
Loaded 1387 entries from dependency cache.
ERROR: No recipes available for:
  /home/joshual/Projects/Yocto/oe-core/meta-demo/recipes-kernel/linux/linux-wrs_git.bbappend
  /home/joshual/Projects/Yocto/oe-core/meta-demo/recipes-kernel/linux/linux-yocto_git.bbappend
ERROR: Command execution failed: Exited with 1
Comment 1 Joshua Lock - Disabled 2011-07-28 11:46:12 UTC
The knotty (command-line) interface exits with an error in this situation. The GUI should report an error and if possible aid the user in diagnosing and fixing it.
Comment 2 Joshua Lock - Disabled 2011-07-28 15:13:50 UTC
The solution I've come up with is to show the user a dialog telling them there's been an unrecoverable error and then showing the (often terse) error message from the underlying system. The dialog has an "Exit" button for the user to acknowledge the message and quit the program.
Comment 4 Joshua Lock - Disabled 2011-08-02 16:12:30 UTC
Fix is merged and I've benefited from the change already.