Skip to content

Commit

Permalink
Project ID change to rDNS scheme to comply with AppStream
Browse files Browse the repository at this point in the history
  • Loading branch information
0tkl committed Jun 2, 2024
1 parent 80491ba commit e52e89d
Show file tree
Hide file tree
Showing 3 changed files with 33 additions and 15 deletions.
8 changes: 4 additions & 4 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -222,12 +222,12 @@ jobs:
meson install -C build --destdir=../appimage/appdir
cd appimage
sudo apt-get install libfuse2
curl -L "https://github.com/linuxdeploy/linuxdeploy/releases/download/1-alpha-20220822-1/linuxdeploy-x86_64.AppImage" -o linuxdeploy
curl -L "https://github.com/AppImage/AppImageKit/releases/download/13/appimagetool-x86_64.AppImage" -o appimagetool
sudo apt-get install appstream libfuse2
curl -L "https://github.com/linuxdeploy/linuxdeploy/releases/download/continuous/linuxdeploy-x86_64.AppImage" -o linuxdeploy
curl -L "https://github.com/AppImage/AppImageKit/releases/download/continuous/appimagetool-x86_64.AppImage" -o appimagetool
chmod +x linuxdeploy appimagetool
./linuxdeploy --appdir appdir --desktop-file=appdir/aegisub.desktop
./linuxdeploy --appdir appdir --desktop-file=appdir/org.aegisub.aegisub.desktop
./appimagetool appdir
- name: Upload artifacts - Linux AppImage
Expand Down
34 changes: 26 additions & 8 deletions packages/desktop/aegisub.appdata.xml.in.in
Original file line number Diff line number Diff line change
@@ -1,10 +1,13 @@
<?xml version="1.0" encoding="UTF-8"?>
<component type="desktop">
<id>aegisub.desktop</id>
<metadata_license>CC0-1.0</metadata_license>
<project_license>BSD-3-Clause AND MIT AND MPL-1.1</project_license>
<component type="desktop-application">
<id>org.aegisub.aegisub</id>

<name>Aegisub</name>
<summary>A free, cross-platform open source tool for creating and modifying subtitles</summary>

<metadata_license>CC0-1.0</metadata_license>
<project_license>BSD-3-Clause AND MIT AND MPL-1.1</project_license>

<description>
<p>Aegisub is a free, cross-platform open source tool for creating and modifying subtitles. Aegisub makes it quick and easy to time subtitles to audio, and features many powerful tools for styling them, including a built-in real-time video preview.</p>
<p>Aegisub was originally created as a tool to make typesetting, particularly in anime fansubs, a less painful experience. At the time of the start of the project, many other programs that supported the Advanced Substation Alpha format lacked (and in many cases, still lack; development on several competing programs have since been dropped for various reasons completely unrelated to Aegisub) many vital functions, or were too buggy and/or unreliable to be really useful.</p>
Expand All @@ -19,14 +22,15 @@
<li>Fully scriptable through the Automation module</li>
</ul>
</description>
<!-- XXX: appstreamcli validation warning: cid-desktopapp-is-not-rdns
If improving this, the <id> and filename should probably also be changed. -->
<launchable type="desktop-id">aegisub.desktop</launchable>

<launchable type="desktop-id">org.aegisub.aegisub.desktop</launchable>

<kudos>
<kudo>HiDpiIcon</kudo>
<kudo>HighContrast</kudo>
<kudo>UserDocs</kudo>
</kudos>

<screenshots>
<screenshot type="default">
<caption>Typesetting</caption>
Expand All @@ -41,19 +45,33 @@
<image>https://aegisub.org/img/screenshots/unix/audio-timing.png</image>
</screenshot>
</screenshots>
<developer_name>Aegisub Group</developer_name>

<developer id="org.aegisub">
<name>Aegisub Group</name>
</developer>

<url type="bugtracker">https://github.com/Aegisub/Aegisub/issues</url>
<url type="faq">https://aegisub.org/docs/latest/faq</url>
<url type="help">https://aegisub.org/docs/latest</url>
<url type="homepage">https://aegisub.org</url>
<url type="translate">https://github.com/Aegisub/Aegisub</url>

