GNOME Bugzilla – Bug 776659
CellRenderer: Remove render() vfunc
Last modified: 2017-01-02 17:39:19 UTC
To match GTK+. Currently, cellrenderer.cc just throws a pile of errors. https://git.gnome.org/browse/gtk+/commit/?id=a36e5ceea74fecb7b8f5f196e592a4b633ed6eb3
Created attachment 342672 [details] [review] Regenerate GDK and GTK+ docs and defs With some minor detective work, I think I figured it how you do this: > cd tools/gen_scripts > export JHBUILD_SOURCES=~/jhbuild/checkout/gnome; for f in *.sh; do ./$f; done This fixes the problems with CellRenderer, and takes me to a new one: > GEN /home/daniel/jhbuild/checkout/gnome/gtkmm/gtk/gtkmm{entrycompletion.cc,entrycompletion.h,private/entrycompletion_p.h} > No conversion from const Glib::Error& to GError* defined (line: 82, parameter name: error) > m4 failed with exit code 1. Aborting... > Makefile:798: recipe for target '/home/daniel/jhbuild/checkout/gnome/gtkmm/gtk/src/.stamps/cssprovider.stamp' failed After swearing at make -j8 for making it hard to figure out where the problem was, I traced it to cssprovider.h
Review of attachment 342672 [details] [review]: What's this? Have you used a mixture of files from the master branch and the gtkmm-3-22 branch? Much of what the patch adds to gtk_signals.defs is info on classes that have been removed from the master branch, such as GtkAction and GtkUIManager. I have pushed a different patch that removes some render functions, because the corresponding gtk+ functions have been removed. https://git.gnome.org/browse/gtkmm/commit/?id=7535c82fea6f84fb598d3a9ca798b5287da8bcf5
git must not be removing obsolete files when I switch branches or something as I've seen a few oddities like this. Should've checked the result in more depth. Thanks.
When you switch branches, git deletes obsolete files that are stored in git's repository. But it does not care about generated files. If you expect to switch repeatedly back and forth between two branches, it can be better to keep your local copies of those two branches in different directories.