Bug 566569 - UnboundLocalError: local variable 'partitions' referenced before assignment
Summary: UnboundLocalError: local variable 'partitions' referenced before assignment
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: anaconda
Version: rawhide
Hardware: x86_64
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Hans de Goede
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard: anaconda_trace_hash:7e29bae22bf9409fa...
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-02-18 20:12 UTC by Mathieu Bridon
Modified: 2010-02-26 02:59 UTC (History)
4 users (show)

Fixed In Version: anaconda-13.29-1
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2010-02-20 19:22:54 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
Attached traceback automatically from anaconda. (341.65 KB, text/plain)
2010-02-18 20:12 UTC, Mathieu Bridon
no flags Details

Description Mathieu Bridon 2010-02-18 20:12:47 UTC
The following was filed automatically by anaconda:
anaconda 13.26 exception report
Traceback (most recent call first):
  File "/usr/lib/anaconda/platform.py", line 245, in checkBootRequest
    for p in partitions:
  File "/usr/lib/anaconda/storage/__init__.py", line 1011, in sanityCheck
    errors.extend(self.anaconda.platform.checkBootRequest(boot))
  File "/usr/lib/anaconda/iw/partition_gui.py", line 844, in getNext
    (errors, warnings) = self.storage.sanityCheck()
  File "/usr/lib/anaconda/gui.py", line 1307, in nextClicked
    rc = self.currentWindow.getNext ()
UnboundLocalError: local variable 'partitions' referenced before assignment

Comment 1 Mathieu Bridon 2010-02-18 20:12:53 UTC
Created attachment 394981 [details]
Attached traceback automatically from anaconda.

Comment 2 Hans de Goede 2010-02-18 20:25:23 UTC
Thanks for the report, this is caused by a patch of mine

A patch fixing this has been send to the list, and will get added to anaconda
as soon as it has been reviewed.

Note that in order to trigger this you need to do something not supported
like trying to put /boot on a LV. We still should not trace back though.

Comment 3 Mathieu Bridon 2010-02-18 20:30:19 UTC
(In reply to comment #2)
> Note that in order to trigger this you need to do something not supported
> like trying to put /boot on a LV.

That's what I did. I didn't know if that was still not supported so I tried, and was pleasantly surprised to not see a message saying it's not possible.

> We still should not trace back though.

A « You can't do that » message would be nice, yes :)

Comment 4 Hans de Goede 2010-02-20 19:22:54 UTC
This is fixed by this commit:
http://git.fedorahosted.org/git/?p=anaconda.git;a=commitdiff;h=e46b1d5f4a31dbc819a8728a669c8c4e86eeb588
Which will be in the next anaconda build:
anaconda-13.29-1

Comment 5 Fedora Update System 2010-02-23 03:42:55 UTC
anaconda-13.29-1.fc13 has been submitted as an update for Fedora 13.
http://admin.fedoraproject.org/updates/anaconda-13.29-1.fc13

Comment 6 Fedora Update System 2010-02-23 14:05:24 UTC
python-urlgrabber-3.9.1-5.fc13,anaconda-13.29-1.fc13 has been submitted as an update for Fedora 13.
http://admin.fedoraproject.org/updates/python-urlgrabber-3.9.1-5.fc13,anaconda-13.29-1.fc13

Comment 7 Fedora Update System 2010-02-26 02:59:37 UTC
python-urlgrabber-3.9.1-5.fc13 has been submitted as an update for Fedora 13.
http://admin.fedoraproject.org/updates/F13/FEDORA-2010-2688


Note You need to log in before you can comment on or make changes to this bug.