<content_rating type="oars-1.0">
<content_attribute id="social-info">mild</content_attribute>
</content_rating>

<translation type="gettext">aegisub</translation>

<categories>
<category>AudioVideo</category>
<category>AudioVideoEditing</category>
<category>GTK</category>
</categories>

<provides>
<binary>aegisub</binary>
</provides>

<releases>
<!-- TODO: automatic replace at config time -->
<release version="3.2.2" date="2014-12-08"/>
Expand Down
6 changes: 3 additions & 3 deletions packages/meson.build
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ else

i18n = import('i18n')
i18n.merge_file(input: desktop_template,
output: 'aegisub.desktop',
output: 'org.aegisub.aegisub.desktop',
type: 'desktop',
po_dir: '../po',
install: true,
Expand All @@ -34,7 +34,7 @@ else
output: 'aegisub.desktop.appdata.xml.in',
configuration: conf_pkg)
i18n.merge_file(input: appdata_template,
output: 'aegisub.appdata.xml',
output: 'org.aegisub.aegisub.appdata.xml',
type: 'xml',
po_dir: '../po',
install: true,
Expand All @@ -52,6 +52,6 @@ else
if get_option('build_appimage')
install_symlink('AppRun', install_dir: '/', pointing_to: bindir.strip('/') / 'aegisub')
install_symlink('.DirIcon', install_dir: '/', pointing_to: datadir.strip('/') / 'icons' / 'hicolor' / 'scalable' / 'apps' / 'aegisub.svg')
install_symlink('aegisub.desktop', install_dir: '/', pointing_to: datadir.strip('/') / 'applications' / 'aegisub.desktop')
install_symlink('org.aegisub.aegisub.desktop', install_dir: '/', pointing_to: datadir.strip('/') / 'applications' / 'org.aegisub.aegisub.desktop')
endif
endif

2 comments on commit e52e89d

@sosie-js
Copy link

@sosie-js sosie-js commented on e52e89d Jun 10, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello 0tkl

Seems appstream dependant version issue, no?
When I installed flatpak following https://flatpak.org/setup/Ubuntu
i did not noticed it installed appstream from background
apt info appstream gave

Package: appstream
Version: 0.15.2-2flatpak1
Priority: optional
Section: admin
Maintainer: Matthias Klumpp <[email protected]>
Installed-Size: 2 262 kB
Depends: shared-mime-info, libappstream4 (>= 0.15.0), libc6 (>= 2.34), libglib2.0-0 (>= 2.62)
Suggests: apt-config-icons
Download-Size: 406 kB
APT-Manual-Installed: yes
APT-Sources: https://ppa.launchpadcontent.net/flatpak/stable/ubuntu jammy/main amd64 Packages
Description: Software component metadata management

but I ran into the problem rebuilding the AppImage , I used your version but it gave more warnings lol

appimagetool, continuous build (commit 5735cc5), build <local dev build> built on 2023-03-08 22:52:04 UTC
Using architecture x86_64

Ubuntu.AppImage/squashfs-root should be packaged as AegisubPlus.AppImage
AppStream upstream metadata found in usr/share/metainfo/aegisub.appdata.xml
Trying to validate AppStream information with the appstreamcli tool
In case of issues, please refer to https://github.com/ximion/appstream
aegisub.appdata.xml
  I: org.aegisub.aegisub:28: **nonstandard-gnome-extension** kudos
  I: org.aegisub.aegisub:49: **unknown-tag developer**
  W: org.aegisub.aegisub:~: metainfo-filename-cid-mismatch
  W: org.aegisub.aegisub:~: desktop-file-not-found

✘ Validation échouée : alertes : 2, informations : 2
run_external: subprocess exited with status 3Failed to validate AppStream information with appstreamcli

Cheers,
SoSie

@sosie-js
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is even a metainfo validator

Please sign in to comment.