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 666241 - Master branch breaks at build time
Master branch breaks at build time
Status: RESOLVED FIXED
Product: nautilus-sendto
Classification: Applications
Component: general
3.0.x
Other Linux
: Normal normal
: ---
Assigned To: Patricia Santana Cruz
nautilus-sendto-maint
Depends on:
Blocks:
 
 
Reported: 2011-12-15 11:20 UTC by Patricia Santana Cruz
Modified: 2012-01-10 11:10 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Replace deprecated symbols (9.88 KB, patch)
2011-12-15 11:22 UTC, Patricia Santana Cruz
needs-work Details | Review
Replace deprecated symbols (13.87 KB, patch)
2011-12-19 12:49 UTC, Patricia Santana Cruz
none Details | Review
Replace deprecated symbols (14.41 KB, patch)
2011-12-19 15:28 UTC, Patricia Santana Cruz
reviewed Details | Review
Updated patch based on review in comment https://bugzilla.gnome.org/show_bug.cgi?id=666241#c7 (6.56 KB, patch)
2012-01-10 10:53 UTC, Patricia Santana Cruz
committed Details | Review

Description Patricia Santana Cruz 2011-12-15 11:20:14 UTC
There are some deprecated symbols that need to be replaced, GtkVBox for instance.
Comment 1 Patricia Santana Cruz 2011-12-15 11:22:18 UTC
Created attachment 203560 [details] [review]
Replace deprecated symbols

Fixes the bug.
Comment 2 David King 2011-12-16 17:44:46 UTC
Review of attachment 203560 [details] [review]:

You probably want to mention the bug number in the commit message, and I would mention boxes in the first line too.

I think that you missed some other VBox and HBox deprecations, and they are probably worth fixing at the same time:

src/plugins/evolution/evolution.c:      p->vbox = gtk_vbox_new (FALSE, 8);
src/plugins/nautilus-sendto-progress.c: vbox = gtk_vbox_new (FALSE, 6); 
src/plugins/nautilus-sendto-progress.c: hbox = gtk_hbox_new (FALSE, 4); 
src/plugins/removable-devices/removable-devices.c:      box = gtk_vbox_new (FALSE, 8);
src/plugins/socialweb/socialweb.c:      page->page = gtk_vbox_new (FALSE, 8);
data/nautilus-sendto.ui:              <object class="GtkHBox" id="hbox_contacts_widgets">
data/nautilus-sendto.ui:                      <object class="GtkHBox" id="hbox2">

::: src/plugins/nautilus-sendto-packer.h
@@ -32,2 +32,2 @@
 typedef struct {
-	GtkVBox parent;
+	GtkBox parent;

This will need a corresponding change in the .c file. You need to add a call to gtk_orientable_set_orientation() in the instance init function, so that the orientation of the box is set to vertical, because the default orientation of a GtkOrientable is horizontal:

http://developer.gnome.org/gtk3/stable/gtk3-Orientable.html
Comment 3 Patricia Santana Cruz 2011-12-19 12:47:23 UTC
Update:

Build process breaks for master due to some deprecated symbols. This just happens during maintainer mode:

In file included from evolution.c:29:0:
../../../src/plugins/nautilus-sendto-packer.h:33:2: error: unknown type name 'GtkVBox'
../../../src/plugins/nautilus-sendto-packer.h:38:2: error: unknown type name 'GtkVBoxClass'
evolution.c: In function 'evolution_plugin_get_widget':
evolution.c:269:2: warning: implicit declaration of function 'gtk_vbox_new' [-Wimplicit-function-declaration]
evolution.c:269:2: warning: nested extern declaration of 'gtk_vbox_new' [-Wnested-externs]
evolution.c:269:10: warning: assignment makes pointer from integer without a cast [enabled by default]
make[6]: *** [evolution.lo] Error 1
make[6]: Leaving directory `/home/patriciasc/Development/checkout/gnome3/nautilus-sendto/src/plugins/evolution'
make[5]: *** [all] Error 2
make[5]: Leaving directory `/home/patriciasc/Development/checkout/gnome3/nautilus-sendto/src/plugins/evolution'
make[4]: *** [all-recursive] Error 1
make[4]: Leaving directory `/home/patriciasc/Development/checkout/gnome3/nautilus-sendto/src/plugins'
make[3]: *** [all] Error 2
make[3]: Leaving directory `/home/patriciasc/Development/checkout/gnome3/nautilus-sendto/src/plugins'
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory `/home/patriciasc/Development/checkout/gnome3/nautilus-sendto/src'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/home/patriciasc/Development/checkout/gnome3/nautilus-sendto'
make: *** [all] Error 2
Comment 4 Patricia Santana Cruz 2011-12-19 12:49:04 UTC
Created attachment 203862 [details] [review]
Replace deprecated symbols

Fixes this bug.
Comment 5 Patricia Santana Cruz 2011-12-19 15:28:57 UTC
Created attachment 203883 [details] [review]
Replace deprecated symbols

Fixes this bug.
Comment 6 David King 2011-12-19 15:43:16 UTC
Review of attachment 203883 [details] [review]:

Covers all my comments, I think. Looks good.
Comment 7 Bastien Nocera 2012-01-06 10:27:12 UTC
Review of attachment 203883 [details] [review]:

Looks fine apart from the comment.

Please use the bug URL as a reference.

::: data/pack-entry.ui
@@ -18,3 @@
     </data>
   </object>
-                      <object class="GtkVBox" id="vbox4">

Why all those changes here? Seem to me that a simple s/Gtk[VH]Box/GtkBox/ would have been enough as the orientation is already present.
Comment 8 Patricia Santana Cruz 2012-01-10 10:53:56 UTC
Created attachment 204931 [details] [review]
Updated patch based on review in comment https://bugzilla.gnome.org/show_bug.cgi?id=666241#c7
Comment 9 Bastien Nocera 2012-01-10 11:10:05 UTC
commit a7b68931d35a7b0c3aa83e00935401709b6f26c0
Author: Patricia Santana Cruz <patriciasantanacruz@gmail.com>
Date:   Tue Jan 10 10:48:53 2012 +0000

    Replace deprecated symbols
    
    GtkVbox and GtkHbox are deprecated in GTK+3 and therefore, have been
    replaced in the code by GtkBox objects with vertical and horizontal
    orientation.
    
    Fixes https://bugzilla.gnome.org/show_bug.cgi?id=666241.