<?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>14709</bug_id>
          
          <creation_ts>2022-02-01 14:01:38 +0000</creation_ts>
          <short_desc>master as 3.4.x - devtool + github</short_desc>
          <delta_ts>2025-02-14 16:48:20 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>6</classification_id>
          <classification>Yocto Project Subprojects</classification>
          <product>eSDK</product>
          <component>eSDK</component>
          <version>3.4</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>Medium+</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>4.0 M3</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Robert Berger">pokylinux</reporter>
          <assigned_to name="Robert Berger">pokylinux</assigned_to>
          <cc>randy.macleod</cc>
    
    <cc>sgw</cc>
          
          <qa_contact name="Francisco Pedraza">francisco.j.pedraza.gonzalez</qa_contact>
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>Don&apos;t know</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>92441</commentid>
    <comment_count>0</comment_count>
    <who name="Robert Berger">pokylinux</who>
    <bug_when>2022-02-01 14:01:38 +0000</bug_when>
    <thetext>I am testing poky:

commit c3f50f747c8ee739919268e1c10980b93d1f3e64
Author: Richard Purdie &lt;richard.purdie@linuxfoundation.org&gt;
Date:   Mon Jan 17 21:56:15 2022 +0000

plus some other layers.

In my eSDK/devtool testcase I do that:

devtool add nettoe https://github.com/RobertBerger/nettoe.git

It creates a recipe with this SRC_URI:

SRC_URI = &quot;git://github.com/RobertBerger/nettoe.git;protocol=https&quot;

... but when I do that:

devtool build-image core-image-minimal

I get this warning:

WARNING: URL: git://github.com/RobertBerger/nettoe.git;protocol=https does not set any branch parameter. The future default branch used by tools and repositories is uncertain and we will therefore soon require this is set in all git URLs.

I totally get it, but shouldn&apos;t devtool automatically generate something which does not spit out a warning?

Especially when the doc claims that --srcbranch default is master?

devtool add --help
...
--srcbranch SRCBRANCH, -B SRCBRANCH
                        Branch in source repository if fetching from an SCM such 
                        as git (default master)
...

Rhis is my workaround for now:

devtool add --srcbranch master nettoe https://github.com/RobertBerger/nettoe.git

since now the SRC_URI is:

SRC_URI = &quot;git://github.com/RobertBerger/nettoe.git;protocol=https;branch=master&quot;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>92472</commentid>
    <comment_count>1</comment_count>
    <who name="Saul Wold">saul.wold</who>
    <bug_when>2022-02-03 17:05:58 +0000</bug_when>
    <thetext>Hi Robert,

I am reviewing this bug and I can see a change from Stefan

392f2933b6 Stefan Herbrechtsmeier recipetool: Set master branch only as fallback

That does exactly what you asking about, it was merged back in December and the commit you are using seems to be from the development branch of 3.5.  Can you confirm you have the above commit.

I ran both recipetool create and devtool add and the generated BB file contains a correct SRCURI

SRC_URI = &quot;git://github.com/RobertBerger/nettoe.git;protocol=https;branch=master&quot;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>92551</commentid>
    <comment_count>2</comment_count>
    <who name="Robert Berger">pokylinux</who>
    <bug_when>2022-02-14 14:24:27 +0000</bug_when>
    <thetext>It appears that I have this commit:

$ git rev-list 2022-01-15-master-as-honister-3.4.66+ | grep `git rev-parse 392f2933b6`
392f2933b6d4cc213f4498b70338451a5973bd07

git log 392f2933b6d4cc213f4498b70338451a5973bd07

commit 392f2933b6d4cc213f4498b70338451a5973bd07
Author: Stefan Herbrechtsmeier &lt;stefan.herbrechtsmeier@weidmueller.com&gt;
Date:   Wed Dec 8 11:18:35 2021 +0100

    recipetool: Set master branch only as fallback
    
    The commit &apos;meta/scripts: Manual git url branch additions (dc53fe75cc)&apos;
    sets the branch= parameter too early to master and thereby breaks the
    -B/--srcbranch option.
    
    ERROR: branch= parameter and -B/--srcbranch option cannot both be specified - use one or the other
    
    (From OE-Core rev: 34ece8030e7a6a100b5e3e7b94e6c786c0e199a6)
    
    Signed-off-by: Stefan Herbrechtsmeier &lt;stefan.herbrechtsmeier@weidmueller.com&gt;
    Signed-off-by: Richard Purdie &lt;richard.purdie@linuxfoundation.org&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>92552</commentid>
    <comment_count>3</comment_count>
    <who name="Robert Berger">pokylinux</who>
    <bug_when>2022-02-14 14:31:15 +0000</bug_when>
    <thetext>BTW I am on poky master commit c3f50f747c8ee739919268e1c10980b93d1f3e64 (plus some of my own hacks).

https://github.com/RobertBerger/poky

https://github.com/RobertBerger/poky/tree/2022-01-15-master-as-honister-3.4.66+</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>92554</commentid>
    <comment_count>4</comment_count>
    <who name="Robert Berger">pokylinux</who>
    <bug_when>2022-02-14 16:01:20 +0000</bug_when>
    <thetext>It appears that I have this commit:

$ git rev-list 2022-01-15-master-as-honister-3.4.66+ | grep `git rev-parse 392f2933b6`
392f2933b6d4cc213f4498b70338451a5973bd07

git log 392f2933b6d4cc213f4498b70338451a5973bd07

commit 392f2933b6d4cc213f4498b70338451a5973bd07
Author: Stefan Herbrechtsmeier &lt;stefan.herbrechtsmeier@weidmueller.com&gt;
Date:   Wed Dec 8 11:18:35 2021 +0100


and this fixes it!!!

Sorry for the confusion.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>92555</commentid>
    <comment_count>5</comment_count>
    <who name="Saul Wold">saul.wold</who>
    <bug_when>2022-02-14 16:03:30 +0000</bug_when>
    <thetext>Stefan&apos;s patch does indeed fix the issue.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>101014</commentid>
    <comment_count>6</comment_count>
    <who name="Randy MacLeod">randy.macleod</who>
    <bug_when>2025-02-14 16:46:36 +0000</bug_when>
    <thetext>bulk change to add Saul&apos;s non-WR address.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>101058</commentid>
    <comment_count>7</comment_count>
    <who name="Randy MacLeod">randy.macleod</who>
    <bug_when>2025-02-14 16:48:20 +0000</bug_when>
    <thetext>Bulk change: Remove Saul&apos;s old WR address.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>