<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugzilla.yoctoproject.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugzilla.yoctoproject.org/"
          
          maintainer="it-coreprojects-helpdesk@linuxfoundation.org"
>

    <bug>
          <bug_id>7627</bug_id>
          
          <creation_ts>2015-04-17 22:24:28 +0000</creation_ts>
          <short_desc>Kernel checkout requires master branch</short_desc>
          <delta_ts>2015-04-28 19:12:38 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>7</classification_id>
          <classification>Build System, Metadata &amp; Runtime</classification>
          <product>OE-Core</product>
          <component>kernel</component>
          <version>unspecified</version>
          <rep_platform>x86</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>Undecided</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Steve Pomeroy">steve</reporter>
          <assigned_to name="Bruce Ashfield">bruce.ashfield</assigned_to>
          <cc>bruce.ashfield</cc>
    
    <cc>tom.zanussi</cc>
          
          
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>No (bug/feature does not impact docs)</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>50396</commentid>
    <comment_count>0</comment_count>
      <attachid>2480</attachid>
    <who name="Steve Pomeroy">steve</who>
    <bug_when>2015-04-17 22:24:28 +0000</bug_when>
    <thetext>Created attachment 2480
aforementioned patch

http://cgit.openembedded.org/openembedded-core/commit/?id=2ffa3f8be6996877cd552ff22260de35c19c413d introduced a bug where a branch named &quot;master&quot; is required for a kernel checkout to succeed (while usually true, this isn&apos;t necessarily the case in all repositories). The logic seems to have been inverted. 

Attached is a patch that should address the situation.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>50397</commentid>
    <comment_count>1</comment_count>
    <who name="Bruce Ashfield">bruce.ashfield</who>
    <bug_when>2015-04-18 01:15:01 +0000</bug_when>
    <thetext>This is already fixed in master (as mentioned on the mailing list). The
logic was inverted, but we&apos;ll now only fall back to master if the machine
branch doesn&apos;t exist.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>50398</commentid>
    <comment_count>2</comment_count>
    <who name="Bruce Ashfield">bruce.ashfield</who>
    <bug_when>2015-04-18 01:17:06 +0000</bug_when>
    <thetext>In particular, I&apos;m not all that concerned about working with
repositories with no master. If we handled every single repository
variant, the complexity becomes to much.

Having a well known (and sane) fallback of master is appropriate in
nearly every case.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>50400</commentid>
    <comment_count>3</comment_count>
    <who name="Steve Pomeroy">steve</who>
    <bug_when>2015-04-18 14:42:20 +0000</bug_when>
    <thetext>Ah. I didn&apos;t see af691920c0e289fd0eb13d94cad03c799375af21 as I was only looking on the dizzy branch. Is this a change that can make it to 1.7.2?

Just leaving it be the &quot;master&quot; branch sounds good to me.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>50401</commentid>
    <comment_count>4</comment_count>
    <who name="Bruce Ashfield">bruce.ashfield</who>
    <bug_when>2015-04-19 03:59:53 +0000</bug_when>
    <thetext>I&apos;ll post the patch for a backport to stable.

Thanks for working through this, and sorry for issues it may
be causing for your builds!

