driconf: DeprecationWarning with python 2.6

Description of problem:
DRIConf code is becoming more and more stale. Needs some TLC, and maybe some
help with parsing:
[...]
[vince@localhost Thu Jun 05 03:47:13 ~]$ driconf
/usr/lib/python2.5/site-packages/driconf_simpleui.py:461: GtkWarning: Failed to
set text from markup due to error parsing markup: Error on line 1 char 100:
Invalid UTF-8 encoded text - not valid '<b>Mesa DRI Mach64 [Rage Pro] 20051019
AGP 2x x86/MMX/SSE (Gareth Hughes, Leif Delgass,Jos� Fonseca)</b>'
  deviceLabel.set_markup("<b>" + commonui.escapeMarkup(text) + "</b>")
/usr/lib/python2.5/site-packages/driconf_simpleui.py:522: GtkWarning: Failed to
set text from markup due to error parsing markup: Error on line 1 char 100:
Invalid UTF-8 encoded text - not valid '<b>Mesa DRI Mach64 [Rage Pro] 20051019
AGP 2x x86/MMX/SSE (Gareth Hughes, Leif Delgass,Jos� Fonseca)</b>'
  self.add(self.vbox)
/usr/lib/python2.5/site-packages/driconf_complexui.py:843: DeprecationWarning:
  "priv", self.configTree.saveConfig, None, -1)
/usr/lib/python2.5/site-packages/driconf_complexui.py:846: DeprecationWarning:
  "priv", self.configTree.reloadConfig, None, -1)
/usr/lib/python2.5/site-packages/driconf_complexui.py:847: DeprecationWarning:
  self.toolbar.append_space()
/usr/lib/python2.5/site-packages/driconf_complexui.py:850: DeprecationWarning:
  "priv", self.configTree.newItem, None, -1)
/usr/lib/python2.5/site-packages/driconf_complexui.py:853: DeprecationWarning:
  "priv", self.configTree.removeItem, None, -1)
/usr/lib/python2.5/site-packages/driconf_complexui.py:856: DeprecationWarning:
  "priv", self.configTree.moveUp, None, -1)
/usr/lib/python2.5/site-packages/driconf_complexui.py:859: DeprecationWarning:
  "priv", self.configTree.moveDown, None, -1)
/usr/lib/python2.5/site-packages/driconf_complexui.py:862: DeprecationWarning:
  "priv", self.configTree.properties, None, -1)
/usr/lib/python2.5/site-packages/driconf_complexui.py:863: DeprecationWarning:
  self.toolbar.append_space()
/usr/lib/python2.5/site-packages/driconf_complexui.py:872: DeprecationWarning:
  self.aboutHandler, None, -1)
/usr/lib/python2.5/site-packages/driconf_complexui.py:873: DeprecationWarning:
  self.toolbar.append_space()
/usr/lib/python2.5/site-packages/driconf_complexui.py:876: DeprecationWarning:
  self.exitHandler, None, -1)
[Vince@localhost Thu Jun 05 04:02:05 ~]$
[...]

Version-Release number of selected component (if applicable):
driconf-0.9.1-7.fc9.noarch

How reproducible:
Always.

Steps to Reproduce:
1. Install Fedora 9
2. Install driconf
3. Open driconf from terminal
4. Observe meltdown

Actual results:
Numerous python/parsing warnings as noted above.

Expected results:
Absolutely error-free execution; Direct Rendering a bonus.

Additional info:
An otherwise outstanding little utility, no other options/substitutes
available, please don't let this one die.

This looks like a UTF8 bug with driconf due to that video driver authors name. ;(

I will see if I can work up a fix or find someone who can do so...

Created attachment 308461
patch to transform glxinfo information to unicode

Are you able to test the above patch? I don't have the problem so I worked a
bit blind on this. This is not a great solution as there's likely to be other
places in the code with the same sorts of issues. And it would be nice if this
was solved further up the stack rather than in driconf code. But if it works it
might be a workaround and upstream can look at it to see what they're going to
have to do everywhere.

Thanx for the quick response! And the patch....

Patch applied successfully: works perfectly for simple UI, no CLI output
whatsoever as long as in "Simple UI" mode, but still get same "error" messages
as above when switching GUI to "Expert Mode" (everything starting with: "/usr/
lib/python2.5/site-packages/driconf_complexui.py:" and down). Patch for
driconf_complexui.py? :D

I assume this simpleui.py patch will work for F8 as well? - will test shortly.

BTW, these "error" messages do not appear to affect driconf functionality, but
since I don't know too much about the inner workings of this stuff, I didn't
know what they really meant....

Many Thanx and Regards,
VJS

OOPS! Sorry about that - all I did was hit "Refresh"....

Yes, the patch works on F8 too, exact same results.

BTW, glxinfo has the same "stutter" when trying to parse/read that letter: é

Thanx Again, sorry about the "spam",
VJS

Created attachment 308501
Patch to update deprecated toolbar methods

Okay, here's a patch to update the deprecated toolbar methods with the newer
methods from pygtk. I'm not sure if upstream will want this verbatim or if
they'll want to conditionalize to work with older pygtk versions but this is
what they'll want to implement for the newer versions.

Re: glxinfo... you can file a bug against glx-utils if you'd like. I'm
primarily a python programmer these days and glxinfo is written in C so it's not
something I can knock off in an hour.

Yes, this second patch worked perfectly. All fixed. :D

Thank You Mr. Kuratomi for your time and help on this - by far the fastest
response I've ever gotten. I'll feel much more confident recommending driconf to
other users who want/need to change their xorg.conf DRI settings.

Many Thanx and Best Regards,
VJS

driconf-0.9.1-8.fc9 has been submitted as an update for Fedora 9

driconf-0.9.1-7.fc8 has been submitted as an update for Fedora 8

driconf-0.9.1-7.fc7 has been submitted as an update for Fedora 7

Thanks for all the great patches Toshio!

I have pushed out the patched package to updates-testing...
I'll also notify upstream.

Thanks for the bug report!

driconf-0.9.1-7.fc7 has been pushed to the Fedora 7 testing repository. If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with
 su -c 'yum --enablerepo=updates-testing update driconf'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F7/FEDORA-2008-5100

(In reply to comment #11)
> driconf-0.9.1-7.fc8 has been submitted as an update for Fedora 8

Installed from updates-testing repo. Works for me.

Thanx and Regards,
VJS

Excellent. I am going to go ahead and push these to stable as F7 is close to the
end of it's life.

Thanks again for the bug report!

(In reply to comment #10)
> driconf-0.9.1-8.fc9 has been submitted as an update for Fedora 9

Installed from updates-testing repo. Works for me.

Thanx and Regards,
VJS

driconf-0.9.1-7.fc8 has been pushed to the Fedora 8 stable repository. If problems still persist, please make note of it in this bug report.

driconf-0.9.1-8.fc9 has been pushed to the Fedora 9 stable repository. If problems still persist, please make note of it in this bug report.

driconf-0.9.1-7.fc7 has been pushed to the Fedora 7 stable repository. If problems still persist, please make note of it in this bug report.

I'm going to go ahead and close this now.

Feel free to re-open or file a new bug if you spot further issues.