Skip to content

Add org.kde.kwave - #5993

Closed
mrpenner wants to merge 7 commits into
flathub:new-prfrom
mrpenner:kwave-submission
Closed

Add org.kde.kwave#5993
mrpenner wants to merge 7 commits into
flathub:new-prfrom
mrpenner:kwave-submission

Conversation

@mrpenner

@mrpenner mrpenner commented Jan 5, 2025

Copy link
Copy Markdown

Please confirm your submission meets all the criteria

@mrpenner

mrpenner commented Jan 5, 2025

Copy link
Copy Markdown
Author

This is currently building from a git commit, so it should either wait until the next release (in April) or only go to beta. I wanted to get this started so I could make any needed changes and make sure it's ready for merging at the next stable release, when I can switch to building from the released archive. The reason I'm not using the current stable release is because https://invent.kde.org/multimedia/kwave/-/merge_requests/32 is needed to use the file portal.

@mrpenner

mrpenner commented Jan 5, 2025

Copy link
Copy Markdown
Author

There was a previous effort: #2848 but it seems to have stalled

@hfiguiere

Copy link
Copy Markdown
Contributor

cc @flathub/kde

@Justinzobel

Copy link
Copy Markdown
Member

@milohr would be best to test this as the primary architect behind the Maui Project.

@mrpenner

mrpenner commented Jan 6, 2025

Copy link
Copy Markdown
Author

@milohr would be best to test this as the primary architect behind the Maui Project.

Kwave is not related to the Maui Project. Were you thinking of Vvave?

@Justinzobel

Copy link
Copy Markdown
Member

Yeah that's the one, so many audio apps! Oops!

@bbhtt

bbhtt commented Jan 6, 2025

Copy link
Copy Markdown
Contributor

This is currently building from a git commit, so it should either wait until the next release (in April) or only go to beta.

Better to wait as there is no automatic transition for users from beta repo to stable.

Comment thread org.kde.kwave.json Outdated
@bbhtt

bbhtt commented Jan 6, 2025

Copy link
Copy Markdown
Contributor

bot, build org.kde.kwave

@flathubbot

Copy link
Copy Markdown

Queued test build for org.kde.kwave.

@bbhtt bbhtt added the awaiting-changes Pull request waiting for inputs or changes from author label Jan 6, 2025
@flathubbot

Copy link
Copy Markdown

Started test build 172237

@flathubbot

Copy link
Copy Markdown

Build 172237 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/155355/org.kde.kwave.flatpakref

Comment thread org.kde.kwave.json Outdated
@mrpenner

mrpenner commented Jan 8, 2025

Copy link
Copy Markdown
Author

bot, build org.kde.kwave

@flathubbot

Copy link
Copy Markdown

Queued test build for org.kde.kwave.

@flathubbot

Copy link
Copy Markdown

Started test build 172673

@flathubbot

Copy link
Copy Markdown

Build 172673 failed

@mrpenner

mrpenner commented Jan 8, 2025

Copy link
Copy Markdown
Author

bot, build org.kde.kwave

@flathubbot

Copy link
Copy Markdown

Queued test build for org.kde.kwave.

@flathubbot

Copy link
Copy Markdown

Started test build 172680

@flathubbot

Copy link
Copy Markdown

Build 172680 failed

@mrpenner

mrpenner commented Jan 8, 2025

Copy link
Copy Markdown
Author

I was sourcing id3lib from https://github.com/thekhalifa/id3lib. I noticed the Kwave flatpak wasn't saving ID3 tags properly, so I switched to the released version of id3lib from SourceForge plus some patches from Debian. My local build now saves the ID3 tags properly.

I see the bot build failed to build id3lib on aarch64. Shall I just disable aarch64 builds?

I know it's not great to use such an old library that needs so many patches. I would like to port Kwave to TagLib, but I'm not sure when I can get that done.

@Justinzobel

Copy link
Copy Markdown
Member

I have no preference one way or the other.

On the one hand it would be nice to have Kwave in Flathub so we keep the KDE Flatpak train going.

On the other hand, old libraries + patches = maintenance cost, so waiting for the work on supporting TagLib would be good.

@jbruechert

