ams-tschoening removed a comment on issue #20: Windows build static library support URL: https://github.com/apache/logging-log4cxx/pull/20#issuecomment-596384144 If I reviewed correctly, there are multiple issues with this PR: 1. it seems to reintroduce excluding tests, which I already removed in the past. Providing the missing binaries on Windows is easy and as many tests should be executed as possible in general. I won't merge excluding tests without any further discussion on why exactly you want those tests to be excluded. The binaries not being available by default in itself is not enough. 2. This PR seems to contain #19 as well, for which I asked for additional changes on its own already. So please reduce this PR to what is really necessary for static library support itself only. Thanks!
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
