GNOME Bugzilla – Bug 142540
build breaks due to duplicate gtkcellrendererprogress.h
Last modified: 2011-11-11 10:03:55 UTC
compiling procman from CVS HEAD 14/05/04 - gtk+ introduced gtkcellrendererprogress.h on 12/05/04, and so there is a clash between this and procman's own gtkcellrendererprogress.h the following make error: In file included from interface.c:38: cellrenderer.h:29:1: warning: "GTK_TYPE_CELL_RENDERER_PROGRESS" redefined In file included from /opt/gnome-2.7/include/gtk-2.0/gtk/gtk.h:51, from /opt/gnome-2.7/include/libgnomeui-2.0/gnome.h:4, from procman.h:23, from interface.c:31: /opt/gnome-2.7/include/gtk-2.0/gtk/gtkcellrendererprogress.h:34:1: warning: this is the location of the previous definition In file included from interface.c:38: cellrenderer.h:32: redefinition of `GtkCellRendererProgress' `GtkCellRendererProgress' previously declared here cellrenderer.h:33: redefinition of `GtkCellRendererProgressClass' /opt/gnome-2.7/include/gtk-2.0/gtk/gtkcellrendererprogress.h:42: `GtkCellRendererProgressClass' previously declared here cellrenderer.h:36: redefinition of `struct _GtkCellRendererProgress' cellrenderer.h:41: redefinition of `struct _GtkCellRendererProgressClass' make[2]: *** [interface.o] Error 1 make[2]: Leaving directory `/gnome/2.4/nightly/src-2.7-20040514/procman/src' make[1]: *** [all-recursive] Error 1 make[1]: Leaving directory `/gnome/2.4/nightly/src-2.7-20040514/procman' make: *** [all-recursive-am] Error 2 attaching patch to remove gtkcellrendererprogress.h from callbacks.c and interface.c and use the gtk+ header instead.
Created attachment 27695 [details] [review] patch to build with updated gtk+
Patch looks good, though we should also put a configure.in check to get the newer version of gtk+
Created attachment 27845 [details] [review] revised patch to check for updated gtk+
Actually come to think of it, I don't know if we can depend on gtk 2.6 for the next GNOME release. I think for now we should probably do a sed job on procman's local version of gtkcellrenderprogress to change to name to procmancellrendererprogress or something like that.
Created attachment 27846 [details] [review] said sed patch, requiring no new gtk+ dependency
gnome 2.10 will depend on gtk 2.6.
Fixed in HEAD.