<?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>2113</bug_id>
          
          <creation_ts>2012-03-14 23:21:14 +0000</creation_ts>
          <short_desc>[Hob2] Layer changes are not persisted between hob runs</short_desc>
          <delta_ts>2012-04-18 00:16:51 +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>Hob</product>
          <component>hob</component>
          <version>1.2</version>
          <rep_platform>x86</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>VERIFIED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard>(1.2) Patch out</status_whiteboard>
          <keywords></keywords>
          <priority>Medium</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>1.2 M4</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Joshua Lock - Disabled">josh</reporter>
          <assigned_to name="Shane Wang">shane.wang</assigned_to>
          <cc>belen.barros.pena</cc>
    
    <cc>david.c.stewart</cc>
    
    <cc>giulia</cc>
    
    <cc>jessica.zhang</cc>
    
    <cc>jiajun.xu</cc>
    
    <cc>poky.bs.watcher</cc>
    
    <cc>poky.watcher</cc>
          
          
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>---</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>19109</commentid>
    <comment_count>0</comment_count>
    <who name="Joshua Lock - Disabled">josh</who>
    <bug_when>2012-03-14 23:21:14 +0000</bug_when>
    <thetext>When using the Hob2 I can add and remove layers using the layer selection dialogue but if I exit Hob and relaunch the changes to the layer configuration are no longer present.

Steps to reproduce:

1. Start Hob
2. Open Layer dialogue
3. Add one or more layers
4. Close Layer dialogue
5. Exit Hob
6. Start Hob
7. Layers added in step 3 are no longer present in Layer selection dialogue</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>19175</commentid>
    <comment_count>1</comment_count>
    <who name="Shane Wang">shane.wang</who>
    <bug_when>2012-03-16 02:27:11 +0000</bug_when>
    <thetext>I don&apos;t think that was a bug.

What if the users want to start with a clean environment? They will complain this feature you made.

If the users really want to save the layers they selected, they could use &quot;Save as Templates&quot;.

