gWaei 1.2.1 segfaults on any action (lucid_x64)

[kamereon]

DESCRIPTION: Bug causes dictionaries to be uninstallable in gui preferences, trying to do so causes a crash, making the app unusable without resorting to the command line

PATCH: gtk_widget_get_name is deprecated in gtk and so is replaced with calls to gtk_buildable_get_name in attached patch (minimal taken from 1.2.2)

TEST CASE: * Start gwaei, and open preferences edit->preferences
        * In 'install dictionaries' tab, click 'add' next to any dictionary
        * program segfaults

REGRESSION: Currently segfaults with uses of gtk_widget_get_name, patch only changes these calls and no others so should be minimal regression potential

ORIGINAL REPORT:
Binary package hint: gwaei

When started, gwaei will present the gwaei-settings screen (since I'm using it for the first time and have not yet installed any dictionaries). The settings screen cannot be closed (no reaction when clicking 'close'). When selecting any dictionary for installation, gwaei will segfault.

ProblemType: Bug
DistroRelease: Ubuntu 10.04
Package: gwaei 1.2.1-1
ProcVersionSignature: Ubuntu 2.6.32-21.32-generic 2.6.32.11+drm33.2
Uname: Linux 2.6.32-21-generic x86_64
NonfreeKernelModules: nvidia
Architecture: amd64
Date: Wed Apr 28 19:54:00 2010
InstallationMedia: Ubuntu 10.04 "Lucid Lynx" - Release Candidate amd64 (20100419.1)
ProcEnviron:
 LANG=de_DE.utf8
 SHELL=/bin/bash
SourcePackage: gwaei

[kamereon (kamereon)]

If you have gJiten installed, you can get gWaei to work by copying gJitens dictionaries (usr/share/gjiten/dics/) to the .waei folder in your user-directory.
Closing the gwaei-settings window still not working, though.

[Niall Creech (sevenmachines-deactivatedaccount)]

Looks like theres a problem getting the name from the button, causing the dictionary search name to return null, this is fixed in version 1.3 with much tidying up of the way this is done.
Note, to work around this you can install/uninstall dictionaries via the command line.
For example,
$ waei -i
Installable: English Kanji Radicals Names Examples French German Spanish

$ waei -i French
Trying to install French...
* Downloading file...

[Zachary Dovel (pizzach)]

Hi, I'm the upstream programmer of gWaei.

This was fixed in gWaei 1.3 when I was informed of it on Fedora (which runs the same version of gtk). Getting and setting the gtk_buildable names using gtk_widget_set_name was depricated in gtk which is causing the problem.

I haven't been able to get in contact with the package container on this yet. gWaei 1.4 should be released this week. Both of those newer versions do not have this problem.

Zachary

[Zachary Dovel (pizzach)]

In the next week, I'll be releasing 1.2.2 for you guys so you have something that doesn't crash. I don't know how quickly it will make it into the repositories though.

Zachary

[Zachary Dovel (pizzach)]

I did a new release of gWaei today that has no crashes with Ubuntu 10.05. It is available here:

https://sourceforge.net/projects/gwaei/files/gwaei/1.2.2/gwaei-1.2.2.tar.gz/download

If someone could do some testing, it would be appreciated. I am going to poke my package maintainer.

[Zachary Dovel (pizzach)]

Uploaded a deb file to the server for people to try. It isn't official though, so please watch for when that does filter through eventually.

https://sourceforge.net/projects/gwaei/files/gwaei/1.2.2/gwaei-1.2.2-1.i386.deb/download

Feel free to to submit problems with the deb here. You can also visit me on IRC, but please stay longer than 10 minutes. I can't reply to you if you just disappear.

Zachary

[Niall Creech (sevenmachines-deactivatedaccount)]

Hi, sorry for the delay, attached is the most minimal patch to fix this (hopefully). if someone can look it over ( particularly zachary :) ) and i'll see what the good sru people think of it, thanks again zachary

gwaei (1.2.1-1ubuntu1) lucid; urgency=low

  * gtk-fix-names.diff:
   - fix dict names in settings crash dues to gtk gtk_widget_get_name to
     gtk_buildable_get_name change. Fixed in 1.2.2. Thanks to
     Zachary Dovel (LP: #571369)

[Zachary Dovel (pizzach)]

Hey, SevenMachines. That looks about right for a minimal patch. You have my blessing :-p