(In reply to comment #3)
&gt; Ah. I didn&apos;t see af691920c0e289fd0eb13d94cad03c799375af21 as I was only
&gt; looking on the dizzy branch. Is this a change that can make it to 1.7.2?
&gt; 
&gt; Just leaving it be the &quot;master&quot; branch sounds good to me.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>50510</commentid>
    <comment_count>5</comment_count>
    <who name="Bruce Ashfield">bruce.ashfield</who>
    <bug_when>2015-04-24 20:03:36 +0000</bug_when>
    <thetext>send the patch to the list.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>50580</commentid>
    <comment_count>6</comment_count>
    <who name="Bruce Ashfield">bruce.ashfield</who>
    <bug_when>2015-04-28 19:12:38 +0000</bug_when>
    <thetext>Patch posted for backport.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>2480</attachid>
            <date>2015-04-17 22:24:28 +0000</date>
            <delta_ts>2015-04-17 22:24:28 +0000</delta_ts>
            <desc>aforementioned patch</desc>
            <filename>0001-Fixes-kernel-checkout-when-master-branch-is-missing.patch</filename>
            <type>text/plain</type>
            <size>1188</size>
            <attacher name="Steve Pomeroy">steve</attacher>
            
              <data encoding="base64">RnJvbSBmNmFhZDlhYWY5YWYyZjBmNWQ3MmE0ZTE4N2QwNjNlYzA4N2Q0NGE4IE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBTdGV2ZSBQb21lcm95IDxzdGV2ZXBAdGhlbGV2ZWx1cC5jb20+
CkRhdGU6IEZyaSwgMTcgQXByIDIwMTUgMTg6MTk6MzcgLTA0MDAKU3ViamVjdDogW1BBVENIXSBG
aXhlcyBrZXJuZWwgY2hlY2tvdXQgd2hlbiAibWFzdGVyIiBicmFuY2ggaXMgbWlzc2luZwoKKiBG
aXhlcyBpbnZlcnRlZCBicmFuY2ggY2hlY2tvdXQgbG9naWMuCiogTm90IGFsbCByZXBvc2l0b3Jp
ZXMgaGF2ZSBhICJtYXN0ZXIiIGJyYW5jaCwgc28gdGhpcyByZXRyaWV2ZXMgdGhlIEhFQUQKICBy
ZWZlcmVuY2UgYW5kIHVzZXMgdGhhdCBhcyBhIGZhbGxiYWNrLgotLS0KIG1ldGEvY2xhc3Nlcy9r
ZXJuZWwteW9jdG8uYmJjbGFzcyB8ICAgIDUgKysrLS0KIDEgZmlsZSBjaGFuZ2VkLCAzIGluc2Vy
dGlvbnMoKyksIDIgZGVsZXRpb25zKC0pCgpkaWZmIC0tZ2l0IGEvbWV0YS9jbGFzc2VzL2tlcm5l
bC15b2N0by5iYmNsYXNzIGIvbWV0YS9jbGFzc2VzL2tlcm5lbC15b2N0by5iYmNsYXNzCmluZGV4
IGY0MmE1YzIuLmEzMzI3NzQgMTAwNjQ0Ci0tLSBhL21ldGEvY2xhc3Nlcy9rZXJuZWwteW9jdG8u
YmJjbGFzcworKysgYi9tZXRhL2NsYXNzZXMva2VybmVsLXlvY3RvLmJiY2xhc3MKQEAgLTIxMSw4
ICsyMTEsOSBAQCBkb19rZXJuZWxfY2hlY2tvdXQoKSB7CiAJIyBDcmVhdGUgYSB3b3JraW5nIHRy
ZWUgY29weSBvZiB0aGUga2VybmVsIGJ5IGNoZWNraW5nIG91dCBhIGJyYW5jaAogCW1hY2hpbmVf
YnJhbmNoPSIke0AgZ2V0X21hY2hpbmVfYnJhbmNoKGQsICIke0tCUkFOQ0h9IiApfSIKIAlnaXQg
c2hvdy1yZWYgLS1xdWlldCAtLXZlcmlmeSAtLSAicmVmcy9oZWFkcy8ke21hY2hpbmVfYnJhbmNo
fSIKLQlpZiBbICQ/IC1lcSAwIF07IHRoZW4KLQkJbWFjaGluZV9icmFuY2g9Im1hc3RlciIKKwlp
ZiBbICQ/IC1uZSAwIF07IHRoZW4KKwkJIyBVc3VhbGx5IHRoaXMgaXMgdGhlICJtYXN0ZXIiIGJy
YW5jaCwgYnV0IG5vdCBhbHdheXMuCisJCW1hY2hpbmVfYnJhbmNoPSIkKGdpdCBzeW1ib2xpYy1y
ZWYgSEVBRCB8IHNlZCAncyUuKi9cKFteL10qXCkkJVwxJScpIgogCWZpCiAKIAkjIGNoZWNrb3V0
IGFuZCBjbG9iYmVyIGFueSB1bmltcG9ydGFudCBmaWxlcwotLSAKMS43LjkuNQoK
</data>

          </attachment>
      

    </bug>

</bugzilla>