On Fri, Jan 6, 2012 at 2:45 AM, Daniel Shahaf <d...@daniel.shahaf.name> wrote: > Johan Corveleyn wrote on Fri, Jan 06, 2012 at 02:12:15 +0100: >> On Fri, Dec 2, 2011 at 7:04 PM, <hwri...@apache.org> wrote: >> > Author: hwright >> > Date: Fri Dec 2 18:04:14 2011 >> > New Revision: 1209610 >> > >> > URL: http://svn.apache.org/viewvc?rev=1209610&view=rev >> > Log: >> > Fix builds where SVN_DEBUG is not defined. >> > >> > * subversion/libsvn_subr/debug.c: >> > Don't build any of this file if SVN_DEBUG isn't defined. Callers don't >> > have >> > access to it in that case, anyway. >> >> For some reason, after this commit my *Release* build on Windows (VCE >> 2008) fails with: >> >> libsvn_subr.def : error LNK2001: unresolved external symbol svn_dbg__preamble >> libsvn_subr.def : error LNK2001: unresolved external symbol >> svn_dbg__print_props >> libsvn_subr.def : error LNK2001: unresolved external symbol svn_dbg__printf >> ..\..\..\Release\subversion\libsvn_subr\libsvn_subr-1.lib : fatal >> error LNK1120: 3 unresolved externals >> >> I've been trying to understand why, but I don't. Maybe something to do >> with preprocessors and linkers ... but my C compilation knowledge is >> lacking here :-). >> >> Is any of the other Windows devs seeing this? >> >> A Debug build still works, so if SVN_DEBUG is defined everything's ok. >> I guess that's also the reason why the buildbots don't see this >> problem (I guess they perform Debug-builds). >> > Does adding those three functions to the exceptions list in > build/generator/extractor.py fix the release build (and break the > debug build)?
Yes. Release build works. Debug build fails with: libsvn_ra_local-1.lib(ra_plugin.obj) : error LNK2019: unresolved external symbol _svn_dbg__printf referenced in function _ignore_warnings libsvn_ra_local-1.lib(ra_plugin.obj) : error LNK2019: unresolved external symbol _svn_dbg__preamble referenced in function _ignore_warnings ..\..\..\Debug\subversion\libsvn_ra\libsvn_ra-1.dll : fatal error LNK1120: 2 unresolved externals Any ideas on how to fix this? Has anyone been able to make a Release build on Windows since this commit? -- Johan