<?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>7574</bug_id>
          
          <creation_ts>2015-04-07 15:55:23 +0000</creation_ts>
          <short_desc>The project release change process deletes imported layers without warning</short_desc>
          <delta_ts>2016-06-20 14:17:47 +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>unspecified</version>
          <rep_platform>x86</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>OBSOLETE</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>Medium+</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>2.2</target_milestone>
          <dependson>8429</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Belen Barros Pena">belen.barros.pena</reporter>
          <assigned_to name="Michael Wood">michael.g.wood</assigned_to>
          <cc>alexandru.damian</cc>
    
    <cc>belen.barros.pena</cc>
    
    <cc>cristiana.voicu</cc>
    
    <cc>jessica.zhang</cc>
    
    <cc>jose.perez.carranza</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>50124</commentid>
    <comment_count>0</comment_count>
    <who name="Belen Barros Pena">belen.barros.pena</who>
    <bug_when>2015-04-07 15:55:23 +0000</bug_when>
    <thetext>Steps to reproduce:

1. With Toaster in build mode, create a project with release &apos;Yocto Project master&apos;

2. Add a layer that exists in master but does not exist for the fido branch yet (for example, meta-raspberrypi). 

3. Import a layer

4. Change the project release to &apos;Yocto Project 1.8 Fido&apos;. A modal dialog will come up warning you that if you change the release meta-oe and meta-raspberrypi will be deleted from your project because they don&apos;t exist for fido. 

5. Change the release

meta-oe and meta-raspberrypi will be deleted from the project layers, as the dialog warned you, but your imported layer will be deleted too (without prior warning in this case). 

Imported layers are not governed by the same rules as layer source layers, and they should never be deleted from the project layers when you change the release.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>52572</commentid>
    <comment_count>1</comment_count>
    <who name="Michael Wood">michael.g.wood</who>
    <bug_when>2015-07-23 21:48:02 +0000</bug_when>
    <thetext>Part of the issue is that Imported layers get a release set on them, which makes them seem like incompatible layers after the release has been changed.

Maybe they shouldn&apos;t get a release set at all.

Note to self:
views.py:2191
...
def project
...
if &apos;projectVersion&apos; in request.POST</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>52574</commentid>
    <comment_count>2</comment_count>
    <who name="Belen Barros Pena">belen.barros.pena</who>
    <bug_when>2015-07-24 09:31:59 +0000</bug_when>
    <thetext>(In reply to comment #1)
&gt; Part of the issue is that Imported layers get a release set on them, which
&gt; makes them seem like incompatible layers after the release has been changed.
&gt; 
&gt; Maybe they shouldn&apos;t get a release set at all.

This sounds good to me. The goal is keeping imported layers in the project, to avoid user confusion.

From the UI side of things, these layers would need to show a revision, like all other layers do, but they would inherit the revision matching the project release. 

&gt; Note to self:
&gt; views.py:2191
&gt; ...
&gt; def project
&gt; ...
&gt; if &apos;projectVersion&apos; in request.POST</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>58683</commentid>
    <comment_count>3</comment_count>
    <who name="Michael Wood">michael.g.wood</who>
    <bug_when>2016-02-05 19:20:19 +0000</bug_when>
    <thetext>Wider reworking of the import layer mechanism has been identified so this will be part of that work</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>59086</commentid>
    <comment_count>4</comment_count>
    <who name="Belen Barros Pena">belen.barros.pena</who>
    <bug_when>2016-02-16 10:35:49 +0000</bug_when>
    <thetext>We are removing the ability to change the project release (see bug 8917).</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>