Bug 5373 - kernel-yocto::do_validate_kernel test for machine branch broken
Summary: kernel-yocto::do_validate_kernel test for machine branch broken
Status: RESOLVED INVALID
Alias: None
Product: Kernel
Classification: Yocto Project Subprojects
Component: linux-yocto (show other bugs)
Version: unspecified
Hardware: All Multiple
: Medium normal
Target Milestone: 1.5.1
Assignee: Bruce Ashfield
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2013-10-22 10:00 UTC by Ola Nilsson
Modified: 2013-10-28 11:51 UTC (History)
3 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: No (bug/feature does not impact docs)


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ola Nilsson 2013-10-22 10:00:08 UTC
In do_validate_kernel there is a test for whether the machine_branch exists. 
The test is whether the return value of git show-ref --verify is exactly one.
At least on my machine the return value is 128 when the branch does not exist.
Maybe the test should be whether the return value is not 0?
Comment 1 Darren Hart 2013-10-28 10:34:04 UTC
Bruce, this looks like a valid report to me. Giving this one to you for the tooling.
Comment 2 Bruce Ashfield 2013-10-28 11:50:54 UTC
No we really want to check for 1:

       --verify
           Enable stricter reference checking by requiring an exact ref path. Aside from returning an error code of 1, it will also print an error
           message if --quiet was not specified.


We can't target all versions of git at once, so I'm continuing to go with
the man page and recent sources.
Comment 3 Bruce Ashfield 2013-10-28 11:51:48 UTC
I should add that if we can show a real tree, where this fails, and
either a cryptic error message comes out, or another check doesn't
catch the problem later, I'll revisit.