I did try to add the APR folder for 'encoding' to the apr-build.xml, however it was not able to build it because apr_escape.c requires a (generated) header file that the log4cxx build process does not account for.
Ultimately, it doesn't seem like a good idea to essentially duplicate the APR build process in order for log4cxx to be built. The ant build script must be kept up to date with whatever APR is doing, and there's already a build procedure for APR. I have built log4cxx on Windows before, but that was a while ago(it was also not through Maven, which is what I tried this time). I spent a bit of time making up a CMakeLists.txt for log4cxx. It's based off of the CMake for APR. It doesn't do everything that the APR one does, but it does compile and link cleanly(I have not yet created a project to use the dll): https://gist.github.com/rm5248/4c8ad8ca01bd3b5761a7ff942113e638 APR uses CMake for Windows and autotools for Unix-like systems, so adding in a CMake build for Windows would be consistent with what APR does. -Robert Middleton On Sun, Jul 17, 2016 at 4:57 AM, Thorsten Schöning <tschoen...@am-soft.de> wrote: > Guten Tag Robert Middleton, > am Samstag, 16. Juli 2016 um 21:47 schrieben Sie: > > > The functions that are missing are apr_generate_random_bytes and apr_ > escape_hex. > > apr-build.xml: > > > <!-- Suppressing misc/win32/rand.c since it needs to > include rpc.h --> > > <fileset dir="${src.dir}" > > includes="**/${arch}/*.c" > > excludes="**/apr_app.c > misc/win32/rand.c" > > /> > > This was introduced in the following ticket: > > https://issues.apache.org/jira/browse/LOGCXX-74 > > I wonder how this could have ever worked in other environments, if it > doesn't for you. And I'm pretty sure that it did for me in the past > and some others were reporting it working on the list as well. Maybe > some changes in the versions of Visual Studio used? I last tried with > some version from 2013 or so. > > As for apr_escape_hex, it looks to me that nothing of the folder > "encoding" is compiled currently as well. Maybe just adding a fileset > like for "tables" and "strings" with the needed files makes it work. > > Mit freundlichen Grüßen, > > Thorsten Schöning > > -- > Thorsten Schöning E-Mail: thorsten.schoen...@am-soft.de > AM-SoFT IT-Systeme http://www.AM-SoFT.de/ > > Telefon...........05151- 9468- 55 > Fax...............05151- 9468- 88 > Mobil..............0178-8 9468- 04 > > AM-SoFT GmbH IT-Systeme, Brandenburger Str. 7c, 31789 Hameln > AG Hannover HRB 207 694 - Geschäftsführer: Andreas Muchow > >
