<?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>1420</bug_id>
          
          <creation_ts>2011-08-25 23:08:39 +0000</creation_ts>
          <short_desc>[HOB] duplicated brought in package list</short_desc>
          <delta_ts>2013-10-03 08:18:41 +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>unspecified</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></status_whiteboard>
          <keywords></keywords>
          <priority>Undecided</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>1.1</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Jiajun Xu">jiajun.xu</reporter>
          <assigned_to name="Joshua Lock - Disabled">josh</assigned_to>
          <cc>andreea.b.proca</cc>
    
    <cc>jessica.zhang</cc>
    
    <cc>jiajun.xu</cc>
    
    <cc>lianhao.lu</cc>
    
    <cc>liping.ke</cc>
    
    <cc>poky.bs.watcher</cc>
    
    <cc>poky.watcher</cc>
          
          
          <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>15788</commentid>
    <comment_count>0</comment_count>
    <who name="Jiajun Xu">jiajun.xu</who>
    <bug_when>2011-08-25 23:08:39 +0000</bug_when>
    <thetext>Tree/Branch: master
Commit: 6c2b7beac3cd23ed44bd3e195c6360a0932876bf

With latest master, when we click some package, like avahi, some packages will have duplicated &quot;brought in by&quot; packages. For example, e2fsprogs-badblocks is brought in by e2fsprogs and there are a lot of e2fsprogs in &quot;brought in by&quot; column.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>15823</commentid>
    <comment_count>1</comment_count>
    <who name="Joshua Lock - Disabled">josh</who>
    <bug_when>2011-08-26 11:25:44 +0000</bug_when>
    <thetext>I believe this is describing the intended behaviour caused by two recent changes.
Firstly, we switched to storing all packages provided by a recipe, rather than just the recipe itself, in the data model and operating at a package level of granularity.
Further, and per bug 1365, we switched to listing all of the reverse dependencies of a package in the &apos;Brought in by&apos; column, rather than just the first package which brought the package in.

I believe this issue is appropriate to change to RESOLVED-&gt;NOTABUG, do you agree?
We do have bug 1423 querying the legitimacy of the entries in the &apos;Brought in by&apos; column</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>15835</commentid>
    <comment_count>2</comment_count>
    <who name="Jiajun Xu">jiajun.xu</who>
    <bug_when>2011-08-28 19:20:42 +0000</bug_when>
    <thetext>(In reply to comment #1)
&gt; I believe this is describing the intended behaviour caused by two recent
&gt; changes.
&gt; Firstly, we switched to storing all packages provided by a recipe, rather than
&gt; just the recipe itself, in the data model and operating at a package level of
&gt; granularity.
&gt; Further, and per bug 1365, we switched to listing all of the reverse
&gt; dependencies of a package in the &apos;Brought in by&apos; column, rather than just the
&gt; first package which brought the package in.
&gt; I believe this issue is appropriate to change to RESOLVED-&gt;NOTABUG, do you
&gt; agree?
&gt; We do have bug 1423 querying the legitimacy of the entries in the &apos;Brought in
&gt; by&apos; column

Hi Josh,
The issue I meet is that if I select/deselect avahi for N times, N duplicated package name will be shown up in &quot;Brought in By&quot;. You could have a try with avahi and check &quot;Brought in By&quot; column for package e2fsprogs-badblocks.

I find you have a patch to fix duplicated item to binb column as below. Will the patch fix the issue?

Signed-off-by: Joshua Lock &lt;josh@linux.intel.com&gt;
---
 lib/bb/ui/crumbs/tasklistmodel.py |    5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/lib/bb/ui/crumbs/tasklistmodel.py b/lib/bb/ui/crumbs/tasklistmodel.py
index 5ff1f5a..cf9fc59 100644
--- a/lib/bb/ui/crumbs/tasklistmodel.py
+++ b/lib/bb/ui/crumbs/tasklistmodel.py
@@ -441,8 +441,9 @@ class TaskListModel(gtk.ListStore):
             self[item_path][self.COL_INC] = True
 
         bin = self[item_path][self.COL_BINB].split(&apos;, &apos;)
-        bin.append(binb)
-        self[item_path][self.COL_BINB] = &apos;, &apos;.join(bin).lstrip(&apos;, &apos;)
+        if not binb in bin:
+            bin.append(binb)
+            self[item_path][self.COL_BINB] = &apos;, &apos;.join(bin).lstrip(&apos;, &apos;)
 
         # We want to do some magic with things which are brought in by the
         # base image so tag them as so
-- 
1.7.6</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>15852</commentid>
    <comment_count>3</comment_count>
    <who name="Joshua Lock - Disabled">josh</who>
    <bug_when>2011-08-29 16:23:59 +0000</bug_when>
    <thetext>Ah, thanks for clarifying! The patch you mention helps but there&apos;s a second loop in the same method that potentially modifies COL_BINB. I have a patch queued for submission to ensure duplicates aren&apos;t added there too.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>15885</commentid>
    <comment_count>4</comment_count>
    <who name="Joshua Lock - Disabled">josh</who>
    <bug_when>2011-08-30 15:09:05 +0000</bug_when>
    <thetext>http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=e2a3f283304e46b05e4900607d0568aec9b41f22</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>37504</commentid>
    <comment_count>5</comment_count>
    <who name="Andreea Brandusa Proca">andreea.b.proca</who>
    <bug_when>2013-10-03 08:15:43 +0000</bug_when>
    <thetext>Verified in master:4d8ab87d25ea6e59dc88375813ef88ee0ac813d0.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>37507</commentid>
    <comment_count>6</comment_count>
    <who name="Andreea Brandusa Proca">andreea.b.proca</who>
    <bug_when>2013-10-03 08:18:41 +0000</bug_when>
    <thetext>Verified.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>