Skip to content

Commit

Permalink
Fix app ID and move Flatpak manifest
Browse files Browse the repository at this point in the history
This MR does a few changes that fixes most issues related to Flatpak:

- Change the app ID entirely to `org.gnome.Meld`
- Move the Flatpak manifest to `build-aux` directory - everybody in GNOME does it
Add `.Devel` suffix, just like all the other GNOME apps
Add `git` module that builds git from source, instead of copying a file
  • Loading branch information
TheEvilSkeleton committed Oct 22, 2023
1 parent e059cc1 commit c32a273
Show file tree
Hide file tree
Showing 10 changed files with 26 additions and 15 deletions.
4 changes: 2 additions & 2 deletions .gitlab-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,10 +16,10 @@ flatpak:
extends: .flatpak
stage: build
variables:
MANIFEST_PATH: "data/org.gnome.MeldDevel.json"
MANIFEST_PATH: "build-aux/org.gnome.Meld.Devel.json"
FLATPAK_MODULE: "meld"
RUNTIME_REPO: "https://nightly.gnome.org/gnome-nightly.flatpakrepo"
APP_ID: "org.gnome.MeldDevel"
APP_ID: "org.gnome.Meld.Devel"
BUNDLE: "meld-dev.flatpak"

nightly:
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
{
"app-id": "org.gnome.MeldDevel",
"app-id": "org.gnome.Meld.Devel",
"runtime": "org.gnome.Platform",
"runtime-version": "master",
"sdk": "org.gnome.Sdk",
Expand Down Expand Up @@ -48,6 +48,24 @@
}
]
},
{
"name": "git",
"make-args": [
"NO_TCLTK=1",
"INSTALL_SYMLINKS=1"
],
"make-install-args": [
"NO_TCLTK=1",
"INSTALL_SYMLINKS=1"
],
"sources": [
{
"type": "archive",
"url": "https://mirrors.edge.kernel.org/pub/software/scm/git/git-2.42.0.tar.xz",
"sha256": "3278210e9fd2994b8484dd7e3ddd9ea8b940ef52170cdb606daa94d887c93b0d"
}
]
},
{
"name": "meld",
"buildsystem": "meson",
Expand All @@ -61,13 +79,6 @@
"config-opts": [
"-Dprofile=Devel"
]
},
{
"name": "git",
"buildsystem": "simple",
"build-commands": [
"cp /usr/bin/git ${FLATPAK_DEST}/bin"
]
}
]
}
4 changes: 2 additions & 2 deletions data/meson.build
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,8 @@ endif

appdata_file = i18n.merge_file(
input: configure_file(
input: files('org.gnome.meld.appdata.xml.in.in'),
output: 'org.gnome.meld.appdata.xml.in',
input: files('org.gnome.Meld.appdata.xml.in.in'),
output: 'org.gnome.Meld.appdata.xml.in',
configuration: {'appid': application_id},
),
output: '@[email protected]'.format(application_id),
Expand Down
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
2 changes: 1 addition & 1 deletion meld/resources/meld.gresource.xml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
<gresources>
<gresource prefix="/org/gnome/meld/icons">
<file>scalable/apps/org.gnome.Meld.svg</file>
<file>scalable/apps/org.gnome.MeldDevel.svg</file>
<file>scalable/apps/org.gnome.Meld.Devel.svg</file>
<file preprocess="xml-stripblanks" alias="scalable/actions/vc-pull-symbolic.svg">icons/vc-pull-symbolic.svg</file>
<file preprocess="xml-stripblanks" alias="scalable/actions/vc-push-symbolic.svg">icons/vc-push-symbolic.svg</file>
</gresource>
Expand Down
4 changes: 2 additions & 2 deletions meson.build
Original file line number Diff line number Diff line change
Expand Up @@ -25,9 +25,9 @@ dependency('py3cairo', version: '>= 1.15.0')
profile = get_option('profile')
name_suffix = (profile == 'Devel' ? ' (Development Snapshot)' : '')

resource_base_id = 'org.gnome.meld'
resource_base_id = 'org.gnome.Meld'
application_id_no_profile = 'org.gnome.Meld'
application_id = 'org.gnome.Meld@0@'.format(profile)
application_id = (profile == 'Devel' ? application_id_no_profile + '.' + profile : application_id_no_profile)

prefix = get_option('prefix')

Expand Down

0 comments on commit c32a273

Please sign in to comment.