Skip to content

Rework build and prepare for FW 21#11

Merged
averne merged 1 commit intoaverne:masterfrom
ViRb3:master
Feb 23, 2026
Merged

Rework build and prepare for FW 21#11
averne merged 1 commit intoaverne:masterfrom
ViRb3:master

Conversation

@ViRb3
Copy link
Contributor

@ViRb3 ViRb3 commented Feb 23, 2026

This PR supersedes #10.

Changes and notes:

  • Fully dockerized build with two parts:
    1. base image with static dependencies, which is cached and reused for faster builds
    2. SwitchWave source is mounted into the above and built there, allowing you to do incremental builds and debugging the environment
  • gimp-bcn-conver.sh reworked:
    • GIMP 2 and 3 dual support with auto detection. Both verified and working via the dual Dockerfile setup.
    • No more swallowing of errors
    • Serialize with lock file due to race conditions which I hit twice in development
  • GitHub Actions auto-build integration:
  • mpv submodule updated with a critical fix that causes playback crash with latest devkitpro: Fix crash using latest devkitpro mpv#1

I tested my build extensively, and can confirm the following works: video, audio, subtitles, network shares, local playback. If you want me to test anything else, please let me know!

This PR is in draft state since we need to merge the mpv PR first, then I can update to reference your submodule instead of my fork.

@ViRb3 ViRb3 mentioned this pull request Feb 23, 2026
@ViRb3
Copy link
Contributor Author

ViRb3 commented Feb 23, 2026

Switched to your mpv submodule. Ready for review.

@ViRb3 ViRb3 marked this pull request as ready for review February 23, 2026 17:55
@averne
Copy link
Owner

averne commented Feb 23, 2026

Great work, thanks a lot.
It's probably possible to work around this race condition in the gimp script by passing a template to mktemp. I'll see if I can make that work this evening.

@averne averne merged commit fce9278 into averne:master Feb 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants