Add io.gitlab.metadatacleaner.metadatacleaner #7323

segfault3 · 2025-12-15T00:11:28Z

Please confirm your submission meets all the criteria

Please describe the application briefly. Metadata Cleaner is a GTK app which uses mat2 to clean metadata from files. It was available on Flathub before but was removed after the original author stopped maintaining it.
Please attach a video showcasing the application on Linux using the Flatpak. https://github.com/user-attachments/assets/753ac7cd-6552-405e-b0fa-324806d838cf
The Flatpak ID follows all the rules listed in the Application ID requirements.
I have read and followed all the Submission requirements and the Submission guide and I agree to them.
I am a developer to the project.

github-actions · 2025-12-15T02:19:24Z

Starting a test build of the submission. Please fix any
issues reported in the build log. You can restart the build
once the issue is fixed by commenting the phrase below.

bot, build

flathubbot · 2025-12-15T02:19:27Z

🚧 Test build enqueued.

flathubbot · 2025-12-15T02:19:39Z

🚧 Started test build.

flathubbot · 2025-12-15T02:20:37Z

❌ Test build failed.

Help

bot, build - Restart the test build

segfault3 · 2025-12-15T11:11:05Z

Error: 'appid-url-not-reachable' error found in linter manifest check. Details: Tried https://gitlab.com/MetadataCleaner/MetadataCleaner | Status: 301 | Body: You are being redirected.

I've already seen that error locally when using the linter, but I was hoping that it's not a blocker that the app ID uses uppercase. The URL https://gitlab.com/MetadataCleaner/MetadataCleaner redirects to https://gitlab.com/metadatacleaner/metadatacleaner which I have control over.

hfiguiere · 2025-12-15T13:34:31Z

it's a blocker. make it match. all lowercase is 100% fine.

segfault3 · 2025-12-15T17:40:22Z

bot, build

flathubbot · 2025-12-15T17:40:26Z

🚧 Test build enqueued.

flathubbot · 2025-12-15T17:40:37Z

🚧 Started test build.

flathubbot · 2025-12-15T17:44:03Z

❌ Test build failed.

Help

bot, build - Restart the test build

segfault3 · 2025-12-15T17:55:41Z

bot, build

flathubbot · 2025-12-15T17:55:45Z

🚧 Test build enqueued.

flathubbot · 2025-12-15T17:55:54Z

🚧 Started test build.

flathubbot · 2025-12-15T18:06:28Z

❌ Test build failed.

Help

bot, build - Restart the test build

Signed-off-by: segfault <[email protected]>

segfault3 · 2025-12-15T18:43:57Z

bot, build

flathubbot · 2025-12-15T18:44:01Z

🚧 Test build enqueued.

flathubbot · 2025-12-15T18:44:11Z

🚧 Started test build.

flathubbot · 2025-12-15T19:02:12Z

✅ Test build succeeded. To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/236897/io.gitlab.metadatacleaner.metadatacleaner.flatpakref

Built for aarch64 and x86_64 architectures.

⚠️ Linter warnings:

Warnings can be promoted to errors in the future. Please try to resolve them.

'appstream-screenshot-missing-caption' warning found in linter repo check. Details: One or more screenshots are missing captions in the Metainfo file

io.gitlab.metadatacleaner.metadatacleaner.yaml

hfiguiere · 2025-12-15T20:13:51Z

io.gitlab.metadatacleaner.metadatacleaner.yaml

+                sources:
+                  - type: git
+                    url: https://github.com/Perl/perl5.git
+                    tag: v5.40.3


use 5.42.0 which is the same one that is in the SDK.

if it's in the SDK, do we even have to install it as a module? I tried to remove the module but then the installation of exiftool fails:

cp exiftool blib/script/exiftool "/usr/bin/perl5.42.0" -MExtUtils::MY -e 'MY->fixin(shift)' -- blib/script/exiftool Manifying 1 pod document Manifying 34 pod documents Manifying 34 pod documents Manifying 32 pod documents Manifying 33 pod documents Manifying 33 pod documents Manifying 32 pod documents Manifying 9 pod documents !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! ERROR: Can't create '/usr/bin' Do not have write permissions on '/usr/bin' !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! at -e line 1. make: *** [Makefile:1627: pure_site_install] Error 13 Error: module exiftool: Child process exited with code 2

