j.nitsc...@ok.de has asked for review: Issue 126787: fix unit tests on windows https://bz.apache.org/ooo/show_bug.cgi?id=126787
Attachment 85248: remove prewin.h and postwin.h in sal unittests https://bz.apache.org/ooo/attachment.cgi?id=85248&action=edit --- Description --- build process on windows is currently broken[1] some unit tests in sal includes headers from tools which depends on sal (circular dependencies)[2] said headers are prewin.h and postwin.h which frame windows.h includes I got no win build environment, but looking at the files I don't think they are needed for the basic functionality sal (system abstraction layer) provides would be happy to know if someone knows more about prewin and postwin here a patch which removes the includes if someone with a win build environment could test it or if no time let our buildbot do the work. would be great [1] https://ci.apache.org/builders/aoo-win7/builds/139 [2] https://mail-archives.apache.org/mod_mbox/openoffice-dev/201512.mbox/%3C56804CD E.2020201%40ok.de%3E note: all this is related to the work in Issue 125003 - Integrate GoogleTest unit test framework for general usage in the build environment