Skip to content

fix(ci): AppImage meebouwen in de Linux-release - #5

Merged
DutchSailor merged 1 commit into
OpenAEC-Foundation:mainfrom
Nozzit:ci/linux-appimage
Jul 22, 2026
Merged

fix(ci): AppImage meebouwen in de Linux-release#5
DutchSailor merged 1 commit into
OpenAEC-Foundation:mainfrom
Nozzit:ci/linux-appimage

Conversation

@Nozzit

@Nozzit Nozzit commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Probleem

De release bevat op dit moment .deb, .rpm, .msi, -setup.exe en .dmg, maar geen .AppImage. In .github/workflows/build.yml staat bij de ubuntu-22.04-regel van de build-matrix args: --bundles deb,rpm, wat appimage uitsluit — ook al staat bundle.targets in src-tauri/tauri.conf.json op "all".

Dat is inconsistent met de rest van hetzelfde workflow-bestand:

  • de artifact-upload verwijst al naar bundle/appimage/*.AppImage;
  • de publish-stap doet al find artifacts -name '*.AppImage';
  • de release-notitie belooft de gebruiker al **Linux:** \.AppImage` / `.deb` / `.rpm``.

Die AppImage werd dus nooit gebouwd, waardoor de belofte in de release-notitie niet klopt.

Waarom AppImage nodig is

De OpenAEC Installer kan op Linux alleen tools installeren die een .AppImage in hun GitHub-release publiceren: een AppImage is een enkel bestand dat zonder root naar de home-map van de gebruiker geschreven en later weer verwijderd kan worden. Een .deb vereist root en pakketbeheer, wat buiten het bereik van de installer valt. Zonder AppImage is Open Calc Studio niet via de installer beschikbaar op Linux.

Wijziging

Eén regel in .github/workflows/build.yml:

-            args: --bundles deb,rpm
+            args: --bundles appimage,deb,rpm

.deb en .rpm blijven ongewijzigd meegebouwd. Verder is er niets aangepast.

🤖 Generated with Claude Code

De build-matrix beperkte de Linux-bundels tot deb,rpm, waardoor er nooit
een AppImage werd gebouwd. De upload- en publish-stappen verwijzen wel al
naar *.AppImage en de release-notitie belooft `.AppImage` als download.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@Nozzit
Nozzit requested a review from DutchSailor as a code owner July 20, 2026 12:58
@DutchSailor
DutchSailor merged commit b9c0a89 into OpenAEC-Foundation:main Jul 22, 2026
3 checks passed
DutchSailor added a commit that referenced this pull request Jul 22, 2026
PR #5 zet appimage terecht aan in de build-matrix, maar dat pad is nooit
uitgevoerd: de build-job draait alleen op een release-tag, dus geen enkele
PR bouwt een installer. Een fout zou daardoor pas midden in een release
opduiken en de hele Linux-oplevering kosten.

Het bekende struikelblok is FUSE. Het AppImage-gereedschap (linuxdeploy) is
zelf een AppImage en heeft FUSE 2 nodig, terwijl Ubuntu 22.04 alleen fuse3
levert. Twee onafhankelijke maatregelen, want of de bundler dit zelf al
opvangt viel hier niet te verifieren:

- libfuse2 toegevoegd aan de Linux-dependencies
- APPIMAGE_EXTRACT_AND_RUN=1, zodat het gereedschap zichzelf uitpakt in
  plaats van via FUSE te mounten

Beide zijn onschadelijk als de bundler het al regelde.
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