Bug 9665

Summary: quoting bug in poky/scripts/contrib/mkefidisk.sh
Product: [Runtime] General Runtime Reporter: Troy D. Hanson <troy.hanson>
Component: General RuntimeAssignee: Saul Wold <sgw>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: Medium CC: akuster, sgw
Version: unspecified   
Target Milestone: 2.2.2   
Hardware: x86   
OS: Multiple   
See Also: https://bugzilla.yoctoproject.org/show_bug.cgi?id=10451
https://bugzilla.yoctoproject.org/show_bug.cgi?id=10468
Whiteboard:
OS type for building Yocto: --- Type of Regression: ---
Verified: Documentation change: No (bug/feature does not impact docs)
Attachments:
Description Flags
Patch for mkefidisk.sh
none
Corrected patch none

Description Troy D. Hanson 2016-05-23 22:20:42 UTC
Created attachment 3185 [details]
Patch for mkefidisk.sh

The attached patch corrects a quoting bug in mkefidisk.sh. The bug writes a startup.nsh with embedded control characters. This happens because \b etc are control sequences to the shell echo command. The resulting startup.nsh causes the bootup to fail, and the user is dropped into the EFI shell to manually run startup.nsh.

People following the Yocto Quick Start Guide following the Minnowboard Max example encounter the effect of this bug.
Comment 1 Troy D. Hanson 2016-05-23 23:39:26 UTC
Created attachment 3186 [details]
Corrected patch
Comment 2 Saul Wold 2016-05-25 14:43:42 UTC
Troy, could you please send this patch to the openembedded-core@lists.openembedded.org mailing list?  Add your comment below as the commit message.

If you are not able please assign the bug back to me and I will do it.
Comment 3 Benjamin Esquivel 2016-10-20 15:03:13 UTC
if bug 10468 deprecates mkefidisk.sh then this change would be valid just for pre 2.3 point releases.
Comment 4 Saul Wold 2017-06-29 15:06:48 UTC
So this is not really in a newer release yet, so a patch to master is needed first, will be provided shortly.