[Bug 626455] New: libnotify misses a Requires: gtk+-2.0 in its pkg-config file
http://bugzilla.novell.com/show_bug.cgi?id=626455 http://bugzilla.novell.com/show_bug.cgi?id=626455#c0 Summary: libnotify misses a Requires: gtk+-2.0 in its pkg-config file Classification: openSUSE Product: openSUSE 11.3 Version: Final Platform: Other OS/Version: Other Status: NEW Severity: Normal Priority: P5 - None Component: GNOME AssignedTo: bnc-team-gnome@forge.provo.novell.com ReportedBy: cmorve69@yahoo.es QAContact: qa@suse.de Found By: Community User Blocker: --- Ekiga fails to build with a libtool: compile: g++ -DHAVE_CONFIG_H -I. -I../../../.. -I../../../../lib/engine/framework -I../../../../lib/engine/notification -I/usr/include/sigc++-2.0 -I/usr/lib64/sigc++-2.0/include -pthread -I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include -I/usr/include/dbus-1.0 -I/usr/lib64/dbus-1.0/include -fmessage-length=0 -O2 -Wall -D_FORTIFY_SOURCE=2 -fstack-protector -funwind-tables -fasynchronous-unwind-tables -g -fstack-protector -fexceptions -Wall -Wextra -Winit-self -Wswitch-default -Wswitch-enum -Wstrict-aliasing=2 -Wfloat-equal -Wshadow -MT libnotify-main.lo -MD -MP -MF .deps/libnotify-main.Tpo -c ./../../../lib/engine/components/libnotify/libnotify-main.cpp -fPIC -DPIC -o libs/libnotify-main.o In file included from /usr/include/libnotify/notify.h:28:0, from ./../../../lib/engine/components/libnotify/libnotify-main.cpp:41: /usr/include/libnotify/notification.h:28:21: fatal error: gtk/gtk.h: No such file or directory The thing is notification.h includes gtk/gtk.h because it declares functions that take as parameter pointers to GtkWidget and GtkStatusIcon. But libnotify doesn't links against libgtk-x11-2.0.so.0. Since the pkg-config doesn't specifies a gtk+-2.0 dependency the /usr/include/gtk-2.0 include path is never added to CFLAGS. So other software, as Ekiga, can fail to build. I didn't looked in detail. But since we are talking about pointers I suppose that header could be modified to remove that #include and manually declare the pointer types. Otherwise the libnotify .pc file should be modified... not sure if adding a Requires: gtk+-2.0 or if would be enough to add a "-I/usr/include/gtk-2.0" to its cflags field. -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
http://bugzilla.novell.com/show_bug.cgi?id=626455
http://bugzilla.novell.com/show_bug.cgi?id=626455#c1
Li Bin
http://bugzilla.novell.com/show_bug.cgi?id=626455
http://bugzilla.novell.com/show_bug.cgi?id=626455#c2
Cristian Morales Vega
http://bugzilla.novell.com/show_bug.cgi?id=626455
http://bugzilla.novell.com/show_bug.cgi?id=626455#c3
Li Bin
http://bugzilla.novell.com/show_bug.cgi?id=626455
http://bugzilla.novell.com/show_bug.cgi?id=626455#c4
Vincent Untz
http://bugzilla.novell.com/show_bug.cgi?id=626455
http://bugzilla.novell.com/show_bug.cgi?id=626455#c5
--- Comment #5 from Li Bin
http://bugzilla.novell.com/show_bug.cgi?id=626455
http://bugzilla.novell.com/show_bug.cgi?id=626455#c6
--- Comment #6 from Cristian Morales Vega
http://bugzilla.novell.com/show_bug.cgi?id=626455
http://bugzilla.novell.com/show_bug.cgi?id=626455#c7
--- Comment #7 from Vincent Untz
Are you sure this is ok????
It's the best solution, yes.
*libnotify* is adding a dependency against <includedir>/gtk/gtk.h (both GTK+2 and GTK+3 have it?) and that header must provide definitions for GtkWidget and GtkStatusIcon (both GTK+2 and GTK+3?).
Both GTK+ 2 and GTK+ 3 have it. It's up to the user of libnotify to decide which version of GTK+ to use.
Even if both versions of Gtk provide the header and the correct definitions, *libnotify* is the one adding that dependency. Is libnotify supposed to be used only by software that uses Gtk?
Yes.
Would not be better to modify that header, removing the "#include
" line and adding the types declarations?
This could be discussed upstream, but I think that'd be wrong in the end. The thing is that it's really up to the user of libnotify to decide which version of GTK+ to use when building, and libnotify doesn't want to force its users towards one specific version. -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
participants (1)
-
bugzilla_noreply@novell.com