Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

snapcraft is required in building snap (provider: github) after 24.9.2 #8061

Closed
JellyBrick opened this issue Feb 20, 2024 · 8 comments
Closed

Comments

@JellyBrick
Copy link

JellyBrick commented Feb 20, 2024

  • Electron-Builder Version: 24.13.1, 24.12.0
  • Latest Known Working Electron-Builder Version: 24.9.1
  • Node Version: 20 (GitHub Actions)
  • Electron Version: 28.2.3
  • Electron Type (current, beta, nightly): current
  • Target: Linux x64

The build failed with this error.

 ⨯ snapcraft is not installed, please: sudo snap install snapcraft --classic  

Detailed log: https://github.com/th-ch/youtube-music/actions/runs/7969007984/job/21754132043

We are using pnpm (8.15.3)

@mmaietta
Copy link
Collaborator

Hmmm, we never updated app-builder-bin between those versions and that's what pulls the electron-builder-binaries (such as appimage-12.0.1

I'm not seeing any immediately obvious changes in the diff either
v24.9.1...v24.12.0

Is this issue 100% reproducible for you? It's not appearing in our unit tests either

@JellyBrick
Copy link
Author

JellyBrick commented Mar 2, 2024

still reproducible these issues in 24.13.3

https://github.com/th-ch/youtube-music/actions/runs/8125045551/job/22207198684

danielgarthur added a commit to neanes/neanes that referenced this issue Mar 6, 2024
@JellyBrick
Copy link
Author

JellyBrick commented Apr 11, 2024

  • 24.9.2: Successful
  • 24.9.3: Failed

v24.9.2...v24.9.3

Additional Information

CI fails when snap + AppImage builds are enabled.

@JellyBrick JellyBrick changed the title error=rename appimage/388323485 appimage-12.0.1: file exists (ERR_ELECTRON_BUILDER_CANNOT_EXECUTE) in Ubuntu snapcraft is required in building snap after 24.9.2 Apr 13, 2024
@JellyBrick JellyBrick changed the title snapcraft is required in building snap after 24.9.2 snapcraft is required in building snap (provider: github) after 24.9.2 Apr 13, 2024
@JellyBrick JellyBrick closed this as not planned Won't fix, can't repro, duplicate, stale Apr 13, 2024
@cmdcolin
Copy link

what was the resolution to this? I don't currently want to setup snapcraft but want to publish snap files

@mmaietta
Copy link
Collaborator

How would you bundle a snap distributable but not set up snapcraft? That seems like mutually exclusive

@cmdcolin
Copy link

I am not really sure. I just had a setup that "worked" and was creating and releasing .snap files to github but then it stopped working after a electron-builder version upgrade similar to what @JellyBrick had experienced. my understanding is using snapcraft and such would release to the snap store? i just removed snap builds for now in our project...it's pretty low usage

@JellyBrick
Copy link
Author

what was the resolution to this? I don't currently want to setup snapcraft but want to publish snap files

@mmaietta @cmdcolin

It can't; for versions after 24.9.4, you must install snapcraft before you can bundle snap.

th-ch/youtube-music@d09858c

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

No branches or pull requests

3 participants