I don't understand why installing Perl as a module avoids that error 🤔

if it's in the SDK, do we even have to install it as a module?

It did NOT say to not add it, just to use the same version.

It's in the SDK but not in the platform.

oh, the Perl module was configured with -Dprefix=$FLATPAK_DEST

now exiftool won't work. I never said to remove Perl from here. I just said to build 5.42.0

It's in the SDK but not in the platform.

ah, got it. I'll add it again.

io.gitlab.metadatacleaner.metadatacleaner.yaml

segfault3 · 2025-12-15T23:03:43Z

bot, build

flathubbot · 2025-12-15T23:03:48Z

🚧 Test build enqueued.

flathubbot · 2025-12-15T23:03:59Z

🚧 Started test build.

flathubbot · 2025-12-15T23:13:39Z

✅ Test build succeeded. To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/236947/io.gitlab.metadatacleaner.metadatacleaner.flatpakref

Built for aarch64 and x86_64 architectures.

⚠️ Linter warnings:

Warnings can be promoted to errors in the future. Please try to resolve them.

'appstream-screenshot-missing-caption' warning found in linter repo check. Details: One or more screenshots are missing captions in the Metainfo file

hfiguiere

#7323 (comment)

segfault3 · 2025-12-15T23:17:57Z

bot, build

flathubbot · 2025-12-15T23:18:01Z

🚧 Test build enqueued.

flathubbot · 2025-12-15T23:18:11Z

🚧 Started test build.

flathubbot · 2025-12-15T23:21:04Z

❌ Test build failed.

Help

bot, build - Restart the test build

segfault3 · 2025-12-15T23:23:22Z

bot, build

flathubbot · 2025-12-15T23:23:25Z

🚧 Test build enqueued.

flathubbot · 2025-12-15T23:23:35Z

🚧 Started test build.

flathubbot · 2025-12-15T23:37:39Z

✅ Test build succeeded. To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/236953/io.gitlab.metadatacleaner.metadatacleaner.flatpakref

Built for aarch64 and x86_64 architectures.

⚠️ Linter warnings:

Warnings can be promoted to errors in the future. Please try to resolve them.

'appstream-screenshot-missing-caption' warning found in linter repo check. Details: One or more screenshots are missing captions in the Metainfo file

bbhtt · 2025-12-16T03:51:24Z

It was available on Flathub before but was removed after the original author stopped maintaining it.

So this is a fork? If so you have to use a different icon and name. And what changes was made in the fork?

segfault3 · 2025-12-16T07:05:17Z

So this is a fork?

Yes

If so you have to use a different icon and name.

I got the permission of the original author (via email) to use the original icon and name.

And what changes was made in the fork?

The only relevant changes (beside the ones to the Flatpak manifst which resulted from the review of this PR) are:

Change Flatpak app ID to remove the domain of the original author (as he requested).
Update Flatpak runtime and module sources to the latest releases

bbhtt · 2025-12-16T09:01:11Z

So not really a fork.

segfault3 · 2025-12-16T10:10:17Z

Depends on your definition of fork I guess. I don't have access to the original git repository, I'm maintaining the forked repo https://gitlab.com/metadatacleaner/metadatacleaner.

segfault3 · 2025-12-21T22:24:34Z

What's the status here? Is there a blocker? Can I do anything to move things forward?

bbhtt · 2025-12-22T13:40:07Z

I don't see a reason for a new submission with so minimal changes from the original project.

segfault3 · 2025-12-22T16:24:02Z

I don't understand. The original project is unmaintained and not available on Flathub anymore. Do you prefer that we add the original, unmaintained project again? Or that it's not available on Flathub at all?

bbhtt · 2025-12-23T17:53:35Z

Bottom line is, you can't submit a new app with the same name, icon and no code related changes in your fork as a new one.

If that means the app is not available then it's ok.

github-actions bot added the awaiting-review Pull requests waiting for review label Dec 15, 2025

segfault3 force-pushed the metadatacleaner-submission branch 2 times, most recently from 53d1948 to c4f335d Compare December 15, 2025 17:39

