vineyard-preferences refuses to remove bottles

Bug #742554 reported by krank23
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Vineyard
Fix Released
Medium
Christian Dannie Storgaard

Bug Description

This is what I get; Ubuntu 10.10 and the latest source. Same if I use the testing ppa version. No idea as to the cause.

The effect is minor; the program doesn't crash or anything. But I need to go into the ~/.local/share/wineprefixes to remove a bottle, which is a bit of a hassle.

Traceback (most recent call last):
  File "./vineyard-preferences", line 526, in _on_dialog_remove_conf_response
    wine.prefixes.remove() # if no arg, remove current
  File "/home/krank/sources/vineyard/python-wine/wine/prefixes.py", line 408, in remove
    prefix_data = get_metadata_from_name_or_prefix(prefix_name_or_path)
NameError: global name 'prefix_name_or_path' is not defined

Revision history for this message
Christian Dannie Storgaard (cybolic) wrote :

Ah, thanks for catching that :)

Changed in vineyard:
status: New → Fix Committed
Revision history for this message
krank23 (krank23) wrote :

Just wanted to say that I tried getting the latest code, and the error message changed. This is what I get now:

Error occured
There was an error running the threaded function: coercing to Unicode: need string or buffer, NoneType found
/home/krank/sources/vineyard/vineyard/async.py:64
/home/krank/sources/vineyard/vineyard/widgets/widget.py:67
/home/krank/sources/vineyard/vineyard/widgets/widget.py:188
/home/krank/sources/vineyard/python-wine/wine/version.py:102
/home/krank/sources/vineyard/python-wine/wine/registry.py:264
/home/krank/sources/vineyard/python-wine/wine/registry.py:138
/home/krank/sources/vineyard/python-wine/wine/registry.py:26
<type 'exceptions.TypeError'>: coercing to Unicode: need string or buffer, NoneType found
None

Revision history for this message
Christian Dannie Storgaard (cybolic) wrote :

Thanks for the update.

Could you update the code and see if works now?

Revision history for this message
krank23 (krank23) wrote :

It works now! Good work! =)

Revision history for this message
Christian Dannie Storgaard (cybolic) wrote :

Good to hear and thanks! :)

Changed in vineyard:
importance: Undecided → Medium
assignee: nobody → Christian Dannie Storgaard (cybolic)
Changed in vineyard:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.