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

Add deb and rpm builds for distribution #2895

Open
wants to merge 4 commits into
base: v5
Choose a base branch
from

Conversation

pcreech
Copy link

@pcreech pcreech commented Nov 18, 2024

This adds the ability to generate rpm and deb packages as part of linux distribution. It takes advantage of other electron-installer based providers while building an electron app.

Description of the Change

It would be nice to have a native rpm or deb package for installing to a distribution. This is an initial attempt to add this functionality to the build system.

Applicable Issues

Testing

The initial testing was to ensure the app was basically functional post-install on an rpm system (fedora). I still need to verify functionality on a debian based system, but I wanted to also start getting feedback and start conversations around approach and outputs.

Screenshots

@pcreech pcreech requested a review from ebiggz as a code owner November 18, 2024 19:40
@pcreech pcreech changed the base branch from master to v5 November 18, 2024 19:46
This adds the ability to generate rpm and deb packages as part of linux
distribution.  It takes advantage of other electron-installer based
providers while building an electron app.
@dennisrijsdijk dennisrijsdijk added the Blocked Pending third party or upstream dependency label Nov 18, 2024
@dennisrijsdijk
Copy link
Collaborator

marking blocked until build issues resolved to prevent accidental early merges.

@dennisrijsdijk
Copy link
Collaborator

So, after doing some more testing, setting the deb build to amd64 instead of x86_64, it looks like AppArmor is not an issue on my Ubuntu 24.04 install when installed through the .deb package. The .tar.gz currently still throws the SUID helper error.

@dennisrijsdijk
Copy link
Collaborator

I've added a contribution PR to your fork that fixes debian builds, adds some icons and consolidates and explicitly defines some settings. All info available at pcreech#1

@dennisrijsdijk
Copy link
Collaborator

Further testing required: does this work with -beta and -nightly.DATE builds?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Blocked Pending third party or upstream dependency
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants