distro-configs/LibreOfficeFlatpak.conf |    1 +
 solenv/flatpak-manifest.in             |    7 +++++++
 2 files changed, 8 insertions(+)

New commits:
commit 8f72aa782500f456f02f18fa74f6b3b6c9b5e90c
Author:     Stephan Bergmann <[email protected]>
AuthorDate: Mon Jan 15 16:00:53 2024 +0100
Commit:     Xisco Fauli <[email protected]>
CommitDate: Mon Jan 15 21:45:00 2024 +0100

    Flatpak: Work around patched libpng in org.freedesktop.Sdk
    
    see
    
<https://github.com/flathub/org.libreoffice.LibreOffice/pull/268/commits/7c79189f674c7cc88f0ab2790d8d198f4a855182>
    "Work around patched libpng in org.freedesktop.Sdk":
    
    ...which carries
    
<https://gitlab.com/freedesktop-sdk/freedesktop-sdk/-/commit/873b28450476c434a3f6fa86cb1874b11968a479>
    "elements/base/libpng.bst: Add support for animated PNGs", thus handles 
"acTL"
    chunks by itself, thus doesn't call the LibreOffice handle_unknown_chunk
    function (in vcl/source/filter/png/PngImageReader.cxx) for such chunks, thus
    never sets APNGInfo::mbIsApng to true, so causes CppunitTest_vcl_png_test to
    fail with
    
    > PngFilterTest.cxx:382:Assertion
    > Test name: PngFilterTest::testApng
    > assertion failed
    > - Expression: aGraphic.IsAnimated()
    
    (<https://buildbot.flathub.org/#/builders/6/builds/92958>) since
    
<https://git.libreoffice.org/core/+/bf944e33569e4a1d6236a54671b7320cdc6ffaf6%5E%21>
    "tdf#104877 Add basic APNG format support".
    
    The patch appears to originate from
    <https://sourceforge.net/projects/libpng-apng/>, and it might be possible to
    adapt the LibreOffice code to also work with such a patched libpng (whose 
png.h
    defines PNG_APGN_SUPPORTED).  However, for now just use LibreOffice's own
    external/libpng instead of the patched one.
    
    Change-Id: Ib67056d11dfa6456920a18216a3b2bbec45f3662
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/162112
    Tested-by: Jenkins
    Reviewed-by: Stephan Bergmann <[email protected]>
    (cherry picked from commit 223bfa24cdeee2ed45ff8d9ab7bb0b92eb8af587)
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/162107
    Reviewed-by: Xisco Fauli <[email protected]>

diff --git a/distro-configs/LibreOfficeFlatpak.conf 
b/distro-configs/LibreOfficeFlatpak.conf
index 3850269ace0f..911184c337eb 100644
--- a/distro-configs/LibreOfficeFlatpak.conf
+++ b/distro-configs/LibreOfficeFlatpak.conf
@@ -41,6 +41,7 @@
 --without-system-libnumbertext
 --without-system-libodfgen
 --without-system-libpagemaker
+--without-system-libpng
 --without-system-libqxp
 --without-system-librevenge
 --without-system-libstaroffice
diff --git a/solenv/flatpak-manifest.in b/solenv/flatpak-manifest.in
index a8d8b88394a8..74beb39f21b9 100644
--- a/solenv/flatpak-manifest.in
+++ b/solenv/flatpak-manifest.in
@@ -793,6 +793,13 @@
                     "dest": "external/tarballs",
                     "dest-filename": "@ARGON2_TARBALL@"
                 },
+                {
+                    "url": 
"https://dev-www.libreoffice.org/src/@LIBPNG_TARBALL@";,
+                    "sha256": "@LIBPNG_SHA256SUM@",
+                    "type": "file",
+                    "dest": "external/tarballs",
+                    "dest-filename": "@LIBPNG_TARBALL@"
+                },
                 {
                     "url": 
"https://dev-www.libreoffice.org/extern/@OPENSYMBOL_TTF@";,
                     "sha256": "@OPENSYMBOL_SHA256SUM@",

Reply via email to