jbruechert commented Jan 8, 2025

Copy link
Copy Markdown

I think you may need to run autoreconf -fi before building id3lib to fix the aarch64 build.

I guess aarch64 didn't really exist in 2003

@flathubbot

Copy link
Copy Markdown

Queued test build for org.kde.kwave.

@flathubbot

Copy link
Copy Markdown

Started test build 181178

@flathubbot

Copy link
Copy Markdown

Build 181178 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/164339/org.kde.kwave.flatpakref

@mrpenner

Copy link
Copy Markdown
Author

bot, build org.kde.kwave

@flathubbot

Copy link
Copy Markdown

Queued test build for org.kde.kwave.

@flathubbot

Copy link
Copy Markdown

Started test build 193152

@flathubbot

Copy link
Copy Markdown

Build 193152 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/176443/org.kde.kwave.flatpakref

@mrpenner

Copy link
Copy Markdown
Author

I updated to the release candidate (v25.03.90) archive and tested the build, and it seems to work fine. 25.04 release is April 17, is there anything I need to do before then? Just update the archive URL when 25.04 is released?

@bbhtt

bbhtt commented Apr 12, 2025

Copy link
Copy Markdown
Contributor

Just update the archive URL when 25.04 is released?

Yes

@flathubbot

Copy link
Copy Markdown

🚧 Test build enqueued.

@flathubbot

Copy link
Copy Markdown

🚧 Started test build.

@flathubbot

Copy link
Copy Markdown

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

flatpak install --user https://dl.flathub.org/build-repo/178526/org.kde.kwave.flatpakref

@mrpenner

Copy link
Copy Markdown
Author

AFAIK this is good to go. The test build works fine for me.

@bbhtt

bbhtt commented Apr 18, 2025

Copy link
Copy Markdown
Contributor

Seems fine, cc @flathub/kde for a final review?

@tsdgeos

tsdgeos commented Apr 18, 2025

Copy link
Copy Markdown

Oh my sadness so many patches.

@tsdgeos

tsdgeos commented Apr 18, 2025

Copy link
Copy Markdown

Can we please get anitya (or any other kind of x-checker-data) so we get automatic MRs when there's new versions of the tarballs released?

You can get inspired by the okular ones https://github.com/flathub/org.kde.okular/blob/master/org.kde.okular.json

@flathubbot

Copy link
Copy Markdown

🚧 Test build enqueued.

@flathubbot

Copy link
Copy Markdown

🚧 Started test build.

@mrpenner

Copy link
Copy Markdown
Author

Can we please get anitya (or any other kind of x-checker-data)

I added anitya for KWave. For the dependencies, id3lib isn't going to get any more releases and libaudiofile is also looking that way. (And I share your sadness that so many patches are needed. Maybe someday I can get KWave ported to taglib or something.)

I see there's a shared module for libmad, should I be using that?

@flathubbot

Copy link
Copy Markdown

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

flatpak install --user https://dl.flathub.org/build-repo/178929/org.kde.kwave.flatpakref

@tsdgeos

tsdgeos commented Apr 19, 2025

Copy link
Copy Markdown

The problem with the shared module is that we don't get updates to it, so i guess unless the compilation for it is super complicated (which seems not) i would not do it.

Looks reasnoable to me now :)

@bbhtt bbhtt added ready Pull request ready for final review and merge and removed awaiting-changes Pull request waiting for inputs or changes from author labels Apr 19, 2025
@bbhtt

bbhtt commented Apr 19, 2025

Copy link
Copy Markdown
Contributor

/merge

@flathubbot

Copy link
Copy Markdown

A repository for this submission has been created: https://github.com/flathub/org.kde.kwave and it will be published to Flathub in 4-5 hours.

You will receive an invite to be a collaborator to the above repository. Please make sure to enable 2FA on GitHub and accept the invite within one week.

Please go through the App maintenance guide if you have never maintained an app on Flathub before.

If you're the original developer (or an authorized party), please verify your app to let users know it's coming from you.

Please follow the Flathub blog for the latest announcements.

Thanks!

@flathubbot flathubbot closed this Apr 19, 2025
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Apr 19, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

ready Pull request ready for final review and merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants