On Thu, Sep 30, 2010 at 08:10:31PM +0200, Carlos R. Mafra wrote: > On Thu, 30 Sep 2010 at 10:52:41 +0200, Carlos R. Mafra wrote: > > Applied both patches. I will do some testing later. > > I'm getting these warnings: > > CC KeyboardShortcuts.o > In file included from ../src/wconfig.h:26:0, > from WPrefs.h:38, > from KeyboardShortcuts.c:25: > ../src/config.h:151:0: warning: "PIXMAPDIR" redefined > ../config-paths.h:8:0: note: this is the location of the previous definition > ../src/config.h:154:0: warning: "PKGDATADIR" redefined > ../config-paths.h:11:0: note: this is the location of the previous definition
Part of my patch moved config.h from src to the base directory, because otherwise it wasn't building the new config-paths.h header in time. But the file src/config.h is left behind from previous builds, and KeyboardShortcuts.c is picking up both versions (oddly, the correct one through including config.h directly and the incorrect one from including WPrefs.h which includes wconfig.h which includes config.h). The error goes away if you "rm src/config.h*". We could add that to configure.ac, if it seems like a big deal. -- To unsubscribe, send mail to [email protected].
