<?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>8429</bug_id>
          
          <creation_ts>2015-10-01 16:24:11 +0000</creation_ts>
          <short_desc>Re-work import layer behaviour</short_desc>
          <delta_ts>2017-01-04 02:46:32 +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>Toaster</product>
          <component>toaster</component>
          <version>2.1</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>GUI design pending</status_whiteboard>
          <keywords></keywords>
          <priority>Medium</priority>
          <bug_severity>enhancement</bug_severity>
          <target_milestone>Future</target_milestone>
          
          <blocked>7574</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Michael Wood">michael.g.wood</reporter>
          <assigned_to name="Michael Wood">michael.g.wood</assigned_to>
          <cc>belen.barros.pena</cc>
    
    <cc>david.reyna</cc>
    
    <cc>henry.bruce</cc>
    
    <cc>jessica.zhang</cc>
    
    <cc>stanciux.mihail</cc>
          
          <qa_contact name="Libertad">libertad.gonzalez.de.la.cruz</qa_contact>
          <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>54905</commentid>
    <comment_count>0</comment_count>
    <who name="Michael Wood">michael.g.wood</who>
    <bug_when>2015-10-01 16:24:11 +0000</bug_when>
    <thetext>The existing behaviour isn&apos;t quite working out, we need to come back to this feature and re-think the restrictions and scope of imported layers</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>55039</commentid>
    <comment_count>1</comment_count>
    <who name="Michael Wood">michael.g.wood</who>
    <bug_when>2015-10-07 16:00:12 +0000</bug_when>
    <thetext>To expand on this:

- We currently don&apos;t allow layers with the same name as imported layers, this isn&apos;t actually required, although we need to look into the implications of non-unique names properly (e.g. if you add two layers with the same name to a project does that work?)

- We require a git url - if you have a layer on your filesystem does this make sense, if we support non-git checkouts we should be able to provide a local path

- Each imported layer is project specific - does it need to be, should this be optional

- Should we create revisions of existing layers if the layer already exists

- Should the layer imported be assigned to a particular release [fido, master, dizzy..], this has an impact on the release change mechanism and the compatible layers listed in toaster.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>55143</commentid>
    <comment_count>2</comment_count>
    <who name="Belen Barros Pena">belen.barros.pena</who>
    <bug_when>2015-10-11 12:54:27 +0000</bug_when>
    <thetext>How we design this might also be impacted by bug 6640 and bug 6701 (just so that we don&apos;t forget keeping this in mind).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>59543</commentid>
    <comment_count>3</comment_count>
    <who name="Henry Bruce">henry.bruce</who>
    <bug_when>2016-02-29 19:11:17 +0000</bug_when>
    <thetext>It was not clear to me what &quot;revision&quot; means when enter layer info.
For recipes this is a number.
In this context it turned out to be git branch or tag.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>59571</commentid>
    <comment_count>4</comment_count>
    <who name="Belen Barros Pena">belen.barros.pena</who>
    <bug_when>2016-03-01 11:48:39 +0000</bug_when>
    <thetext>(In reply to comment #3)
&gt; It was not clear to me what &quot;revision&quot; means when enter layer info.
&gt; For recipes this is a number.
&gt; In this context it turned out to be git branch or tag.

Can I ask: how did you find out what revision meant? Would changing the label to &quot;Git revision&quot; help in any way? Do you have any suggestions for a better label?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>59598</commentid>
    <comment_count>5</comment_count>
    <who name="Henry Bruce">henry.bruce</who>
    <bug_when>2016-03-01 17:26:15 +0000</bug_when>
    <thetext>I clicked on the &quot;revision&quot; tool tip and it told me exactly what revision meant. The problem was I didn&apos;t notice the tool-tip mini-icon first time round. Renaming &quot;Revision&quot; to &quot;Git Revision&quot; would be a good solution.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>59662</commentid>
    <comment_count>6</comment_count>
    <who name="Belen Barros Pena">belen.barros.pena</who>
    <bug_when>2016-03-03 10:58:23 +0000</bug_when>
    <thetext>(In reply to comment #5)
&gt; Renaming &quot;Revision&quot; to &quot;Git Revision&quot; would be a good solution.

Patch sent 

https://lists.yoctoproject.org/pipermail/toaster/2016-March/004007.html</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>59666</commentid>
    <comment_count>7</comment_count>
    <who name="Belen Barros Pena">belen.barros.pena</who>
    <bug_when>2016-03-03 11:57:19 +0000</bug_when>
    <thetext>This should probably include running git ls-remote &lt;giturl&gt; when importing a layer, which would allow us to

1) check if a git url is reachable and warn users if it isn&apos;t
2) store available branches and show them to users for selection

And maybe even other tricks we haven&apos;t thought of yet. 

For some background on this see

https://bugzilla.yoctoproject.org/show_bug.cgi?id=9200#c1
https://bugzilla.yoctoproject.org/show_bug.cgi?id=9200#c2
https://bugzilla.yoctoproject.org/show_bug.cgi?id=9200#c3</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>69467</commentid>
    <comment_count>8</comment_count>
    <who name="David Reyna">david.reyna</who>
    <bug_when>2017-01-04 02:46:32 +0000</bug_when>
    <thetext>commit 97ff2c0091c654bf01bc4b0809a6a664ad6c579c
Author: Michael Wood &lt;michael.g.wood@intel.com&gt;
Date:   Fri Dec 9 16:52:49 2016</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>