summaryrefslogtreecommitdiffstats
path: root/solenv
Commit message (Collapse)AuthorAgeFilesLines
...
* More blind fix attempt for MSVCStephan Bergmann2016-12-171-0/+3
| | | | Change-Id: Ic3374bbf71cd7975ce1523a5a9ab47e57f5aad51
* More blind fix attempt for MSVCStephan Bergmann2016-12-171-0/+3
| | | | Change-Id: I023d79c0d8928d8b43df67996ca4eb9dc6bfb25e
* dba_ORowSet needs HAVE_FEATURE_DBCONNECTIVITYCaolán McNamara2016-12-171-1/+1
| | | | Change-Id: I35c2121069c2bcf72f6a4db66ea0703b17f96a50
* also read Library check targetsBjoern Michaelsen2016-12-171-1/+1
| | | | | | | | | | | this should fix libscqahelper ending up with a wrong MAKEFILE, which then confuses gbuild-to-ide Change-Id: Ibf030b3d9dbf3ba92b7e119e847d7cd6104dedb2 Reviewed-on: https://gerrit.libreoffice.org/32107 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org> Tested-by: jan iversen <jani@documentfoundation.org>
* Try to fix Android and iOS buildTor Lillqvist2016-12-151-1/+1
| | | | | | | | | | Fix what probably is fallout from a7c35729e00f18f79156b3f8f57472506f786074. Just bypass all of xmlsecurity for Android and iOS. At least the iOS demo app, TiledLibreOffice, builds then. Change-Id: Ibc9486c0d67d1aeafa08932809b23ceeb9b5c2f3
* solenv: adapt SwPositionPrinter to BigPtrEntry renamingMichael Stahl2016-12-141-4/+4
| | | | Change-Id: I096211333c563b501627fef0dbdec8c3c98d501a
* update to gbuildtojson for osxjan Iversen2016-12-141-1/+5
| | | | | | | | Added extra gb_ macros, which seems to be used only for osx Removed second jurt in BLACKLIST (assuming it is not like "not not True") Change-Id: I989eb2e6de6f0edde7902b51f829442bd598eec2
* Don't leak .png files into tmpKatarina Behrens2016-12-141-1/+4
| | | | | | | | | | | | | | | | | | Temp directory created by copy_images (which contains copies of .png files from any given icon-themes folder) was left behind. Happening at least since commit 4e3dc8c141c2efd037c1f214b5edff071812b6a8 Date: Fri Jun 3 17:09:14 2016 +0200 remove use of Archive::Zip With packimages.pl now used to package also screenshot images (in multiple languages), tmp fills up quickly. This is fatal on Windows Change-Id: Ie35c33277a80ed9085815f30b846bb97a365afb1 Reviewed-on: https://gerrit.libreoffice.org/31947 Reviewed-by: Katarina Behrens <Katarina.Behrens@cib.de> Tested-by: Katarina Behrens <Katarina.Behrens@cib.de>
* Remove some not used SCPDEFSMatúš Kukan2016-12-121-1/+1
| | | | Change-Id: Ib85895c21cef9baeccc468b476cbd8170ee4c9ce
* Fix removal of the temporary dyldpathfileStephan Bergmann2016-12-091-6/+10
| | | | Change-Id: I2a4efc9ade69ab53da81c72528e18ecb99e99757
* Retire the --disable-export optionTor Lillqvist2016-12-081-1/+0
| | | | | | It was never fully implemented, and its usefulness is unclear. Change-Id: I86dee8ee6326bdfaa178260c76188261aef2fb4c
* tdf#97803: gbuild: Check that every package is registeredMatúš Kukan2016-12-072-0/+22
| | | | | | | Change-Id: Idedad62ac28804f4725d6bda2f4633ed79c01535 Reviewed-on: https://gerrit.libreoffice.org/31571 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* Make the CPPUNITTRACE="lldb --" thing work again (on macOS)Tor Lillqvist2016-12-061-0/+6
| | | | | | | | | | | | It doesn't seem to work (any longer in 10.12) to pass DYLD_LIBRARY_PATH in the environment to the process to be run under lldb. We need to make lldb execute an explicit 'settings set target.env-vars' command. Use a temporary file to pass that command. I could not figure out where to call the gb_CppunitTest_POSTGDBTRACE macro I define, to remove the temporary file, sorry. Help appreciated. Change-Id: I46bd05f14623639e6ce10112b92ef497cdbb0755
* for oss-fuzz check LDFLAGS for -fsanitize tooCaolán McNamara2016-12-051-1/+1
| | | | Change-Id: I3de0776060c88c3f5705c67ea32ea1843876553e
* loplugin:unnecessaryoverride (dtors)Stephan Bergmann2016-12-051-0/+1
| | | | Change-Id: Ia38672028668959bf3d5541fe4ddb9fb72848617
* make passstuffbyref plugin ignore std::move'd paramsNoel Grandin2016-12-051-0/+1
| | | | | | | | | request from vmiklos Change-Id: If263beb0623d725e406003bb1660df10fe4b4e35 Reviewed-on: https://gerrit.libreoffice.org/31555 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* gbuildtoide: builddir is not necessarily same as srcdirChristian Lohmaier2016-12-041-3/+6
| | | | | | | Change-Id: Ied9f9abcc75f2edcd518ac247907f573aa21f35e Reviewed-on: https://gerrit.libreoffice.org/31158 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
* No component condition for a package file listMatúš Kukan2016-12-021-1/+1
| | | | | | | | This is to fix a build after 67398a8d9746dbed743beb702a8cff2eac2bbf99 No idea if packages should contain conditions too, but at least gid_File_Bin_UnpackUpdate did not. Change-Id: I56e1b23f6eeca9c62cdadd8d93de6ccda89b20c4
* Remove space at end of comment lineStephan Bergmann2016-12-021-1/+1
| | | | Change-Id: I37ee0cc58614dbf56a67a2bf0b08d3bf85d34ea7
* Remove unnecessary -wdXXXXStephan Bergmann2016-12-021-25/+2
| | | | | | | | | | ...that no longer trigger any warnings with the current code base under MSVC 2013 and beyond Change-Id: I14cad441518134f7c4321fdd132e0ff049d3b1b9 Reviewed-on: https://gerrit.libreoffice.org/31495 Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Stephan Bergmann <sbergman@redhat.com>
* gbuild: replace -O0 with -Og for GCC --enable-debug/dbgutil buildsMichael Stahl2016-12-011-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With Fedora 25 GCC 6.2.1 and my usual config (except without --with-package-format since that is slowest now) on commit 086631af59636cd9a6a45b747a1bc59b4b547794 i get with CCACHE_DISABLE=1: With -O0: "make check" 44490.70user 3127.01system 1:15:23elapsed 1052%CPU (0avgtext+0avgdata 2541312maxresident)k "make check", incremental 7368.19user 153.98system 15:30.06elapsed 808%CPU (0avgtext+0avgdata 510200maxresident)k "make CppunitTest_sc_functions_test" 272.13user 0.65system 4:33.47elapsed 99%CPU (0avgtext+0avgdata 258884maxresident)k With -Og: "make check" 46076.01user 3170.91system 1:16:19elapsed 1075%CPU (0avgtext+0avgdata 2613716maxresident)k "make check", incremental 5478.33user 157.07system 11:50.42elapsed 793%CPU (0avgtext+0avgdata 454980maxresident)k "make CppunitTest_sc_functions_test" 188.48user 0.62system 3:09.59elapsed 99%CPU (0avgtext+0avgdata 259096maxresident)k So we now have so many tests that the time taken by GCC doing optimization is brought back via the tests finishing faster, particularly if we assume a non-negligible ccache hit rate for developers in practice. Add this to gb_DEBUG_CFLAGS instead of gb_COMPILERNOOPTFLAGS because presumably the bridges code that uses gb_COMPILERNOOPTFLAGS really wants -O0; the gb_DEBUG_CFLAGS is added later so -Og overrides -O0. It is an open question how well debugging in gdb actually works with -Og, some experimentation is needed; "man gcc" claims: If you are not using some other optimization option, consider using -Og with -g. With no -O option at all, some compiler passes that collect information useful for debugging do not run at all, so that -Og may result in a better debugging experience. Change-Id: I103499f398b69397cf5aa9993a242680966ce999 Reviewed-on: https://gerrit.libreoffice.org/31334 Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com> Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* tdf#97803: gbuild: Check that resource targets are registeredMatúš Kukan2016-12-013-3/+9
| | | | | | | | | | | And replace gb_AllLangResTarget_ALLTARGETS with gb_AllLangResTarget_REGISTERED which should have the same content and is already used. This actually helped to find a problem, fixed in 52d409f0d657f314a53f945c9ffb5b8025bea034 Change-Id: Iae551d7be221c5655dee1bc9ad273c8822d45178
* fix cross-toolset: gb_Helper_optional_for_host can't contain 'newline'Matúš Kukan2016-12-011-2/+3
| | | | | | | Otherwise, the 'newline' is passed as a second parameter to gb_Module_add_moduledir which breaks build. :/ Change-Id: I02eba5ddef9f7a6dccbb7d135a7e8c68629da9c4
* gbuild: Prevent wrong use of gb_Helper_optional_for_hostMatúš Kukan2016-11-303-0/+6
| | | | | | | | | | It's empty, unless gb_Side=build. This fixes commit 1193443b15b0e38cbfb41573b31acf9a8d90ce2c Change-Id: I83efad36d8f40c460d09ee1bd1addd7fb26c177b Reviewed-on: https://gerrit.libreoffice.org/31419 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* loplugin:vclwidgets check for assigning from VclPt<T> to T*Noel Grandin2016-11-291-0/+1
| | | | | | | | | | | | | | Inspired by a recent bug report where we were assigning the result of VclPtr<T>::Create to a raw pointer. As a consequence, we also need to change various methods that were returning newly created Window subclasses via raw pointer, to instead return those via VclPtr Change-Id: I8118e0195a5b2b4780e646cfb0e151692e54ae2b Reviewed-on: https://gerrit.libreoffice.org/31318 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* de-blacklist writerfilter from gbuildtojsonBjoern Michaelsen2016-11-281-1/+1
| | | | | | | Change-Id: Ic3454a1fb7f32998ba468273ae7265ea7814debd Reviewed-on: https://gerrit.libreoffice.org/31310 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
* add cppunit tests to gbuildtojsonMarkus Mohrhard2016-11-282-0/+22
| | | | | | | Change-Id: I655190464e2b4289f5c51b194821f9288272d301 Reviewed-on: https://gerrit.libreoffice.org/31271 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
* fix around cornercases with cxxobjectsBjoern Michaelsen2016-11-251-0/+9
| | | | | | | Change-Id: Ibd5dad8ca69fa952e4751af961ad957017fbbe40 Reviewed-on: https://gerrit.libreoffice.org/31181 Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com> Tested-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
* rename foo target to something more meaningfulBjoern Michaelsen2016-11-251-5/+6
| | | | | | | | | | - also use gb_Helper_MISCDUMMY were possible (this might fix the 'circular deps' warnings Change-Id: I419f39973b5c785c1d8ccba9f8ed75a77f08ffa2 Reviewed-on: https://gerrit.libreoffice.org/31117 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
* store makefile for targets in json tooBjoern Michaelsen2016-11-253-1/+5
| | | | | | | Change-Id: Iae2f497da5d0ee4f8961a5655573e45a562dd76d Reviewed-on: https://gerrit.libreoffice.org/31116 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
* use gbuild prefix for tempdirsBjoern Michaelsen2016-11-251-5/+5
| | | | | | | Change-Id: Icc262c51fd70ae35b44722be8d4ab767b69ae2a8 Reviewed-on: https://gerrit.libreoffice.org/31094 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
* rename the core parts of this from gbuildtoide to gbuildtojsonBjoern Michaelsen2016-11-255-25/+28
| | | | | | | | | | - the json stuff is universal and not limited in use to IDEs - the IDE stuff should keep its gbuildtoide name Change-Id: If4f190fc6dffba219334d57a53c826515ed57c34 Reviewed-on: https://gerrit.libreoffice.org/31093 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
* Revert "Revert "improvements to gbuildtoide""Bjoern Michaelsen2016-11-252-6/+23
| | | | | | | | | | This reverts commit ab9c578ed17cfd9c9a49f22d83f33b0546faedba, except for the problematic copy-all-SRCDIR top-level test. Change-Id: Ie9c2fd725199768ec04852a9bc09b09a331ce6ad Reviewed-on: https://gerrit.libreoffice.org/31092 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
* Revert "improvements to gbuildtoide"Michael Stahl2016-11-222-74/+5
| | | | | | | | | This reverts commit 294bfdfaada9ba85ddcf853650a8da9c10f25281. Unfortunately this shutil.copytree(self.srcdirnative, self.tempsrc) copies the entire SOURCEROOT into /tmp, with WORKDIR too if it's not an out-of-tree build, which is quite an excessive use of space, tmpfs or otherwise, and will no doubt make CI very happy.
* improvements to gbuildtoideBjoern Michaelsen2016-11-222-5/+74
| | | | | | | | | | | | | | | - move supported module selection from the test directly into gbuild, it should not be possible now to have 'make gbuildtoide' failing, it should skip problematic parts - move from whitelisting to blacklisting bad modules - toplevel 'make gbuildtoide' should work now, so add test (limit to non-Windows for now, as Windows is stupid and slow) - remove spurious debug output in test Change-Id: I9838dd2fa091e5fb3e56f414f60164086e8f2076 Reviewed-on: https://gerrit.libreoffice.org/31072 Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com> Tested-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
* Exit gb_CppunitTest_localized loop on first failureStephan Bergmann2016-11-222-2/+3
| | | | | | ...instead of ignoring failures in every but the last iteration Change-Id: Ie80e78f75aaec5e9b1625c2e1ef740adf31fab1b
* New loplugin:oslendianStephan Bergmann2016-11-221-0/+3
| | | | | | | | | | ...to catch new places where defined'ness of OSL_BIG/LITENDIAN would be checked without osl/endian.h being included; cf. e2f08f9def0869460ad38a1c2adb450778290f6e "connectivity, sc: add missing #include <osl/endian.h>" and 2b14fb3a4f92b928f0a5fc536c6a5f4a6e51a9b8 "cppcanvas, oox: add missing #include <osl/endian.h>". Change-Id: I167c8916a01391b7dacad7325153ccf35d3ba9dc
* Sort lines lexicographicallyStephan Bergmann2016-11-221-2/+2
| | | | Change-Id: I4dc6bebb41b0e1beee58631fdf92dea82ae38db7
* add missing deps on ui filesDavid Tardon2016-11-211-0/+10
| | | | Change-Id: I28bb2a33c02d8eadced584d3d3f2b62b2e847324
* new loplugin datamembershadowNoel Grandin2016-11-211-0/+1
| | | | | | | Change-Id: Ib14319848bafd1fe7e0e663c434bbdeef5e98ecf Reviewed-on: https://gerrit.libreoffice.org/30963 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* gbuild allow unitest to run once per lang in WITH_LANG_LISTNorbert Thiebaud2016-11-201-1/+15
| | | | | | | | | | | | | | | this is motivated by the new screenshot feature the initial proposed solution involved running make screenshot once per lang which took ~6 hours for --with-lang=ALL on tb68 a reasonnably big windows slavebot. with this patch, one can run make screenshot just once and get all the screenshot the elapsed time is 36 inutes on the same box/same config a 10x improvement. Change-Id: I4339caebf915c118aa455de2a7e56e1a4e413939 Reviewed-on: https://gerrit.libreoffice.org/30970 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com>
* Have a specific TARGETTYPE for CompilerTest after allStephan Bergmann2016-11-172-1/+3
| | | | | | | | | | | ...instead of leaving it empty. (645583dfd374c8b02f3c0eeba6233a0bb5884d68 "New compilerplugins/clang unit tests": "Checking the input files is implicitly phony, as the compilation step doesn't generate any object files, and the link step does nothing because there is no gb_LinkTarget_set_targettype for CompilerTest.") In preparation for using compilerplugins/clang with clang-cl on Windows. Change-Id: Ica4f16a4b249537f78ce21fcbe7c4afea8214821
* new loplugin finalprotectedNoel Grandin2016-11-161-0/+1
| | | | | | | | | look for final classes, and make sure they don't have protected members Change-Id: I1fa810659bba02b61a5160dbfd8e24185ec9abf4 Reviewed-on: https://gerrit.libreoffice.org/30895 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* New compilerplugins/clang unit testsStephan Bergmann2016-11-158-7/+91
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ...to check that loplugin produces warnings/errors as expected: * Clang has a -verify switch that makes it easy to write test input .cxx files that list in comments all the warnings/errors that are expected, and let Clang check those expectations instead of generating object code. See include/clang/Frontend/VerifyDiagnosticConsumer.h in the Clang source tree for documentation. * Introduce a CompilerTest gbuild class that uses the existing LinkTarget class as much as possible. Checking the input files is implicitly phony, as the compilation step doesn't generate any object files, and the link step does nothing because there is no gb_LinkTarget_set_targettype for CompilerTest. The setup at least works for Clang on Linux (will need adaptions for Clang on Windows; compilers other than Clang are not relevant for now given this is used to check compilerplugins). * Definition of gb_CFLAGS_WERROR in solenv/gbuild/platform/com_GCC_defs.mk needs to be lazy ('=' vs. ':=') so that CompilerTest can override it: The Clang -verify mode wants the input files to specify whether the loplugin diagnostics are warnings or errros, so they consistently need to be errors independent of --enable-werror configuration. * A first (example) test is in compilerplugins/clang/test/salbool.cxx. The corresponding gbuild CompilerTest instance is in solenv/CompilerTest_compilerplugins_clang.mk for now. The reason for that odd split across compilerplugins/ and solenv/ is that there is no compilerplugins/Modules_compilerplugins.mk file, so this setup is the easiest hack for now (to be cleaned up). (Another area that could be improved is that all test files need to be listed explicitly in the CompilerTest_*.mk file, instead of, say, using all .c/.cxx/.m/.mm files in a specified directory.) * The test is run somewhat late during a top-level 'make', after loplugin has already been used in compilation. But it can be run manually (e.g., 'make solenv') when making changes to loplugin during development. Change-Id: I01e12fb84887d264ac03ef2484807458c2075af4
* typo fix: ressource -> resourceAndras Timar2016-11-081-3/+3
| | | | | | | Change-Id: Id8d1c01b17e711a057a9f3f20e1d0f955b2ea6c8 Reviewed-on: https://gerrit.libreoffice.org/30690 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Andras Timar <andras.timar@collabora.com>
* Fix typos (also in the code)Andrea Gelmini2016-11-071-1/+1
| | | | | | | Change-Id: I45d45513b102f4fdcb55e8de20b95b37f66ea463 Reviewed-on: https://gerrit.libreoffice.org/30658 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
* Allow including/packaging help-specific imagesKatarina Behrens2016-11-041-6/+24
| | | | | | | | | | from other than icon-themes folder Change-Id: Iba1808e642fde8d0370b59550c7bdac1a3e4ac79 Reviewed-on: https://gerrit.libreoffice.org/28513 Reviewed-by: jan iversen <jani@documentfoundation.org> Tested-by: jan iversen <jani@documentfoundation.org> Reviewed-by: Olivier Hallot <olivier.hallot@edx.srv.br>
* Disable gbuildtoide test on macOS, tooTor Lillqvist2016-11-031-0/+2
| | | | Change-Id: I630fd447992ada459190acd1ee310bd2d38820b9
* disable gbuildtoide test on Windows as CI builders are unstableBjoern Michaelsen2016-11-021-0/+2
| | | | | | | Change-Id: Ib5eb4fece2bae3e04089f2389198df800620e6f8 Reviewed-on: https://gerrit.libreoffice.org/30483 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Miklos Vajna <vmiklos@collabora.co.uk>
* also stubify generated object addersBjoern Michaelsen2016-10-311-0/+2
| | | | | | | | | - This should unbreak vcl when configured with KDE/TDE CustomTargets Change-Id: I239e97b90174f3daef39e4a806a126cafba9d8e0 Reviewed-on: https://gerrit.libreoffice.org/30436 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>