<?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>11385</bug_id>
          
          <creation_ts>2017-04-19 18:04:29 +0000</creation_ts>
          <short_desc>poky-container: clarify that meta-data should be checked out using native tools that run the host and not with tools in container</short_desc>
          <delta_ts>2024-07-18 14:51:33 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>6</classification_id>
          <classification>Yocto Project Subprojects</classification>
          <product>CROPS</product>
          <component>crops-default</component>
          <version>2.1.2</version>
          <rep_platform>x86</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>INVALID</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>Medium</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>5.1</target_milestone>
          
          <blocked>11097</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Henry Bruce">henry.bruce</reporter>
          <assigned_to name="Tim Orling">tim.orling</assigned_to>
          <cc>amber.n.elliot</cc>
    
    <cc>brian.avery</cc>
    
    <cc>javier.carrasco.cruz</cc>
    
    <cc>pidge</cc>
    
    <cc>randy.macleod</cc>
    
    <cc>stephano</cc>
    
    <cc>tim.orling</cc>
    
    <cc>tvgamblin</cc>
          
          
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>Yes (doc changes required)</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>72666</commentid>
    <comment_count>0</comment_count>
    <who name="Henry Bruce">henry.bruce</who>
    <bug_when>2017-04-19 18:04:29 +0000</bug_when>
    <thetext>Google&apos;s repo tool is a popular way of checking out multiple repos at known revisions so is very useful when setting up Yocto layers.

For more about repo, see https://source.android.com/source/downloading#installing-repo

For an example of repo in use, see https://github.com/intel-aero/meta-intel-aero/wiki/Quickstart-Guide#yocto-for-intel-aero.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>72667</commentid>
    <comment_count>1</comment_count>
    <who name="Henry Bruce">henry.bruce</who>
    <bug_when>2017-04-19 18:15:08 +0000</bug_when>
    <thetext>As shown by initial comment it is not obvious for developers new to containers that the poky-container should *not* be used to obtain the meta-data, but just to build it.

Docs at https://github.com/crops/poky-container/blob/master/README.md should be update accordingly.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>75455</commentid>
    <comment_count>2</comment_count>
    <who name="brian avery">brian.avery</who>
    <bug_when>2017-07-30 13:31:00 +0000</bug_when>
    <thetext>Moving to M3</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>76170</commentid>
    <comment_count>3</comment_count>
    <who name="brian avery">brian.avery</who>
    <bug_when>2017-08-25 22:06:17 +0000</bug_when>
    <thetext>to m4</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>78623</commentid>
    <comment_count>4</comment_count>
    <who name="brian avery">brian.avery</who>
    <bug_when>2017-12-08 22:06:57 +0000</bug_when>
    <thetext>Moving to Joshua

seems like this should be the documentation bug for the solution in Bug 11097&apos;s comments.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>97137</commentid>
    <comment_count>5</comment_count>
    <who name="Randy MacLeod">randy.macleod</who>
    <bug_when>2023-10-30 15:37:30 +0000</bug_when>
    <thetext>Build move to 5.0 -- ../Randy</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>98499</commentid>
    <comment_count>6</comment_count>
    <who name="Javier Carrasco">javier.carrasco.cruz</who>
    <bug_when>2024-03-16 12:34:57 +0000</bug_when>
    <thetext>It has been addressed, and a fix has been suggested:

