-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
AppImage build seems corrupted #6624
Comments
Could you please post your snap configuration for electron-builder? We upgraded to enable |
No change with the compression option added. My directories:
output: dist
buildResources: build
appId: com.github.mscharley.app
files:
- filter:
- '**/*'
- '!**/node_modules/*/{CHANGELOG.md,README.md,README,readme.md,readme}'
- '!**/node_modules/*/{test,__tests__,tests,powered-test,example,examples}'
- '!**/node_modules/*.d.ts'
- '!**/node_modules/.bin'
- '!**/*.{iml,o,hprof,orig,pyc,pyo,rbc,swp,csproj,sln,xproj}'
- '!.editorconfig'
- '!**/._*'
- '!**/{.DS_Store,.git,.hg,.svn,CVS,RCS,SCCS,.gitignore,.gitattributes}'
- '!**/{__pycache__,thumbs.db,.flowconfig,.idea,.vs,.nyc_output}'
- '!**/{appveyor.yml,.travis.yml,circle.yml}'
- '!**/{npm-debug.log,yarn.lock,.yarn-integrity,.yarn-metadata.json}'
linux:
target: AppImage
category: Utility
snap:
compression: "xz"
publish:
- github
electronVersion: 17.0.0 |
Whoops. Maybe |
I tried under both |
Still an issue with the final release of electron-builder 23.0.0 when building against either electron 17.0.1 or 16.0.9. |
Adding this to the top level gets a working build as it triggers xz encryption:
But the resulting build is prohibitively slow to run. Neither of the other two valid top level values ( |
I've tried hacking around in node_modules a bit, but I can't seem to find a valid else case for this, which seems to be the root cause: https://github.com/electron-userland/electron-builder/blob/master/packages/app-builder-lib/src/targets/AppImageTarget.ts#L90-L92 Either I try
Or I try
The current version which specifies no encryption option generates the broken build. |
I'm wondering if compression is no longer optional param. Could you give this patch a try using patch-package? That should unblock you for the interim
|
@mmaietta using that patch:
See my comment above where I already tried various values for the compression option. It looks like |
Closing as duplicate of #6678 to consolidate convos |
@mmaietta I think you forgot to hit close on that comment. |
I'm using the alpha version because I was running into #6563 .
This seems to be an issue with electron 17.x, if I roll back to 16.0.8 then it builds and runs fine. I'm building on Arch Linux targeting AppImage. The build runs fine, but when I try to run it I get the following output:
The text was updated successfully, but these errors were encountered: