GNOME Bugzilla – Bug 359065
reference documentation: stock icons not included
Last modified: 2008-02-19 02:28:00 UTC
docs/reference/Makefile sets IMAGE_PATH so that doxygen can find and used the stock icon .png files. I think it tries to use them from the installed location, but I think that location has changed. I also think that the directory structure in gtk+/gtk/stock-icons has changed, so this patch might be half of a solution.
Created attachment 73877 [details] [review] refdocs_stock.patch
I wonder if we shouldn't just use a svn:external property to fetch the gtk stock icons from GTK+ svn into our own tree and use them instead of trying to do some trickery to get them from a GTK+ checkout. It seems like that would be more robust. Also, as far as I can tell, the GTK+ stock icons are never actually installed anywhere, I think they might actually be compiled into GTK+ somehow...
OK, I've got a fix for this in my local tree. It involves fetching the GTK+ stock items from svn into the gtkmm tree using svn:externals as I described above. The one problem with this approach is that there are apparently some icons that are not unique but are symlinked to other names (e.g. media-next-ltr is symlinked to media-prev-rtl). When I just fetch the stock items via svn:externals, we don't get the GTK+ build rules which create the symlinked versions. So there are a few stock items that are missing. Should I come up with a better approch or just commit it this way?
btw, part of the issue with attachment #73877 [details] is that doxygen (at least the version that I have) can't seem to handle images that are in subdirectories of IMAGE_PATH. So for example, if you set IMAGE_PATH to '../gtk+/gtk/stock-icons/', the following command won't work @image html 48/foo-icon.png instead you have to set IMAGE_PATH to the directory that directly contains the icon. So you'd need to set it to something like IMAGE_PATH=../gtk+/.../48/ ../gtk+/.../24/ ../gtk+/.../16/
OK, I committed this even though it's not perfect yet. It should be easy to revert, so let me know if you have any concerns about it.