Age | Commit message (Collapse) | Author |
|
Change-Id: Ic9912cdebfc75816c54495db4fef68ebeed32c50
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/166417
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
|
|
Previously, a custom BreakIterator was used for Thai grapheme clusters.
This change deletes the custom BreakIterator, in favor of the ICU
implementation.
Change-Id: Icec94c73a5734c2059786dfbba085f487c488d7c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/166156
Tested-by: Jenkins
Reviewed-by: Eike Rathke <erack@redhat.com>
|
|
Linking Executable/soffice.html kept warning about
> warning: JS library symbol '$allocateUTF8' is deprecated. Please open a bug if you have a continuing need for this symbol [-Wdeprecated]
> warning: deprecated item in EXPORTED_RUNTIME_METHODS: printErr use err instead.
> em++: warning: warnings in JS library compilation [-Wjs-compiler]
printErr had been introduced in f090004c5f236275ca5142fc578f0375872c0336 "WASM
adapt link and debug flags" and allocateUTF8 had been introduced in
ce60a3dd4dbff0dcb5b82c9053ae5d90f8ac929d "Add LOKit functions and whitelist
export for it to WASM", but in both cases no code seems to ever have made use of
either of those two methods, before or after.
Change-Id: I1e81ca6ddad748d91653f709c272328cc8f2b679
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/166296
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <stephan.bergmann@allotropia.de>
|
|
distros typically have their own set via C[XX]FLAGS, so make
this an optional argument
some notes on the options:
-Wp,-U_FORTIFY_SOURCE,-D_FORTIFY_SOURCE=2
https://www.redhat.com/en/blog/enhance-application-security-fortifysource
(I see Fedora has recently bumped to to 3 since Jan 2024
https://fedoraproject.org/wiki/Changes/Add_FORTIFY_SOURCE%3D3_to_distribution_build_flags
but here use 2 for now instead)
-Wp,-D_GLIBCXX_ASSERTIONS
https://fedoraproject.org/wiki/Changes/HardeningFlags28
-fstack-protector-strong
(We already apply this by default)
-fstack-clash-protection
https://fedoraproject.org/wiki/Changes/HardeningFlags28
-fcf-protection
https://fedoraproject.org/wiki/Changes/HardeningFlags28
https://cgit.freedesktop.org/libreoffice/core/commit/?id=af55dc3891f7950d392175004b2090cb0e54828e
and record the compiler flags in debuginfo
-grecord-gcc-switches
Change-Id: Ib05387bad8324b188bd4ed0ee327d6a7cf83973b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/163312
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Andras Timar <andras.timar@collabora.com>
(cherry picked from commit 33483058f6e27f39633114721f7329c90571101d)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/166289
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
|
|
writerfilter wants to convert incoming RTF and OOXML files into
writer's document model. But it currently has to do so by
manipulating the limited subset that we expose through the UNO
API.
This is both slower and less accurate than having access
to the full document model.
So move it inside, and then we can strip out various hacks, and
optimise imports.
Change-Id: Ie1114d28130ef5f9a786531bc552cb8ee7768015
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/165953
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
And fix a vector-related filter
Change-Id: Icd3f5bd5a04a1385dff3f3413792a7aad544a506
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/166029
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
|
|
<https://github.com/flathub/org.libreoffice.LibreOffice/commit/29463ebd29eb10ba7891c87bd699047e5e13deb7>
"Merge pull request #288 from
EliasTheGrandMasterOfMistakes/EliasTheGrandMasterOfMistakes-patch-2: Update JDK,
GVFS and ANT"
Change-Id: I021ddb9ca6b1fbec6cfe22784a9d023db2fa737c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/165920
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <stephan.bergmann@allotropia.de>
|
|
Change-Id: I193fd0c10d297555faa7a832718dbd6cd929a406
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/165762
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <stephan.bergmann@allotropia.de>
|
|
Change-Id: I19f466a272c821185bea4b45efd34392e525c0d1
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/165785
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
since fe4cd0cebdbf9412e26f8182e79f97bb844d94af
Date: Thu Feb 22 14:09:12 2024 +0100
Set button label directly in ui file
Change-Id: I53a39dffe79c5c5c4c4c8a0d95a333ba3af833bc
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/165603
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Tested-by: Jenkins
|
|
and remove OBroadcastHelperHolder, since nothing uses it anymore
Change-Id: Id07b4589e0893950974c6bf41e0d4f2d8dc7ee81
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/165491
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: Ie58bde91e1c96a32f02f64bc0eb8d864d3680e6e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/165388
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
These will likely be significantly changed in the process
of updating the code to support current CPDB
(Common Print Dialog Backends) versions, see [1].
[1] https://lists.freedesktop.org/archives/libreoffice/2024-March/091748.html
Change-Id: I2d8fe1cfedb703e4843ade261e5c09e24dfe30b9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/165366
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
|
|
...and reorder the code a bit, so that the dbglevel-releated code is separated
from the gb_ENABLE_SYMBOLS_FOR-related code
Change-Id: I80649b32fd6cceb8df1e4e2e29e98508e28f338b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/165136
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <stephan.bergmann@allotropia.de>
|
|
...so no need to check the former in addition to the latter (which is done a few
lines up)
Change-Id: Iefe1ccbd3d22ca4e97425d20d1ff21f68d641b2c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/165135
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <stephan.bergmann@allotropia.de>
|
|
Change-Id: I6bb6046968a74c49cc8441a1529f9934c90ed50c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/165134
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <stephan.bergmann@allotropia.de>
|
|
Change-Id: I352a7c7659e11a4c0800d1a9d73468123b8a5654
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/165133
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <stephan.bergmann@allotropia.de>
|
|
Change-Id: I284e3601ad3d8fe7489e21182a98df40e8d9dbb5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/165132
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <stephan.bergmann@allotropia.de>
|
|
...left inconsistent by recent 76a986da868c52e9bd1cc6d219ed8f9441afd32b "Drop
upper-case DBGLEVEL, consolidate on lower-case dbglevel"
Change-Id: Ie10c011f2942a013e90c75e0fadd123695d97374
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/165131
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <stephan.bergmann@allotropia.de>
|
|
...(that was defined iff OSL_DEBUG_LEVEL >= 2) and replace its uses with
OSL_DEBUG_LEVEL directly
Change-Id: I807c15a02cc8ced9852287df0afb4808761d19d2
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/165067
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <stephan.bergmann@allotropia.de>
|
|
...than gb_DEBUGLEVEL as had been chosen by
448008d64c643d5a1aa2dc5cccc479efcd709a50 "only enable windows incremental
linking for debug builds"
Change-Id: Iabd2904596b3ac2a9d1c55d074cc929572615265
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/165077
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <stephan.bergmann@allotropia.de>
|
|
...and not for whatever dbglevel is set. (The latest claim that this hack is
still needed at all is in the 2020-12-14 commit message
e0a90aa3239621958bddbb30f98163e8c1ef857f "The workaround for MSVC C4702 is still
needed".)
Change-Id: Ia73be9417f2d738b0356e36b978beff019388ae1
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/165073
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <stephan.bergmann@allotropia.de>
|
|
...which appears to be a better fit. Now, gb_JavaClassSet_JAVACDEBUG is
unconditionally defined as -g, but conditionally only used if
gb_target_symbols_enabled is true for the given target (where the target is
spelled "Jar/..." resp. "CustomTarget_jvmfwk/jreproperties" for the
--enable-sybmols=... mini-language in configure.ac). Also, the odd special case
to "force debug information for OOoRunner" has just been dropped.
Change-Id: Ief10efc689d65e2f50888d3e96cb40f43cbb1b2d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/165071
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <stephan.bergmann@allotropia.de>
|
|
Change-Id: I89e0121ba3d63852e39bf4d6141b0dc15b2408f2
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/165057
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <stephan.bergmann@allotropia.de>
|
|
Change-Id: I991b6e908d4139723c2d408d4ba3586676fded18
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164693
Tested-by: Jenkins
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
|
|
...where it recently started to fail for me with
> ======================================================================
> ERROR: test_gbuildtojson (gbuildtojson.CheckGbuildToJsonModules)
> ----------------------------------------------------------------------
> Traceback (most recent call last):
> File "solenv/qa/python/gbuildtojson.py", line 147, in test_gbuildtojson
> jsonfiles = os.listdir(os.path.join(self.tempwork, 'GbuildToJson', 'Library'))
> FileNotFoundError: [Errno 2] No such file or directory: '/tmp/gbuildffqxqxbf/GbuildToJson/Library'
presumably because none of the modules iterated over there happen to put any
files into the GbuildToJson/Library directory (and, by doing so, would create
that directory)
Change-Id: I2622f300d6d9c45d28a98fb6bb82aa684d1d5507
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/165033
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <stephan.bergmann@allotropia.de>
|
|
...which had been added there with ae17922b7d0f797129dd5fbd5514d691ac9ebeaa "add
more libs into --enable-mergelibs=more build". But: For one, I'm not quite
sure why the postgresql-sdbc functionality is split into two libraries
Library_postgresql-sdbc and Library_postgresql-sdbc-impl, but there's supposedly
a reason why the latter should stay in a dynamic library of its own? And for
another, at least my --enable-ldap --enable-mergelibs=more
--without-system-openldap etc. Linux build now started to crash during exit of
CppunitTest_sd_export_tests-ooxml2 with
> Invalid free() / delete / delete[] / realloc()
> at 0x4845B2C: free (vg_replace_malloc.c:985)
> by 0x1C5C11E3: ber_memfree_x (memory.c:152)
> by 0x62895DC6: ldap_int_destroy_global_options (init.c:543)
> by 0x40010F1: _dl_call_fini (dl-call_fini.c:43)
> by 0x400506D: _dl_fini (dl-fini.c:114)
> by 0x4EE7FD5: __run_exit_handlers (exit.c:111)
> by 0x4EE811D: exit (exit.c:141)
> by 0x4ECF150: (below main) (libc_start_call_main.h:74)
> Address 0x103070f1f7f00 is not stack'd, malloc'd or (recently) free'd
apparently because there are two competing at-exit calls to
ldap_int_destroy_global_options in both LO's
workdir/UnpackedTarball/openldap/libraries/libldap/init.c in
instdir/program/libmergedlo.so (where that code ends up now, instead of in a
library of its own that would apparently not happen to be loaded for this test)
and in
/usr/src/debug/openldap-2.6.6-1.fc39.x86_64/openldap-2.6.6/libraries/libldap/init.c
in my (Fedora 39) system's /usr/lib64/libldap.so.2 (which apparently happens to
get loaded into the process due to whatever other dependency).
Change-Id: I5f4b944dfe2e09df2c6d33fc31d4dded4a453c60
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/165025
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <stephan.bergmann@allotropia.de>
|
|
Change-Id: I2ab1cd2d0be81b15b4701581093b696aaac2846c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164979
Tested-by: Jenkins
Reviewed-by: Armin Le Grand <Armin.Le.Grand@me.com>
|
|
probably broken since around:
commit 347112fbd89ebd2a244052f14e085c9d3c90838d
Date: Sun Oct 8 13:14:39 2023 +0300
Drop temporary aliases from Tuple2D and Tuple3D
Change-Id: I117e719dd654fb063811750fdf20d353a82b6996
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164934
Tested-by: Caolán McNamara <caolan.mcnamara@collabora.com>
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
|
|
This reverts commit 806b6014db796a3970332f9cefaaabf9355d0d8f.
Reverting for now, this does not work with --enable-mergelib
Change-Id: Ib44f819d0da9ed3769f76fc40b5c15ce097390a4
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164872
Tested-by: René Engelhard <rene@debian.org>
Reviewed-by: René Engelhard <rene@debian.org>
|
|
We only need it inside one library,
and this reduces the symbols we export from libmerged by over 1000.
Change-Id: I8bf1b2479ba61e931dab0a5cfa81484d534c5658
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164846
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
This reverts commit 87d3f768f3ecded1e1392442181edb287aed9e2e.
The unit test at
starmath/CppunitTest_starmath_qa_cppunit.mk
is doing
$(eval $(call gb_CppunitTest_use_library_objects,starmath_qa_cppunit,\
sm \
))
which it is using to access to all the internal functions of starmath.
Unfortunately that linking trick is just not compatible with the mergelib linking.
Change-Id: Ie3c81a0eeea5bed192de18981463497f251531a6
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164730
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: I2b9acf1a897c962a60ec0c955ae6e9c6bea8ff8d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164656
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Which means
(1) We need to extend the weak linkage magic for the sfx2 SFX_TYPEMAP
stuff. Just make it unconditional, since it makes no difference for the
individual items.
(2) The initialisation of global const Sequence data in starmath stops
working, because it runs too early. Use function-local static to
initialise it on-demand
Change-Id: Idc397515cd1d9621a06d237606c19acee600081a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164532
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: Ib916270a32aa6f3c0f2d6a8ec43f99241f74b972
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164426
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: Ia0898c820cfcddf981f414b33a2854c3917edca2
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164424
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: I04e6516c0bd8d4e0c08c93c59f7c5b0aea88b0ea
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164420
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: Iaabb3356ced33186eeb2bd545bebb20918010638
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164419
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: I3f3bba92202e833066b1ef2e9f76ae34698a6b16
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164361
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: Ided4d536d39de64b0c86d26fb11468a1dc7b0fb3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164292
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
|
|
Change-Id: Ib1af1a98993aabb8a03f4ef19d8da4d9a71fdbc0
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164226
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: I1168eb43638c3b18e3b9de96d89646420f80b52e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164218
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: I6df8cd8a11681cac971d53ec52b6dd97c18fd459
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164217
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: Ib3de04a85332430895bb6096a2cfaf1463f33b79
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164216
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: I0d04eb2ab9b6384576194dc2c6421310da227b8c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164215
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Also
(*) fix the definition of ENABLE_MERGELIBS_MORE in configure.ac
(*) Remove the dummy accessible factory in vcl, it creates
more problems than it solves, because code will break when
trying to use it, and then I get crashes far removed from the
source of the problem (failure to find the acc factory).
Change-Id: I969481d5ad2cfd7104d8240fdd0dce9d285fdb61
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164176
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: I5b04b5e4a1023fc4da61bdcf148829f899e501d2
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164152
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: I124b60d91491b4fecc8c564728f4d36c23845a68
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164151
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
...which caused a recent
<https://buildbot.flathub.org/#/builders/33/builds/12708> test build to fail
with
> {
> "errors": [
> "appstream-failed-validation"
> ],
> "warnings": [
> "appstream-screenshot-missing-caption"
> ],
> "appstream": [
> "E: org.libreoffice.LibreOffice.desktop:~: desktop-app-launchable-omitted"
> ],
> "message": "Please consult the documentation at https://docs.flathub.org/docs/for-app-authors/linter"
> }
Change-Id: I3655d3985c70780e5980bec2d4753120fb302f1a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164134
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <stephan.bergmann@allotropia.de>
|
|
This reverts commit e4717342e5bea23f590a42d565a09ace600e2ca5. cppumaker no
longer generates *_embind.cxx files since
b2ade3e63e34556cad8157e25f8c75d29e79537d "Extract embindmaker from cppumaker
into its own tool ...that directly generates one large .cxx."
Change-Id: Ic4f60a502f3e592489cf0cee86878c327bbb4894
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164055
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <stephan.bergmann@allotropia.de>
|