| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|
|
|
| |
Change-Id: Ic3374bbf71cd7975ce1523a5a9ab47e57f5aad51
|
|
|
|
| |
Change-Id: I023d79c0d8928d8b43df67996ca4eb9dc6bfb25e
|
|
|
|
| |
Change-Id: I35c2121069c2bcf72f6a4db66ea0703b17f96a50
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
| |
Change-Id: I096211333c563b501627fef0dbdec8c3c98d501a
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
| |
Change-Id: Ib85895c21cef9baeccc468b476cbd8170ee4c9ce
|
|
|
|
| |
Change-Id: I2a4efc9ade69ab53da81c72528e18ecb99e99757
|
|
|
|
|
|
| |
It was never fully implemented, and its usefulness is unclear.
Change-Id: I86dee8ee6326bdfaa178260c76188261aef2fb4c
|
|
|
|
|
|
|
| |
Change-Id: Idedad62ac28804f4725d6bda2f4633ed79c01535
Reviewed-on: https://gerrit.libreoffice.org/31571
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Michael Stahl <mstahl@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
| |
Change-Id: I3de0776060c88c3f5705c67ea32ea1843876553e
|
|
|
|
| |
Change-Id: Ia38672028668959bf3d5541fe4ddb9fb72848617
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
| |
Change-Id: I37ee0cc58614dbf56a67a2bf0b08d3bf85d34ea7
|
|
|
|
|
|
|
|
|
|
| |
...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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
| |
Otherwise, the 'newline' is passed as a second parameter to
gb_Module_add_moduledir which breaks build. :/
Change-Id: I02eba5ddef9f7a6dccbb7d135a7e8c68629da9c4
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
| |
Change-Id: Ic3454a1fb7f32998ba468273ae7265ea7814debd
Reviewed-on: https://gerrit.libreoffice.org/31310
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
| |
- 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>
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
| |
- 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>
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
| |
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.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- 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>
|
|
|
|
|
|
| |
...instead of ignoring failures in every but the last iteration
Change-Id: Ie80e78f75aaec5e9b1625c2e1ef740adf31fab1b
|
|
|
|
|
|
|
|
|
|
| |
...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
|
|
|
|
| |
Change-Id: I4dc6bebb41b0e1beee58631fdf92dea82ae38db7
|
|
|
|
| |
Change-Id: I28bb2a33c02d8eadced584d3d3f2b62b2e847324
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
| |
...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
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
...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
|
|
|
|
|
|
|
| |
Change-Id: Id8d1c01b17e711a057a9f3f20e1d0f955b2ea6c8
Reviewed-on: https://gerrit.libreoffice.org/30690
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Andras Timar <andras.timar@collabora.com>
|
|
|
|
|
|
|
| |
Change-Id: I45d45513b102f4fdcb55e8de20b95b37f66ea463
Reviewed-on: https://gerrit.libreoffice.org/30658
Reviewed-by: Michael Stahl <mstahl@redhat.com>
Tested-by: Michael Stahl <mstahl@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
| |
Change-Id: I630fd447992ada459190acd1ee310bd2d38820b9
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
| |
- 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>
|