Bug 489208

Summary: traceback on yum.misc.unique()
Product: [Fedora] Fedora Reporter: Jeroen van Meeuwen <vanmeeuwen+fedora>
Component: yumAssignee: Seth Vidal <skvidal>
Status: CLOSED UPSTREAM QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: rawhideCC: ffesti, james.antill, pmatilai, tim.lauridsen
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2009-03-08 16:49:22 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
snippet of log with traceback on yum.misc.unique() none

Description Jeroen van Meeuwen 2009-03-08 16:32:21 UTC 
Created attachment 334442 [details]
snippet of log with traceback on yum.misc.unique()

Description of problem:

in branch yum-3_2_X, commit 12d9d60702a41a55bce19e01bb3151cfff0c31f0 [1] seems to have caused situations in which u is referenced before it is defined, causing a traceback[2] when using yum.misc.unique().

[1] http://yum.baseurl.org/gitweb?p=yum.git;a=commitdiff;h=12d9d60702a41a55bce19e01bb3151cfff0c31f0

[2] http://www.kanarip.com/revisor/20090308/rawhide-i386-respin/log/revisor-rawhide-i386-respin.log (near the end, snippet attached)

Version-Release number of selected component (if applicable):

yum.noarch 0:3.2.21-12.fc11

How reproducible:

Run revisor in respin mode so that it calls yum.misc.unique()

Steps to Reproduce:
1. Create a transaction with a few packages (~2400)
2. Get all the requirements for these packages, similar to:

    reqs = []
    for txmbr in yumobj.tsInfo.getMembers():
        reqs.append(txmbr.po.requires)
    reqs = yum.misc.unique(reqs)

3. A traceback is the result
Comment 1 James Antill 2009-03-08 16:49:22 UTC 
This should be fixed in commit: a0cd169628adcabc80575eb22447a9cf091e8251

.,..basically just s/del u/pass/
Comment 2 Jeroen van Meeuwen 2009-03-08 17:15:57 UTC 
You're the best, thanks!