summaryrefslogtreecommitdiff
path: root/solenv/gbuild
AgeCommit message (Collapse)Author
2020-12-26My Windows build started to need an explicit -Zc:twoPhase- for CLI nowStephan Bergmann
...to acknowledge > [build CLR] cli_ure/source/climaker/climaker_emit.cxx > cl : Command line warning D9025 : overriding '/EHs' with '/EHa' > c1xx: error C2338: two-phase name lookup is not supported for C++/CLI or C++/CX; use /Zc:twoPhase- > c1xx: fatal error C1903: unable to recover from previous error(s); stopping compilation etc. Change-Id: I09cf54123345f5816f7c5b21cc9779567fd0cece Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108320 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2020-12-25Revert "Use { ... } for grouping that does not need a sub-shell"Stephan Bergmann
This reverts commit 51f4691e7f4f3bceac1d5ec9cbbf37de7e71e471. The gb_CppunitTest_localized case did need the sub-shell for its $(if $(gb_CppunitTest_localized),|| exit $$?; done) \ part, or else a failed sub-command would exit the whole recipe line and suppress the non-gb_CppunitTest__interactive postprocessing part. Change-Id: I115388ce4fe834f73ebd0abf2591775189c0121b Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108280 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2020-12-23Don't lock galleries build into DESKTOPJan-Marek Glogowski
And add the missing dependency of Executable_gengal on Package_svx_gengal, so the actual executable script is created in instdir_for_build for the cross-toolset. Change-Id: I98ea1d58273c871f0a3b804a93970eedfb7f8908 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108108 Tested-by: Jenkins Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
2020-12-17include everything from BUILDDIR, e.g. config_host/*Mike Kaganski
... otherwise any change in config that modifies those does not trigger related units to be rebuilt Change-Id: I680fc1e004208f7ccc752186d5e765560dffebf9 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/107875 Tested-by: Jenkins Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
2020-12-14Add -wd4505 also back to gb_CXXFLAGSStephan Bergmann
...which was accidentally missing from aca847cf1283e242c2d1106309ad4bb73d8f37a6 "-wd4505 is still needed" Change-Id: If0d3523f05d47da5dfe6dae45dedcdf075033daf Reviewed-on: https://gerrit.libreoffice.org/c/core/+/107723 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2020-12-14-wd4505 is still neededStephan Bergmann
...see e.g. <https://ci.libreoffice.org//job/lo_tb_master_win/29066> > C:/cygwin/home/tdf/lode/jenkins/workspace/lo_tb_master_win/unotools/source/config/bootstrap.cxx(269): error C2220: the following warning is treated as an error > C:/cygwin/home/tdf/lode/jenkins/workspace/lo_tb_master_win/unotools/source/config/bootstrap.cxx(269): warning C4505: 'utl::dbgCheckStatusOfURL': unreferenced local function has been removed where the OSL_ASSERT used in > OSL_ASSERT(aStatus != Bootstrap::PATH_VALID || dbgCheckStatusOfURL(sDerivedURL) == Bootstrap::PATH_VALID); is defined in terms of SAL_DETAIL_WARN_IF_FORMAT, which in turn is careful to make its contents visible to the compiler even under --disable-sal-log. That MSVC warning, about a function whose only reference is in code that the compiler can statically determine to be unreachable, is really unhelpful. This partially reverts 5c082fc1b59022252d90423afde174123976cdee "remove some MSVC -wd". Change-Id: Ie337989ee0ca6b7d7f29a50026fa249b0b266474 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/107716 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2020-12-14The workaround for MSVC C4702 is still neededStephan Bergmann
...see e.g. <https://ci.libreoffice.org//job/lo_tb_master_win/29063/> > C:\cygwin\home\tdf\lode\jenkins\workspace\lo_tb_master_win\svtools\source\misc\acceleratorexecute.cxx(213) : error C2220: the following warning is treated as an error > C:\cygwin\home\tdf\lode\jenkins\workspace\lo_tb_master_win\svtools\source\misc\acceleratorexecute.cxx(213) : warning C4702: unreachable code This partially reverts 5c082fc1b59022252d90423afde174123976cdee "remove some MSVC -wd". Change-Id: I24bc9e1b1d93cecc6d79346e6112411b1fa5989f Reviewed-on: https://gerrit.libreoffice.org/c/core/+/107676 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2020-12-14remove some MSVC -wdNoel Grandin
which appear to no longer be necessary on master with our updated MSVC requirements. Change-Id: I55a0267bfc5e2c36d9da7b9b7281b54e9f770f01 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/107470 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2020-12-09One more gbuild $ -> $$ fixStephan Bergmann
...similar to cc630e3d8ee5d07b7341b071443845ef309eeeb2 "One more gbuild $ -> $$ fix". (In a Linux --with-package-format=archive build against a GCC 11 trunk libstdc++ installed in a non-standard location, so requiring LD_LIBRARY_PATH set from outside the GNU Make invocation, `make Extension_nlpsolver` failed with > [XHP] nlpsolver com.sun.star.comp.Calc.NLPSolver/Options.xhp qtz > S=~/lo/core && I=$S/instdir && W=$S/workdir && mkdir -p $W/Extension/nlpsolver/help/qtz/com.sun.star.comp.Calc.NLPSolver/ && LD_LIBRARY_PATH="$I/program:$I/program" $W/LinkTarget/Executable/helpex -i $S/nlpsolver/help/en/com.sun.star.comp.Calc.NLPSolver/Options.xhp -o $W/Extension/nlpsolver/help/qtz/com.sun.star.comp.Calc.NLPSolver/Options.xhp -l qtz -m && touch $W/Extension/nlpsolver/root/help/qtz-xhp.done > ~/lo/core/workdir/LinkTarget/Executable/helpex: /lib64/libstdc++.so.6: version `GLIBCXX_3.4.29' not found (required by ~/lo/core/workdir/LinkTarget/Executable/helpex) > ~/lo/core/workdir/LinkTarget/Executable/helpex: /lib64/libstdc++.so.6: version `GLIBCXX_3.4.29' not found (required by ~/lo/core/instdir/program/libicuuc.so.68) > ~/lo/core/workdir/LinkTarget/Executable/helpex: /lib64/libstdc++.so.6: version `GLIBCXX_3.4.29' not found (required by ~/lo/core/instdir/program/libuno_sal.so.3) > make: *** [~/lo/core/nlpsolver/Extension_nlpsolver.mk:29: ~/lo/core/workdir/Extension/nlpsolver/help/qtz/com.sun.star.comp.Calc.NLPSolver/Options.xhp] Error 1 ) Change-Id: Ib096de1c71c7d0c6d016c68599b421a925bebaae Reviewed-on: https://gerrit.libreoffice.org/c/core/+/107454 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2020-10-31Simplify makefile logic using new gb_notStephan Bergmann
Change-Id: I19cb227a52fe6cd55dbba7b28689c4de995837fd Reviewed-on: https://gerrit.libreoffice.org/c/core/+/105075 Tested-by: Jenkins Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
2020-10-22Add gbuild/platform/FREEBSD_POWERPC[64]_GCC.mk files.Gleb Popov
Add "freebsd_powerpc64" case to the checkOSandCPU switch in dp_platform.cxx. Change-Id: Iedee32ecb5b49ee99cd5cf7f8d7e0e33aef1c312 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104616 Tested-by: Jenkins Reviewed-by: Michael Stahl <michael.stahl@cib.de>
2020-10-22Cosmetic change for clarificationTor Lillqvist
It's better to have these two statements on just one line each so that they align nicely and it is more obvious that nothing weird is going on. Change-Id: Ia95f2d9691f05b14786f32bc530e4602f3d46877 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104647 Tested-by: Tor Lillqvist <tml@collabora.com> Reviewed-by: Tor Lillqvist <tml@collabora.com>
2020-10-16Rename CLANG_CC, CLANG_CXX configuration vars (avoid clash with scan-build)Stephan Bergmann
Clang's scan-build tool uses the CLANG_CXX environment variable (setting it up in the scan-build script to pass it to the ccc-analyzer script), but happens to erroneously set it to a non-existing path (see <https://reviews.llvm.org/D89481> "[scan-build] Fix clang++ pathname again"). So wrapping LO's autogen.sh and make in scan-build picked up that broken CLANG_CXX and caused build failures like > [CXX] external/skia/source/SkMemory_malloc.cxx > /bin/sh: ~/llvm/inst/bin/clang-12++: No such file or directory (see <https://lists.freedesktop.org/archives/libreoffice/2020-October/086113.html> "Re: llvm/clang static analyzer reports"). So rename CLANG_CXX, and for consistency also CLANG_CC and the various CLANG_CXXFLAGS_INTRINSICS_*, by prefixing each with LO_. Change-Id: Ib41cabe940f8bfb1997f74e865cca5725f859e07 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104383 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2020-10-09Split off REAL_BUILD_CC_FLAGS from REAL_BUILD_CCStephan Bergmann
...same as is done with REAL_CC and REAL_CC_FLAGS in solenv/gbuild/platform/com_MSC_class.mk. Otherwise, my clang-cl build (where CC and those REAL*_CC vars derived from it contains the executable plus a bunch of command line arguments) would fail in ExternalProject_libgpg-error with > Error: could not create process ""C:/llvm/inst/bin/clang.exe --driver-mode=cl -FIIntrin.h -fmsc-version=1927 -Qunused-arguments --target=x86_64-pc-windows-mscv -fansi-escape-codes -fcolor-diagnostics -ferror-limit=0" -nologo -EHsc -MDd -Gy -Ob1 -Oxs -Oy- -IC:/lo-clang/core/include -IC:/PROGRA~2/WI3CF2~1/10/Include/10.0.18362.0/ucrt -IC:/PROGRA~2/WI3CF2~1/10/include -IC:/PROGRA~2/MIB055~1/2019/COMMUN~1/VC/Tools/MSVC/1427~1.291/Include -IC:/PROGRA~2/WI3CF2~1/10/Include/10.0.18362.0/um -IC:/PROGRA~2/WI3CF2~1/10/Include/10.0.18362.0/shared -IC:/java/ADOPTO~1.NET/x64/JDK-15~1/include/win32 -IC:/java/ADOPTO~1.NET/x64/JDK-15~1/include -Zi -FS -I. -I. -Femkheader.exe ./mkheader.c -link -debug -LIBPATH:. -LIBPATH:C:/java/ADOPTO~1.NET/x64/JDK-15~1/lib -LIBPATH:C:/PROGRA~2/MIB055~1/2019/COMMUN~1/VC/Tools/MSVC/1427~1.291/lib/x64 -LIBPATH:C:/PROGRA~2/WI3CF2~1/10/lib/x64 -LIBPATH:C:/PROGRA~2/WI3CF2~1/10/lib/10.0.18362.0/um/x64 -LIBPATH:C:/PROGRA~2/WI3CF2~1/10/lib/10.0.18362.0/ucrt/x64 -LIBPATH:C:/PROGRA~2/WI3CF2~1/NETFXSDK/4.8//Lib/um/x64": 2 Change-Id: I7246377874d89d3c0c9bf1c7951d29b7e3ae674a Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104118 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2020-09-30Fix typosAndrea Gelmini
Change-Id: Ie6146de848b7c5bb7a8edc76a0652c9c623b7024 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/103723 Reviewed-by: Julien Nabet <serval2412@yahoo.fr> Tested-by: Jenkins
2020-09-30use -fpch-codegen rather than -fmodules-codegenLuboš Luňák
The -fmodules-codegen flag has been in Clang for quite a while, but it officially works with PCHs only with Clang11+, and even there's it's better to use the properly named -fpch-codegen. This also fixes a problem with Clang9 having only -fmodules-codegen but not the -fno-* variant, causing the build to break. Change-Id: I9a8c979426f95e8c1f77cbeab1df64390d7243b8 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/103677 Tested-by: Jenkins Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
2020-09-30bridges: add a Windows Arm64 UNO bridgeJan-Marek Glogowski
Since Microsoft follows the general ARM64 ABI calling conventions, and the SEH exception handling is the same, this result is a mixed port of the gcc3_linux_aarch64 bridge and the refactored x86-64 exception handling. I have no idea, if the complicated 32-bit handling in RaiseInfo() is needed, as the ARM64 trampolines definitly use 64-bit code. But since this is the first working version, I currently don't mind much ;-) There is definitly more potential for refactoring in the whole bridges directory... Change-Id: I9782a2e99c0231cdd1286af156ad312229eccf39 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/103642 Tested-by: Jenkins Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
2020-09-25Adapt PythonTest on macOS to PRE-/POSTGDBTRACEStephan Bergmann
Change-Id: Ie23f2fc2e588721ab87e35f52f306e342a5ab246 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/103368 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2020-09-25Run gdb with PYTHONWARNINGS=defaultStephan Bergmann
Otherwise, --enable-werror (setting PYTHONWARNINGS=error) would cause Cppunit-/PythonTest with CPPUNITTRACE='gdb --args' on Linux to print lots of messages like > Python Exception <class 'DeprecationWarning'> the imp module is deprecated in favour of importlib; see the module's documentation for alternative uses: > gdb: warning: > Could not load the Python gdb module from `/usr/share/gdb/python'. > Limited Python support is available from the _gdb module. > Suggest passing --data-directory=/path/to/gdb/data-directory. and > Traceback (most recent call last): > File "instdir/program/python.bin-gdb.py", line 48, in <module> > import gdb > File "/usr/share/gdb/python/gdb/__init__.py", line 23, in <module> > from imp import reload > File "/usr/lib64/python3.8/imp.py", line 31, in <module> > warnings.warn("the imp module is deprecated in favour of importlib; " DeprecationWarning: the imp module is deprecated in favour of importlib; see the module's documentation for alternative uses Change-Id: Id85b2ee6d3cfaeaf91659bfb649a3cb78c707447 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/103349 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2020-09-24Use -flto=thin in gb_LTOFLAGS for Apple's Clang (for macOS and iOS)Tor Lillqvist
Don't add any LTO flags into $CC and $CXX as we don't do that for other platforms either. But maybe we should? Currently, with the separate gb_LTOFLAGS, we have to handle each external project separately to make it build with LTO. Change-Id: I9761426585ebdfd976c74168218bd26bcc0e8517 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/103351 Tested-by: Jenkins Reviewed-by: Tor Lillqvist <tml@collabora.com>
2020-09-24PythonTest CPPUNITTRACE='gdb --args' needs to tunnel setting LD_LIBRARY_PATHStephan Bergmann
...so that it doesn't affect gdb itself, just like CppunitTest already did. Otherwise, e.g. make PythonTest_sc_python CPPUNITTRACE='gdb --args' in a Linux ASan and --enable-python=fully-internal build on Fedora 32 would fail with > gdb: symbol lookup error: instdir/program/libpython3.8.so.1.0: undefined symbol: __asan_option_detect_stack_use_after_return as /usr/bin/gdb would try to use LO's instdir/program/libpython3.8.so.1.0 instead of /usr/lib64/libpython3.8.so.1.0. So, at least on Linux, the gb_PythonTest_PRECOMMAND settings are now passed into the test process either directly upfront (if gb_PythonTest_GDBTRACE is not set) or tunneled past gdb via gb_PythonTest_GDBTRACE (if it is set). At least on some platforms, gb_PythonTest_PRECOMMAND has a different value than gb_CppunitTest_CPPTESTPRECOMMAND, so gb_CppunitTest_GDBTRACE would reflect the settings from gb_CppunitTest_CPPTESTPRECOMMAND but not from gb_PythonTest_PRECOMMAND, so introduce a new gb_PythonTest_GDBTRACE (which is to gb_PythonTest_PRECOMMAND what gb_CppunitTest_GDBTRACE is to gb_CppunitTest_CPPTESTPRECOMMAND). macOS with its split gb_CppunitTest_{,PRE,POST}GDBTRACE variables will likely need some work to adopt that to PythonTest, but which is rather orthogonal to this change, so just note that as a TODO for now. Change-Id: I7e6681de638095cb58b44de381248ae7326ce412 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/103345 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2020-09-24Implement DEBUGCPPUNIT as a special case of CPPUNITTRACEStephan Bergmann
...so that gb_CppunitTest_GDBTRACE is only defined in a single place now (in solenv/gbuild/CppunitTest.mk), which should make reasoning about the code simpler Change-Id: I275a20c2fe473e68f4bb08a3fe2691a5809f91a7 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/103298 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2020-09-24Move some settings from just DEBUGCPPUNIT to all gdb CPPUNITTRACEStephan Bergmann
For one, it is probably a good idea to always (a) return the tested process' exit code, and (b) help users without a proper .gdbinit by setting auto-load-safe-path. And for another, this change works towards the goal of implementing DEBUGCPPUNIT as a special case of CPPUNITTRACE, ultimately simplifying the setting of gb_CppunitTest_GDBTRACE. Change-Id: I186434fa8645d8b068c69dbcfedf9b7d6374f99c Reviewed-on: https://gerrit.libreoffice.org/c/core/+/103297 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2020-09-24Expand the Windows CPPUNITTRACE=TRUE abbreviation in placeStephan Bergmann
...as a means towards defining gb_CppunitTest_GDBTRACE only in a single place (in solenv/gbuild/CppunitTest.mk), which should make reasoning about the code simpler Change-Id: I1a587ee08deb4e969e615b2544265b2ba34bd8af Reviewed-on: https://gerrit.libreoffice.org/c/core/+/103296 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2020-09-23Switch from -Oz to -O2 for iOSTor Lillqvist
Change-Id: Ie672543de3b3a4ab090309010aa9dcf637b01df5
2020-09-22Move MimeContentTypeFactory into vclJan-Marek Glogowski
The code is just used in vcl from LO's POV. This way we can drop the dtrans directory and get rid of yet an other library. Change-Id: Id77568e63a6fef4af30b49e035a9d76211b127a1 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/103210 Tested-by: Jenkins Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
2020-09-19The config.{guess,sub} in $SRCDIR is good for external projects, tooTor Lillqvist
No need for the older second copies in solenv/gbuild. Change-Id: I088f7d06b0727d1b336e3ba314b5c874d8ce3776 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/103027 Tested-by: Jenkins Reviewed-by: Tor Lillqvist <tml@collabora.com>
2020-09-17Report each value of LO_TEST_LOCALE in localized CppunitTestStephan Bergmann
Change-Id: I7ad091ec163f2324f2b8481e4caa6beb12188ea4 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/102896 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2020-09-16configure + gbuild: handle Windows Arm64Jan-Marek Glogowski
Change-Id: Idfc20c1234d693d6b402158b8bc782bd17cd3f4b Reviewed-on: https://gerrit.libreoffice.org/c/core/+/102850 Tested-by: Jenkins Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
2020-09-15WIN add and apply default msbuild platform+configJan-Marek Glogowski
Adds three Windows gb_* variables: - gb_MSBUILD_CONFIG_AND_PLATFORM can be passed as msbuild flags - gb_MSBUILD_PLATFORM maps debug / release settings - gb_MSBUILD_CONFIG maps the CPUTYPE to the default msbuild names and converts the users in external projects. Change-Id: Ie9b817721180d78d104db11c44241e4b3e46bba9 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/102701 Tested-by: Jenkins Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
2020-09-12Try fix multiple assignments of gidsMike Kaganski
************************************************ WARNING: multiple assignments of gids: ************************************************ GID: gid_Dir_Shlxthdl Assignments: 2 Change-Id: I4c2f53d8b996fc750465219f320fd8538a4d23f4 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/102494 Tested-by: Jenkins Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
2020-09-11cross-build: fix Java NI linkingJan-Marek Glogowski
LibreOffice has a JNI component on Windows and Linux, the officebean. Therefore we need a host JDK for linkage to the jawt, and a build JDK to compile the Java code. Change-Id: I4138628ab3ea2ef5900a5b4e9281050ae84e4eb5 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/102483 Tested-by: Jenkins Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
2020-09-11WIN cross: fix gpg-related library buildsJan-Marek Glogowski
Cross compiling these libraries requires to supply the cross- compiler via the CC_FOR_BUILD environment variable. Since we have to use the gcc-wrappers, we now need two different invocations with different inclues and libraries, but just have fixed environment variables. Also, the CC_FOR_BUILD clashes with LO's own variant, but that is easy to fix. So this change includes: - gcc-wrappers: new option --wrapper-env-prefix to add a prefix to the environment variable names - gcc-wrappers: new option --wrapper-print-cmdline to dump the real command called, when a verbose build is executed - gcc-wrappers: default to exe, if the output has no extension - unify build flags for gpg related libraries Change-Id: I4e6a6ba3c6e09237c8ffefa40ce61131290a3852 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/102482 Tested-by: Jenkins Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
2020-09-11Fix the minimal build-tools targetJan-Marek Glogowski
The revert commits change the build-tools target for a DESKTOP build to build the complete LO. This restores the original, minimal one and also adds a whitelist of allowd build types. OpenCL needs a configure switch, as it's status is also stored in a config header, so preventing the build is not enough. This also reverts: - commit 802161a505272732566210e9ebbd8fe1b23fb86d - commit 02d931a59e2966d0c2736db8dee7be3e3dcd6bae Change-Id: Ibfcb0c54e72da1b7c2e63c082ea6586520a787fa Reviewed-on: https://gerrit.libreoffice.org/c/core/+/102480 Tested-by: Jenkins Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
2020-09-11WIN drop --enable-64bit to select Windows targetJan-Marek Glogowski
This changes the Windows build to use the default configure switch to select the target / host of the compiled binaries to get the possibility to cross compile on Windows the "default" way. Note that selecting i686-pc-cygwin on x86_64 doesn't do a cross- compilation, as no special build tools are needed, because x86_64 can run x86 binaries just fine. A consequence of the change is the default target host, which is now the same then the build system, instead of the previous x86 default. Change-Id: I5584f34f665573ebac40d5d7753d96addeb84dbb Reviewed-on: https://gerrit.libreoffice.org/c/core/+/102479 Tested-by: Jenkins Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
2020-09-11Use { ... } for grouping that does not need a sub-shellStephan Bergmann
...in case that makes ever so small a difference, esp. on Windows (where spawning processes is known to be expensinve; but note that at least Bash seems to not spawn a sub-shell anyway when what is enclosed in ( ... ) is just a simple single process, which is commonly the case for these cppunittester invocations) Change-Id: Ie9200270743754e02fbfdb2b17239e9873031dab Reviewed-on: https://gerrit.libreoffice.org/c/core/+/102407 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2020-09-05fix macOS malloc debuggingNoel Grandin
which was introduced with insufficient testing in commit 302d85a931586313e04f82d206970dbc3bce9a47 add macOS malloc debugging env vars And this fix also avoids the potential problem of turning on malloc debugging for lldb itself. Change-Id: I0d65cefbd3ce8aea7223c6f0a3470b7ecbc8da89 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/102082 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2020-09-04revert the macos part of..Noel Grandin
commit 217122387f6e0ef657b8ba85eae082b448901cec Date: Tue Aug 25 12:23:13 2020 +0200 dont turn on malloc debugging for gdb itself because it doesn't work Change-Id: If6f1ac2ddc1683dab9d57df742c647083304af07 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/102056 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2020-08-27Remove tempfile againStephan Bergmann
Apparently missing from 941b567a41569260fef14a7337511e8f42337323 "tdf#131572 Add java 9 module info for libreoffice.jar". Change-Id: I6c87c79846a1711f78716410a5939e4b88a4f298 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/101490 Reviewed-by: Michael Stahl <michael.stahl@cib.de> Tested-by: Jenkins
2020-08-25dont turn on malloc debugging for gdb itselfNoel Grandin
fix the command such that, when we insert the malloc debugging env vars, we only do it for the executable itself, not for gdb, or some gdb operations will be very very slow. Before this fix, if I did a make Cppunit... CPPUNITRACE=... ran the program, then ran it again, the second time would take 5 min to get going. Now the second time it runs in seconds. Change-Id: Id08c3a82679164588b88af65a6a7b5f7dd19ff05 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/101318 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2020-08-20Use OSL_UNREACHABLEStephan Bergmann
...in those places that used some code conditional on ENABLE_LTO to work around (non-)unreachability wranings. This removes all uses of the ENABLE_LTO C/C++ macro, so it can go completely. Change-Id: I67544986cb2d3fcd8051caf87c5129bd1086408c Reviewed-on: https://gerrit.libreoffice.org/c/core/+/101087 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2020-08-14solenv: fix typo in CppunitTestMiklos Vajna
Added in commit 2edc86a592bc943fcffef0a8ae8db220e18e2b37 (Default all tests to run with the svp plugin, 2018-11-23), if svp is to be avoided, it has to be filtered out from the environment variables, not filtered for. With this, the unexpected SAL_USE_VCLPLUGIN=gen is gone from the generated cmdline when using gb_CppunitTest_use_vcl_non_headless. Change-Id: Ib666f3df007898165f2019f0a9b0677f679aa6e1 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100742 Tested-by: Jenkins Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
2020-08-12Enable --enable-lto --enable-skia when building with GCC+Clang on LinuxStephan Bergmann
...where all the Library_skia objects are built with Clang, so contain intermediate LLVM bytecode, but were then linked via GCC and ld or gold, which do not understand such object files and thus failed. So in gb_LinkTarget__command_dynamiclink use T_CC/T_CXX also for the linker command line. But then Clang would still be used for linking with the -fuse-ld=(ld or gold) $(USE_LD) suitable for GCC, and would still fail. So break T_USE_LD out of T_LDFLAGS and let gb_LinkTarget_use_clang override T_USE_LD with CLANG_USE_LD. At least for now, that CLANG_USE_LD (containing something like -fuse-ld=lld or -fuse-ld=lld --ld-path=... for the latter see d668c9a04d04d256fcbbd2165fe226f1db88256b "Adapt --enable-ld to Clang 12 trunk --ld-path") needs to manually be passed into autogen.sh, it is not computed in configure.ac. Then building Library_skia would still fail to link against StaticLibrary_libpng, as that only contains intermediate GCC object code, so make sure to build it with -ffat-lto-objects in this specific case. Change-Id: I0a104e53a8898cd9c2eb3b643e21954e853608cc Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100556 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2020-08-07Update config.{guess,sub} with latest versions and handle fallout of thatTor Lillqvist
From http://git.savannah.gnu.org/gitweb/?p=config.git;a=blob_plain;f=config.guess;hb=HEAD and http://git.savannah.gnu.org/gitweb/?p=config.git;a=blob_plain;f=config.sub;hb=HEAD . This time, do not apply the add-on change from 25a09c8776cc6088a5b2bf13dc84eb386c26bb7e to config.sub, but keep it pristine. Instead, let's start using the name "aarch64" instead of "arm64" for macOS and iOS in the autofoo context, as that is what those tools call it. Clang and Apple call it arm64, though. Change-Id: I1e05866c5fb08e0800cdfeaf7f6a71bfb43d1777 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100272 Tested-by: Jenkins Reviewed-by: Tor Lillqvist <tml@collabora.com>
2020-08-05Adapt --enable-lto to --without-parallelismStephan Bergmann
...which sets PARALLELISM to 0 and thus caused cc1: error: unrecognized argument to ‘-flto=’ option: ‘0’ at least with recent GCC 11 trunk Change-Id: Ifcfb6485c1c2efc8ab798686ca7fc2392d2cfc2c Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100171 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2020-07-29Port to FreeBSD aarch64miki
Change-Id: Ib18894db8c2943dd1502c96951545bb75a1944eb Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91978 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2020-07-29Add verbose climaker outputJan-Marek Glogowski
If you run gbuild verbose, make climaker output verbose too. Change-Id: I712f156bae33bfd403403675f4e2117282a7079c Reviewed-on: https://gerrit.libreoffice.org/c/core/+/99684 Tested-by: Jenkins Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
2020-07-28Adapt --enable-ld to Clang 12 trunk --ld-pathStephan Bergmann
...split from -fuse-ld with <https://github.com/llvm/llvm-project/commit/ 1bc5c84710a8c73ef21295e63c19d10a8c71f2f5> "[Driver] Add --ld-path= and deprecate -fuse-ld=/abs/path and -fuse-ld=rel/path", and now causing warnings (or even errors with -Werror) like '-fuse-ld=' taking a path is deprecated. Use '--ld-path=' instead when --enable-ld is configured as a full path. (--enable-ld was vague whether it supports full paths, but it appeared to work reasonably well at least with old versions of Clang.) Change-Id: I5a7dfd992b56aba78dd3646045fb9a827dc40321 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/99569 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2020-07-17use Clang11's -fpch-instantiate-templates if availableLuboš Luňák
The optimization to instantiate templates already while creating the PCH instead for every single compilation has been finally accepted upstream, but it's opt-in. Change-Id: Ia6456df05512329fc594f9ed0e400250c4f5029e Reviewed-on: https://gerrit.libreoffice.org/c/core/+/98948 Tested-by: Jenkins Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
2020-07-10replace usage of whitelist with allowlistThorsten Behrens
Background and motivation: https://tools.ietf.org/html/draft-knodel-terminology-02 [API CHANGE] officecfg::Office::Common::Misc::OpenCLWhiteList -> OpenCLAllowList Change-Id: I65636b19b13e4af1e4851f70e78053f3443d6bb1 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/98181 Tested-by: Thorsten Behrens <Thorsten.Behrens@CIB.de> Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>