https://github.com/crops/poky-container/pull/104</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99067</commentid>
    <comment_count>7</comment_count>
    <who name="Randy MacLeod">randy.macleod</who>
    <bug_when>2024-05-23 15:21:10 +0000</bug_when>
    <thetext>Does not seem to be merged yet.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99070</commentid>
    <comment_count>8</comment_count>
    <who name="Javier Carrasco">javier.carrasco.cruz</who>
    <bug_when>2024-05-23 15:27:26 +0000</bug_when>
    <thetext>(In reply to Randy MacLeod from comment #7)
&gt; Does not seem to be merged yet.

No, it has not been reviewed yet either. Apparently, the CROPS project has a very limited amount of reviewers. The PR will stay open, though.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99084</commentid>
    <comment_count>9</comment_count>
    <who name="Tim Orling">tim.orling</who>
    <bug_when>2024-05-24 16:36:42 +0000</bug_when>
    <thetext>This bug is not exactly correct. The problem is if the workspace has not been created before launching the container (and properly passed in as a volume). This is documented in the README and the wiki (for Windows and MacOS).

See example of fully functional cloning of metadata in:
https://github.com/crops/poky-container/pull/104#discussion_r1613742020

While this could perhaps be better clarified in the README, the statement that it MUST be done outside of the container is false. It is only conditionally true when the user did not first create the workspace and pass in the volume, per the instructions.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99085</commentid>
    <comment_count>10</comment_count>
    <who name="Tim Orling">tim.orling</who>
    <bug_when>2024-05-24 16:38:30 +0000</bug_when>
    <thetext>Perhaps the real issue here is &apos;repo&apos; tool. The way to fix that is to create a derivative container with &apos;repo&apos; added to it. This was rejected for the mainstream container because not everyone needs &apos;repo&apos; tool.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99086</commentid>
    <comment_count>11</comment_count>
    <who name="Tim Orling">tim.orling</who>
    <bug_when>2024-05-24 16:40:17 +0000</bug_when>
    <thetext>See https://github.com/crops/yocto-dockerfiles/pull/42 for discussion about git-lfs and repo tool and why they are not going to be added to the maintstream poky-container.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99087</commentid>
    <comment_count>12</comment_count>
    <who name="Tim Orling">tim.orling</who>
    <bug_when>2024-05-24 16:58:30 +0000</bug_when>
    <thetext>Dockerfile.repo:
FROM crops/poky:ubuntu-18.04

USER root

RUN  wget https://storage.googleapis.com/git-repo-downloads/repo -O /usr/local/bin/repo &amp;&amp; \
     chmod +x /usr/local/bin/repo

# We always need to return to the usersetup and ENTRYPOINT
USER usersetup
ENV LANG=en_US.UTF-8

ENTRYPOINT [&quot;/usr/bin/distro-entry.sh&quot;, &quot;/usr/bin/dumb-init&quot;, &quot;--&quot;, &quot;/usr/bin/poky-entry.py&quot;]

Build:
docker build -t crops/poky-repo:ubuntu-18.04 -f ./Docker.repo .


Execution:
ttorling@chiron:/srv/build/ttorling$ docker run --rm -it -v /srv/build/ttorling/crops-workspace:/workdir crops/poky-repo:ubuntu-18.04 --workdir=/workdir
pokyuser@aa90fb6f937a:/workdir$ which repo
/usr/local/bin/repo
pokyuser@aa90fb6f937a:/workdir$ repo --help
usage: repo COMMAND [ARGS]

repo is not yet installed.  Use &quot;repo init&quot; to install it here.

The most commonly used repo commands are:

  init      Install repo in the current working directory
  help      Display detailed help on a command

For access to the full online help, install repo (&quot;repo init&quot;).

Bug reports: https://issues.gerritcodereview.com/issues/new?component=1370071</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99088</commentid>
    <comment_count>13</comment_count>
    <who name="Tim Orling">tim.orling</who>
    <bug_when>2024-05-24 17:00:02 +0000</bug_when>
    <thetext>Because `repo` will always try to &quot;phone home&quot; and update itself, you will likely need to rebuild this container from time to time to have the latest version and avoid warnings.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99393</commentid>
    <comment_count>14</comment_count>
    <who name="Javier Carrasco">javier.carrasco.cruz</who>
    <bug_when>2024-07-18 05:41:14 +0000</bug_when>
    <thetext>Should this ticket be closed? Maybe updated to restrict it to a better clarification in the README as stated in comment 9?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99399</commentid>
    <comment_count>15</comment_count>
    <who name="Randy MacLeod">randy.macleod</who>
    <bug_when>2024-07-18 14:24:17 +0000</bug_when>
    <thetext>Javier,

Assigning to Tim and putting in NEEDINFO.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>99405</commentid>
    <comment_count>16</comment_count>
    <who name="Tim Orling">tim.orling</who>
    <bug_when>2024-07-18 14:51:33 +0000</bug_when>
    <thetext>I made significant comments on the pull request and do not believe this is valid.

https://github.com/crops/poky-container/pull/104</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>