After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 582511 - Use GtkBuilder instead of libglade
Use GtkBuilder instead of libglade
Status: RESOLVED DUPLICATE of bug 584303
Product: gdl
Classification: Other
Component: general
2.27.x
Other Linux
: Normal normal
: ---
Assigned To: Anjuta maintainers
Anjuta maintainers
Depends on:
Blocks: 572883
 
 
Reported: 2009-05-13 17:45 UTC by André Klapper
Modified: 2009-06-02 09:57 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Move ui file to gdl/ui and add [type: gettext/glade] in POTFILES.in (21.01 KB, patch)
2009-06-01 15:37 UTC, Javier Jardón (IRC: jjardon)
none Details | Review
Fix the location of gui file (674 bytes, patch)
2009-06-02 00:08 UTC, Javier Jardón (IRC: jjardon)
none Details | Review

Description André Klapper 2009-05-13 17:45:45 UTC
According to http://www.gnome.org/~fpeters/299.html
this module depends on libglade.

In GNOME 2.27, libglade has been deprecated in favor of GtkBuilder. See
http://library.gnome.org/devel/gtk/stable/gtk-migrating-GtkBuilder.html for
migration instructions. Also see
http://live.gnome.org/GnomeGoals/RemoveLibGladeUseGtkBuilder
Comment 1 Johannes Schmid 2009-05-27 09:41:07 UTC
This problem has been fixed in the development version. The fix will be available in the next major software release. Thank you for your bug report.
Comment 2 Javier Jardón (IRC: jjardon) 2009-06-01 15:37:53 UTC
Created attachment 135743 [details] [review]
Move ui file to gdl/ui and add [type: gettext/glade] in POTFILES.in
Comment 3 André Klapper 2009-06-01 15:45:22 UTC
NOte that part of the patch makes bug 584303 obsolete
Comment 4 Sébastien Granjoux 2009-06-01 19:30:33 UTC
(In reply to comment #2)
> Created an attachment (id=135743) [edit]
> Move ui file to gdl/ui and add [type: gettext/glade] in POTFILES.in

Thanks for you patch but it seems that one part is not working.

The fix in POTFILES.in is fine, it has already been reported in bug #584303.

But I don't understand why you put layout.ui in a new ui subdirectory. In fact, I get a build failure with this because the Makefile expect to get this file directly in the gdl directory.

It is the install step that copies this file typically from gdl/layout.ui to /usr/local/share/gdl/ui/layout.ui.

Comment 5 Javier Jardón (IRC: jjardon) 2009-06-02 00:08:11 UTC
Created attachment 135779 [details] [review]
Fix the location of gui file

(In reply to comment #4)
>
> But I don't understand why you put layout.ui in a new ui subdirectory. In fact,
> I get a build failure with this because the Makefile expect to get this file
> directly in the gdl directory.
> 

I check the gdl/Makefile.am file and uidir variable is defined as uidir = $(datadir)/gdl/ui. Should be uidir = $(datadir)/gdl then ;)


Regards
Comment 6 Johannes Schmid 2009-06-02 09:57:55 UTC
uidir=$(datadir)/gdl/ui is fine but that the directory where the library expects the file AFTER it was installed. Anything in POTFILES.in contains paths of the source directory (BEFORE the installation).

There is absolutely no need to move the file anywhere.

*** This bug has been marked as a duplicate of 584303 ***