Age | Commit message (Collapse) | Author |
|
Breaks tinderbox build in cppumaker
This reverts commit 3393e04edd0f4f6baf5faeda2ed761332f45d9e5.
|
|
acc, expwrap and store are loaded at startup anyway
Change-Id: I28020962238be3fe73a1b1dbba59a45c7d3c1e1d
Reviewed-on: https://gerrit.libreoffice.org/23459
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Oliver Specht <oliver.specht@cib.de>
|
|
-Werror is generally suppressed in Bison-generated C/C++ code (as in all other
generated code) to silence warnings from the Bison skeleton code. And the Clang
plugins suppress warnings in generated WORKDIR code based on the presumed source
location (i.e., taking #line directives into account). So introduce a new
PLUGIN_WARNINGS_AS_ERRORS mode where warnings from Clang plugins are reported as
errors even if -Werror is suppressed. That way, any warnings in the Bison
skeleton code still do not lead to compilation errors, while (at least plugin-
emitted) warnings in the genuine source code do.
Unfortunately this cannot also be enabled for Flex source code, as at least
Flex 2.5.39 generates poor code that does not properly prefix all skeleton code
with appropriate #line directives, so that some skeleton code would be mistaken
for genunie source code, and compilation would fail due to errors.
Also, %glr-parser Bison input appears to generate no #line directives at all (at
least with Bison 3.0.4), so all of connectivity/source/parse/sqlbison.y is
considered generated code and plugin warnings are still suppressed throughout.
Change-Id: Id746e81cbfa5f77628b0a34c7b82780948e7db08
|
|
...not already automatically enabled through the existing -W switches.
The set is assembled from information in
<https://gnu.wildebeest.org/blog/mjw/2016/02/15/looking-forward-to-gcc6-many-new-warnings/>.
All issues found have already been fixed with earlier commits.
Change-Id: I1abb6ba3a12c0ffb2756d06476e395b24e596f8e
Reviewed-on: https://gerrit.libreoffice.org/24219
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Tested-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Change-Id: I1920ece6ca683403d72a68b6705da2dea7c9ad9b
|
|
Change-Id: I8e053f5f0c6994c44485f61f5b3dc489271b8c5f
|
|
With MSVC 2015 it is triggered by odd code in Windows 8.1 SDK
DbgHelp.h and imagehlp.h.
Change-Id: I8694e4598fa8e3a6d6c6e8a0f94429af16f40c6b
|
|
Change-Id: I4ac94475b663c1ec1f5148b766a81a31adb1d49e
Reviewed-on: https://gerrit.libreoffice.org/23038
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Change-Id: I5445837ec4f647e91fe2aeab1251e48628f5e7f1
|
|
Change-Id: I55ba6e9c51a5bbfc366a36e8e0baef9cf05d5bbc
Reviewed-on: https://gerrit.libreoffice.org/22933
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com>
|
|
Change-Id: Ia1fb09b89d3ccc3c7f4a11799f0d8eac99c6c7a9
|
|
Change-Id: I006d21b6240898cbaed2b382276654d3ad8076a2
Reviewed-on: https://gerrit.libreoffice.org/22732
Reviewed-by: David Ostrovsky <david@ostrovsky.org>
Tested-by: David Ostrovsky <david@ostrovsky.org>
|
|
Change-Id: I10b3269e3ac81f0c7e3fa60229e2b1e3fcab512b
Reviewed-on: https://gerrit.libreoffice.org/22590
Reviewed-by: David Ostrovsky <david@ostrovsky.org>
Tested-by: Jenkins <ci@libreoffice.org>
|
|
Change-Id: Id367ad150fa50cb3ed4d32cb5c7c8408f91e371a
Reviewed-on: https://gerrit.libreoffice.org/22544
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com>
|
|
Much more readable.
Change-Id: I87c696e3f2f17a98c83eab9ed8149b79902be0fe
Reviewed-on: https://gerrit.libreoffice.org/22431
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: David Ostrovsky <david@ostrovsky.org>
Reviewed-by: Michael Stahl <mstahl@redhat.com>
|
|
... command line limit, so use try to use $(file) on that platfrom while
preserving the existing echo code for Macs.
Change-Id: I65e3f6b082df201830fa16375a9fad407cff8085
Reviewed-on: https://gerrit.libreoffice.org/22409
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Michael Stahl <mstahl@redhat.com>
|
|
(regression from c33fb4828e2fda2648f11614a20ce0172d4af3db)
Change-Id: I8d20343401cab13a3b6721d0d3016edaeb7dd898
|
|
Pyuno creates packages with a "/" in the name, so need to escape that
too. Since it's getting a bit repetitive facotr out a function to do
the escaping.
Change-Id: I4481d6a291b384390723a04e3e82615e73edc7eb
Reviewed-on: https://gerrit.libreoffice.org/22392
Reviewed-by: David Ostrovsky <david@ostrovsky.org>
Tested-by: Jenkins <ci@libreoffice.org>
|
|
using the gb_DBGARGS variable.
This value is appended to the gdb "set args" command.
Change-Id: Ic15754d7edcc448f283522338900684a29a03cf3
Reviewed-on: https://gerrit.libreoffice.org/21364
Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
Tested-by: Jan-Marek Glogowski <glogow@fbihome.de>
|
|
Change-Id: I28e55bbcae6d16af4ef230d1ec263ab356e1049f
Reviewed-on: https://gerrit.libreoffice.org/21586
Reviewed-by: Michael Stahl <mstahl@redhat.com>
Tested-by: Michael Stahl <mstahl@redhat.com>
|
|
Change-Id: I2cbf3fc3ba2df7131ce9328de8bada701ecb3b65
Reviewed-on: https://gerrit.libreoffice.org/21524
Reviewed-by: Michael Stahl <mstahl@redhat.com>
Tested-by: Michael Stahl <mstahl@redhat.com>
|
|
Change-Id: I560384d91fa5c6ba42123330369628419fe1bc1c
|
|
Change-Id: I534848c4be8eadf86b7f31ac742a80982405f56f
|
|
Seems to break the tinderbox, needs more work?
This reverts commit 6aaf1ec5a781b50ceda6d0d288a43dba435be2ce.
|
|
Change-Id: I2d4cedac4081260c5147d8c11904d042c765e3a6
Reviewed-on: https://gerrit.libreoffice.org/21557
Tested-by: Jenkins <ci@libreoffice.org>
Tested-by: Tor Lillqvist <tml@collabora.com>
Reviewed-by: Tor Lillqvist <tml@collabora.com>
|
|
When building with GCC, the gcc-ar, gcc-ranlib, and gcc-nm wrappers
must be used for the linker to properly resolve symbols from static
libraries containing LTO objects.
Likewise, clang requires the llvm-ar, llvm-ranlib, and llvm-nm
wrappers.
Change-Id: I539017b6da7732f998a49820dc01d0e6b302952a
Reviewed-on: https://gerrit.libreoffice.org/21502
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
... at least for dbulo.dll there are no such warnings.
Change-Id: Ib0c77e71ba5737ecd3e2f6812c3cf03010b7eff9
|
|
Change-Id: I9a5940027423ff0791fa7da0b79b617412ce6b86
Reviewed-on: https://gerrit.libreoffice.org/21209
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
|
|
... to avoid calling C:/Windows/system32/{sort,find}.exe, if those
happen to be first in PATH.
On a Windows 7 system, the other conflicts appear to be harmless,
we don't use "more", "expand", "timeout", "whoami".
Change-Id: Iceefeb7ee6725291b04c0eba465991bb1df96b57
Reviewed-on: https://gerrit.libreoffice.org/21175
Tested-by: Jenkins <ci@libreoffice.org>
Tested-by: Tor Lillqvist <tml@collabora.com>
Reviewed-by: Tor Lillqvist <tml@collabora.com>
|
|
Change-Id: I41e0731ddafc39ebcff1c3d8984f4f4f69d35aaf
|
|
...which contained the bulk of .cxx files that should logically go into
Library_pdfimport. It was likely (only) used so that check_targets
CppunitTest_sdext_pdfimport, Executable_pdf2xml, and Executable_pdfunip could
access the library's internals without exporting them. For the CppunitTest, use
the standard gb_CppunitTest_use_library_objects hack instead. For the two
Executables, make that _use_library_objects hack available for Executables, too.
(It is a bit unclear whether those two Executables are really needed, they are
only referenced from the dead dmake-based
sdext/source/pdfimport/test/testdocs/makefile.mk and from vcl/README,
respectively; but just keep them alive for now.)
Change-Id: Ia2478508de216678be7a2302aba0c48f80de9d91
Reviewed-on: https://gerrit.libreoffice.org/20645
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Change-Id: I10d30097a7e7099d43a6760489dcbbf405828960
Reviewed-on: https://gerrit.libreoffice.org/20267
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
Tested-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
|
|
...as in com_GCC_defs.mk
Change-Id: Ic086a6f08945698cd35a312f98491cba403b9582
|
|
since it's Visual Studio specific, not Windows-specific
Change-Id: I740b392f1aa9ff9c534b0e2fe021de7f93495e4c
|
|
Change-Id: I6e65f70848eb466c7c08fcffb478a2963b91cdb6
|
|
it just makes the build error logs harder to read.
it's not like we are ever going to fix it.
Change-Id: I26be4deb7b72d142fcee62ea3eee2dd0bd607716
Reviewed-on: https://gerrit.libreoffice.org/20284
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
|
|
These are presumably a relic from pre-INSTDIR days: gengal should find
all this stuff via redirect.rc, fundamental.rc and services.rdb.
The UNO services should be found via services.rdb too after the
dependency was added in the previous commit.
Change-Id: I3d452acc7c4b05f254bef936c4f18a2ea8615cf8
|
|
Change-Id: I717d3289d63787cd8148d480783092868a60873b
|
|
Change-Id: I59af9f5a9d0d3e189280f98bb18e522dc9ebf2fb
|
|
Change-Id: Ic1e25f3f945e32f2b99c2cfedc4b8d695635bd8b
|
|
Change-Id: I17ebccb268e317346ca05d6a112ec33357684876
|
|
The component files are not necessary as gb_Executable__register_gengal
already adds the service.rdb; the gengal.rc is insufficient as that file
references fundamental.rc which is missing, so depend on the Package
instead. Also, INSTDIR is wrong for cross-compile.
Change-Id: I30711a51e04539832657a242d2573c626b1cf8ec
|
|
Change-Id: I628b7986253108a6f9ea8eed28ae3b3f0f8b91b1
|
|
Change-Id: Iab196e87e0eb4b063d77fe4af10c58b8355665c4
Reviewed-on: https://gerrit.libreoffice.org/20210
Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
Tested-by: Michael Meeks <michael.meeks@collabora.com>
|
|
Change-Id: I9d1ae01d29ff08ce9de34d86088c1bbbe53bd159
|
|
This reverts commit de37e7ae4808dc9cdd84a8c21411958ed51db56.
This should now be properly fixed with a2185987.
nitpick: That commit was refering to a wrong commit anyway, it should
have refered to a2185987d instead.
Change-Id: Ic46db94568ac4074f471f42678cdf8d9fb32ef67
|
|
Change-Id: Icc7f6f063e6a31d3d84e9757dc45859f6fe1703c
|
|
...broken with 4f76e46290b26deeb4f9660c580a5ffef58a557e "make CPPUNITTRACE=TRUE
a shortcut for debuging on WNT"
Change-Id: Ie6a601da3d4310ea4d43410f8a8e2ddaa16e287d
|
|
Change-Id: Ie834e1e1592163977435f0af6dde04c4fe86c1bf
Reviewed-on: https://gerrit.libreoffice.org/19977
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: jan iversen <jani@documentfoundation.org>
Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
|
|
- get platform switch logic out of shell script
- platform specific stuff should be in $(GBUILDDIR)/platform, if
possible
- do not suggest shell-polluting exports, which might lead to confusion
later, suggest setting the var as a one-off addition to the make
command
- move exception caching down, as it is assumed to be used rather
rarely
Change-Id: I01db3ca07582a6c6213ebac86fddabb28c925d50
Reviewed-on: https://gerrit.libreoffice.org/19976
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: jan iversen <jani@documentfoundation.org>
Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
|