can't configure dvdrip-queue

[Danny Sauer]

Binary package hint: dvdrip-queue

The dvdrip-queue program (0.1.3-0.1ubuntu2 on maverick, and probably all the other releases) uses an invalid name in the ConfigParser module. When you go to the app's preferences and attempt to change the directory, the below happens:

sauer@trogdor:~$ dvdrip-queue
/usr/bin/dvdrip-queue.real:53: GtkWarning: Ignoring the separator setting
  self.wTree = gtk.glade.XML(self.gladefile)

(dvdrip-queue.real:21217): libglade-WARNING **: could not find a parent that handles internal children for `vbox'
loading *.rip files... done
/usr/share/dvdrip-queue/dvdripQueue/prefsWindow.py:32: GtkWarning: GtkSpinButton: setting an adjustment with non-zero page size is deprecated
  self.wTree = gtk.glade.XML(self.gladefile)
Traceback (most recent call last):
  File "/usr/share/dvdrip-queue/dvdripQueue/prefsWindow.py", line 113, in applyPrefs
    config.add_section("DEFAULT")
  File "/usr/lib/python2.6/ConfigParser.py", line 242, in add_section
    raise ValueError, 'Invalid section name: %s' % section
ValueError: Invalid section name: DEFAULT

Note that it's trying to use a section in teh config file named DEFAULT. That's not an allowed section name. If I open up /var/lib/mythtv/videos/tmp//swc2/avi/006/audio-video-psu/swc2-006-av-psu-00.av and replace all instances of DEFAULT with some other string - I used "jimmie" - it suddenly works fine. This is not surprising, since all case variantrs of default are ignored by the functions being called: http://docs.python.org/library/configparser.html#ConfigParser.RawConfigParser.add_section This code could never possibly work; I'm not sure how it was released since it clearly has never been tested.

Therefore, my suggestion is to change the string DEFAULT to pretty much any word other than default.

I would have reported this on the project Sourceforge page, but there doesn't appear to be a bug reporting area defined for the project. :/

[Danny Sauer (ubuntu-dannysauer)]

Ok, there's more to it than that, since the rest of the code assumes it's reading defaults. Creating the .ini file by hand to look like

sauer@trogdor:~$ cat .dvdrip-queue
[DEFAULT]
maxencodetime = 360.0
datadirectory = /mnt/dvdrip/tmp

works. But that's aggrevating and poorly documented. :)