diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2023-06-08 13:08:14 +0200 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2023-06-08 19:49:28 +0200 |
commit | 48d09aa85e01d881995c660f845b60bd8088e7c0 (patch) | |
tree | 79821907cb5027e7a5595c5ad645352616c5416d /solenv/bin/assemble-flatpak-appdata-step2.sh | |
parent | 7f5e2fc7c1ae7c0c2bec76809e11365b255601d5 (diff) |
No longer append appdata for components to flatpak's metainfo
This had originally been added with
<https://github.com/flathub/org.libreoffice.LibreOffice/commit/945f6caad87658b1df1e8918bd5f64939058ab7f>
"clean up desktop integration", see its part "append the appdata for the
separate apps (Base, Draw, etc) to the appdata XML we export for Flatpak, so
they are visible in software centers etc". But it now started to cause Flathub
builds (like <https://buildbot.flathub.org/#/builders/14/builds/3985>) to fail
its "Check for right id in AppStream xml" step, and reportedly "the problem is
there are multiple components in the metainfo file and I'm not sure it's
allowed"
(<https://matrix.to/#/!RfXaBjokqHAbzZrgHz:matrix.org/$jWhob-a4LEH-e_TXsJwGWNfMYLF1kcWpXV_oW982ZpI?via=matrix.org&via=gnome.org&via=kde.org>).
The solution appears to be to just remove that again
(<https://matrix.to/#/!RfXaBjokqHAbzZrgHz:matrix.org/$Yrzf_ZjJxBgUVs6kuP_10Tneu1t1wKQJCM_SC1vNHC4?via=matrix.org&via=gnome.org&via=kde.org>):
> I think maybe the problem was from the times that GNOME Software was also
> moonlighting as a menu editor
> that without Flatpak providing the metainfo for LibreOffice's different
> desktop launchers, they were invisible in GNOME Software, which meant you
> couldn't add/remove them to menus or the desktop
> but since then GNOME just shows all of the launchers, and menu editing /
> categories mostly died or sank to the bottom of the 1990s ocean
> so maybe this is a non-problem
[...]
> ah ok
> ah so they don't show any more 🤣
So remove the now-unused solenv/bin/assemble-flatpak-appdata-step2.sh and
consequently rename the sole remaining
solenv/bin/assemble-flatpak-appdata-step1.sh to just
solenv/bin/assemble-flatpak-appdata.sh.
Change-Id: I3d3fef502f2114274a856102b77edafdd0ecac83
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/152744
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'solenv/bin/assemble-flatpak-appdata-step2.sh')
-rwxr-xr-x | solenv/bin/assemble-flatpak-appdata-step2.sh | 26 |
1 files changed, 0 insertions, 26 deletions
diff --git a/solenv/bin/assemble-flatpak-appdata-step2.sh b/solenv/bin/assemble-flatpak-appdata-step2.sh deleted file mode 100755 index 4f06e6b19148..000000000000 --- a/solenv/bin/assemble-flatpak-appdata-step2.sh +++ /dev/null @@ -1,26 +0,0 @@ -#! /bin/bash -# -# This file is part of the LibreOffice project. -# -# This Source Code Form is subject to the terms of the Mozilla Public -# License, v. 2.0. If a copy of the MPL was not distributed with this -# file, You can obtain one at http://mozilla.org/MPL/2.0/. -# - -# Part of solenv/bin/assemble-flatpak.sh that is shared with a downstream mechanism of building a -# Flatpak from a Fedora libreoffice.spec file. -# -# Arguments: -# $1 pathname, ending in a slash, of the directory containing the source libreoffice-*.appdata.xml -# files -# $2 pathname, ending in a slash, of the directory containing the target -# org.libreoffice.LibreOffice.appdata.xml file - -set -e - -# append the appdata for the different components -for i in "${1?}"libreoffice-*.appdata.xml -do - sed "1 d; s/<id>libreoffice-/<id>org.libreoffice.LibreOffice./" "$i" \ - >>"${2?}"org.libreoffice.LibreOffice.appdata.xml -done |