segfault3 changed the title ~~Add io.gitlab.MetadataCleaner.MetadataCleaner~~ Add io.gitlab.metadatacleaner.metadatacleaner Dec 15, 2025

segfault3 force-pushed the metadatacleaner-submission branch from c4f335d to a949a4f Compare December 15, 2025 17:55

Add io.gitlab.metadatacleaner.metadatacleaner

cc6a91d

Signed-off-by: segfault <[email protected]>

segfault3 force-pushed the metadatacleaner-submission branch from a949a4f to cc6a91d Compare December 15, 2025 18:43

hfiguiere reviewed Dec 15, 2025

View reviewed changes

io.gitlab.metadatacleaner.metadatacleaner.yaml Outdated Show resolved Hide resolved

hfiguiere reviewed Dec 15, 2025

View reviewed changes

io.gitlab.metadatacleaner.metadatacleaner.yaml Outdated Show resolved Hide resolved

hfiguiere reviewed Dec 15, 2025

View reviewed changes

io.gitlab.metadatacleaner.metadatacleaner.yaml Outdated Show resolved Hide resolved

hfiguiere reviewed Dec 15, 2025

View reviewed changes

io.gitlab.metadatacleaner.metadatacleaner.yaml Outdated Show resolved Hide resolved

fixup

6ef948e

segfault3 requested a review from hfiguiere December 15, 2025 23:13

hfiguiere requested changes Dec 15, 2025

View reviewed changes

update perl

6a8b6a2

segfault3 force-pushed the metadatacleaner-submission branch from 115e571 to 6a8b6a2 Compare December 15, 2025 23:16

segfault3 requested a review from hfiguiere December 15, 2025 23:16

fixup

15795f2

This comment was marked as outdated.

Sign in to view

github-actions bot added pr-check-blocked Pull requests marked as blocked by automation and removed pr-check-blocked Pull requests marked as blocked by automation labels Dec 24, 2025

Add io.gitlab.metadatacleaner.metadatacleaner #7323

Are you sure you want to change the base?

Add io.gitlab.metadatacleaner.metadatacleaner #7323

Uh oh!

Conversation

segfault3 commented Dec 15, 2025

Please confirm your submission meets all the criteria

Uh oh!

github-actions bot commented Dec 15, 2025

Uh oh!

flathubbot commented Dec 15, 2025

Uh oh!

flathubbot commented Dec 15, 2025

Uh oh!

flathubbot commented Dec 15, 2025

Uh oh!

segfault3 commented Dec 15, 2025

Uh oh!

hfiguiere commented Dec 15, 2025

Uh oh!

segfault3 commented Dec 15, 2025

Uh oh!

flathubbot commented Dec 15, 2025

Uh oh!

flathubbot commented Dec 15, 2025

Uh oh!

flathubbot commented Dec 15, 2025

Uh oh!

segfault3 commented Dec 15, 2025

Uh oh!

flathubbot commented Dec 15, 2025

Uh oh!

flathubbot commented Dec 15, 2025

Uh oh!

flathubbot commented Dec 15, 2025

Uh oh!

segfault3 commented Dec 15, 2025

Uh oh!

flathubbot commented Dec 15, 2025

Uh oh!

flathubbot commented Dec 15, 2025

Uh oh!

flathubbot commented Dec 15, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

hfiguiere Dec 15, 2025

Choose a reason for hiding this comment

Uh oh!

segfault3 Dec 15, 2025

Choose a reason for hiding this comment

Uh oh!

hfiguiere Dec 15, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

segfault3 Dec 15, 2025

Choose a reason for hiding this comment

Uh oh!

hfiguiere Dec 15, 2025

Choose a reason for hiding this comment

Uh oh!

segfault3 Dec 15, 2025

Choose a reason for hiding this comment

Uh oh!

segfault3 Dec 15, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

segfault3 commented Dec 15, 2025

Uh oh!

flathubbot commented Dec 15, 2025

Uh oh!

flathubbot commented Dec 15, 2025

Uh oh!

flathubbot commented Dec 15, 2025

Uh oh!

hfiguiere left a comment

Choose a reason for hiding this comment

hfiguiere Dec 15, 2025 •

edited

Loading