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 636272 - [PATCH] Doesn't compile against the latest GTK3
[PATCH] Doesn't compile against the latest GTK3
Status: RESOLVED FIXED
Product: gnome-nettool
Classification: Applications
Component: general
git master
Other Linux
: Normal normal
: ---
Assigned To: Germán Poo-Caamaño
Rodrigo Moya
Depends on:
Blocks:
 
 
Reported: 2010-12-02 05:06 UTC by Robert Ancell
Modified: 2011-01-11 07:35 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Updates to the latest GTK3 (15.48 KB, patch)
2010-12-02 05:06 UTC, Robert Ancell
committed Details | Review

Description Robert Ancell 2010-12-02 05:06:18 UTC
Created attachment 175695 [details] [review]
Updates to the latest GTK3

Doesn't compile against the latest GTK3, the attached patch fixes this.  Note the patch doesn't support GTK2 anymore, my recommendation is to only support GTK2 in the 2.32 series.
Comment 1 Germán Poo-Caamaño 2010-12-02 05:23:09 UTC
Review of attachment 175695 [details] [review]:

As far as I see, there are two issues:
1. Use of GtkComboBox instead of GtkComboBoxEntry.
2. Use of a Cairo context instead of GdkWindow.

Both of them should compile against gtk2, or I missing something?
Comment 2 Robert Ancell 2010-12-02 06:32:23 UTC
3. Replacement of expose-event with draw
(that's the annoying one).
Comment 3 Rodrigo Moya 2010-12-02 09:41:29 UTC
Germán, GtkComboBox has new API that is not available in 2.x, so it won't compile on both. So either we #ifdef that part of the code, or we do as Robert suggests, which is to force use of GTK3. I'm ok with doing that for the 3.x version, so unless Germán has something against it, please push your patch
Comment 4 Germán Poo-Caamaño 2010-12-02 15:24:12 UTC
(In reply to comment #3)
> Germán, GtkComboBox has new API that is not available in 2.x, so it won't
> compile on both. So either we #ifdef that part of the code, or we do as Robert
> suggests, which is to force use of GTK3. I'm ok with doing that for the 3.x
> version, so unless Germán has something against it, please push your patch

Rodrigo,

According to the documentation, it is available since 2.24:
http://library.gnome.org/devel/gtk/unstable/GtkComboBox.html#gtk-combo-box-set-entry-text-column
Comment 5 Germán Poo-Caamaño 2011-01-11 07:35:02 UTC
Review of attachment 175695 [details] [review]:

Ok, I just committed.
Comment 6 Germán Poo-Caamaño 2011-01-11 07:35:17 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.