What do you think, Josh?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>19200</commentid>
    <comment_count>2</comment_count>
    <who name="Joshua Lock - Disabled">josh</who>
    <bug_when>2012-03-16 19:40:24 +0000</bug_when>
    <thetext>(In reply to comment #1)
&gt; I don&apos;t think that was a bug.
&gt; 
&gt; What if the users want to start with a clean environment? They will complain
&gt; this feature you made.

I disagree. I think a user would complain about having to add the layers they want to use each time. Imagine a user who always has a layer for their companies customisations, they have to add that layer each time they use hob.

For a user who wants to start with a clean environment we could add a &quot;Defaults&quot; button, or similar.
 
&gt; If the users really want to save the layers they selected, they could use &quot;Save
&gt; as Templates&quot;.

I&apos;ve not played around with the Templates functionality much but I think this solution is sub-optimal.

Adding Belen and Giulia for their UX input.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>19222</commentid>
    <comment_count>3</comment_count>
    <who name="Belen Barros Pena">belen.barros.pena</who>
    <bug_when>2012-03-19 15:53:09 +0000</bug_when>
    <thetext>Hi Joshua and Shane,

You both have a point: it all depends on what the most frequent use case is. What we are missing in order to make the right decision is information about how Yocto (and by extension Hob) is used / will be used in the real world. 

Since we won&apos;t have that information in the short term, we need to make some assumptions. Would both of you agree that it is reasonable to assume that lots of Yocto users will be tweaking a certain image over and over again? This assumption is in line with what I&apos;ve been hearing in my conversations with people like Inaky and Damien, but you might have a different point of view about it.

If you agree with the assumption above, then it is important we keep the changes to the layers configuration between Hob sessions (i.e. changes made on the Layers dialog should not be lost when I close Hob).

I would not add a &apos;restore defaults&apos; option just yet: I would keep the Layers dialog as simple as possible for the moment. The design will allow users to delete a layer with a single click, which is quite efficient. If users request the option to restore the default layers configuration, then we can work on a way to bring the option in.

What do you think?

Belen</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>19228</commentid>
    <comment_count>4</comment_count>
    <who name="Joshua Lock - Disabled">josh</who>
    <bug_when>2012-03-19 20:23:26 +0000</bug_when>
    <thetext>(In reply to comment #3)
&gt; Since we won&apos;t have that information in the short term, we need to make some
&gt; assumptions. Would both of you agree that it is reasonable to assume that lots
&gt; of Yocto users will be tweaking a certain image over and over again? This
&gt; assumption is in line with what I&apos;ve been hearing in my conversations with
&gt; people like Inaky and Damien, but you might have a different point of view
&gt; about it.

This is also my assumption.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>19273</commentid>
    <comment_count>5</comment_count>
    <who name="Shane Wang">shane.wang</who>
    <bug_when>2012-03-21 03:34:45 +0000</bug_when>
    <thetext>Belen, I think your assumption is reasonable.

Belen and Josh, I agree.

By the way, should we remember machine setting between Hob sessions, some advanced settings as well (say file system type, bbthreads, etc). I also assume frequent users don&apos;t want to set them every time.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>19274</commentid>
    <comment_count>6</comment_count>
    <who name="Shane Wang">shane.wang</who>
    <bug_when>2012-03-21 03:36:08 +0000</bug_when>
    <thetext>Machine means an arm company/user will not build x86 image at all.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>19283</commentid>
    <comment_count>7</comment_count>
    <who name="Belen Barros Pena">belen.barros.pena</who>
    <bug_when>2012-03-21 12:24:38 +0000</bug_when>
    <thetext>As a general rule, I would keep values set via the Settings dialogue between Hob sessions. So if I set the number of threads in the Settings dialogue, then I close Hob, when I open Hob again the number of threads is the same as when I left it (I don&apos;t need to set it again).

All other stuff (e.g. machine, base image, etc) should not be kept between sessions.

Do you think that&apos;s reasonable?

Belen</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>19285</commentid>
    <comment_count>8</comment_count>
    <who name="Joshua Lock - Disabled">josh</who>
    <bug_when>2012-03-21 16:24:11 +0000</bug_when>
    <thetext>(In reply to comment #7)
&gt; As a general rule, I would keep values set via the Settings dialogue between
&gt; Hob sessions. So if I set the number of threads in the Settings dialogue, then
&gt; I close Hob, when I open Hob again the number of threads is the same as when I
&gt; left it (I don&apos;t need to set it again).
&gt; 
&gt; All other stuff (e.g. machine, base image, etc) should not be kept between
&gt; sessions.
&gt; 
&gt; Do you think that&apos;s reasonable?

I agree with this, it&apos;s what we aimed for in the first version of Hob.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>19314</commentid>
    <comment_count>9</comment_count>
    <who name="Shane Wang">shane.wang</who>
    <bug_when>2012-03-22 01:43:02 +0000</bug_when>
    <thetext>Clear. I am going to fix it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>20064</commentid>
    <comment_count>10</comment_count>
    <who name="Dave Stewart">david.c.stewart</who>
    <bug_when>2012-04-09 15:30:06 +0000</bug_when>
    <thetext>Guys - we talked about this, and it seems like the thread count should be persistent between hob invocations, but there are other things in Settings which we might want to reset when we spin it up again. 

Let&apos;s prefer the minimal change for 1.2 and we can revisit after, OK?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>20142</commentid>
    <comment_count>11</comment_count>
    <who name="Belen Barros Pena">belen.barros.pena</who>
    <bug_when>2012-04-10 18:04:56 +0000</bug_when>
    <thetext>Hi Dave,

Could you tell us a bit more about which Settings options you&apos;d rather reset?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>20358</commentid>
    <comment_count>12</comment_count>
    <who name="Shane Wang">shane.wang</who>
    <bug_when>2012-04-14 08:10:33 +0000</bug_when>
    <thetext>Nothing in the advanced setting will be reset.
In the current implementation, we are going to remember all layers and settings.

No worry.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>20359</commentid>
    <comment_count>13</comment_count>
    <who name="Shane Wang">shane.wang</who>
    <bug_when>2012-04-14 08:10:56 +0000</bug_when>
    <thetext>Under Testing...</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>20392</commentid>
    <comment_count>14</comment_count>
    <who name="Shane Wang">shane.wang</who>
    <bug_when>2012-04-14 23:40:14 +0000</bug_when>
    <thetext>http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=62c908c88b342370d2e179953c9e07094a6060b5</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>20597</commentid>
    <comment_count>15</comment_count>
    <who name="Joshua Lock - Disabled">josh</who>
    <bug_when>2012-04-18 00:16:51 +0000</bug_when>
    <thetext>I can set layers and package format and have them remembered the next time I launch hob. Verified!</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>