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

Fix #5531 bump snapcraft base to core22 #5555

Merged
merged 2 commits into from
Jan 4, 2023

Conversation

filfreire
Copy link
Member

@filfreire filfreire commented Dec 19, 2022

changelog(Fixes): Fixed an issue that prevented running Insomnia when installed via Snapcraft / .snap package

Closes #5531

See this comment

For .snap builds we rely on this template, which uses Ubuntu 18 base image that is too old for latest Electron & node-libcurl - causing folks the issue in #5531

The solution was:

  1. Add ability on electron-builder to override the base to recent Ubuntu version (e.g. core22) - done in feat: Add base option for snapcraft electron-userland/electron-builder#7320
  2. Use the updated electron-builder dependency (app-builder-lib) that allows the override to base

Seems to work OK in Ubuntu 22
image

@filfreire filfreire requested a review from a team December 19, 2022 20:10
@filfreire filfreire mentioned this pull request Dec 25, 2022
1 task
@filfreire filfreire force-pushed the fix/issue-5531-ubuntu-snap branch from e086b4b to 41b474c Compare January 4, 2023 10:24
@filfreire filfreire enabled auto-merge (squash) January 4, 2023 10:37
@filfreire filfreire merged commit 856c859 into Kong:develop Jan 4, 2023
@filfreire filfreire deleted the fix/issue-5531-ubuntu-snap branch January 4, 2023 10:40
filfreire added a commit to filfreire/insomnia that referenced this pull request Jan 9, 2023
filfreire added a commit that referenced this pull request Jan 9, 2023
* Revert "Fix INS-2242, add afterPack workaround for electron-builder (#5627)"

This reverts commit 33aa835.

* Revert "Fix #5531 bump snapcraft base to core22 (#5555)"

This reverts commit 856c859.

* Revert "use same prettify mechanic for both editors (#5616)"

This reverts commit 21c7817.
kreosus pushed a commit to kreosus/insomnia that referenced this pull request Jan 10, 2023
* Revert "Fix INS-2242, add afterPack workaround for electron-builder (Kong#5627)"

This reverts commit 33aa835.

* Revert "Fix Kong#5531 bump snapcraft base to core22 (Kong#5555)"

This reverts commit 856c859.

* Revert "use same prettify mechanic for both editors (Kong#5616)"

This reverts commit 21c7817.
filfreire added a commit to jackkav/insomnia that referenced this pull request Jan 11, 2023
filfreire added a commit to jackkav/insomnia that referenced this pull request Jan 11, 2023
filfreire added a commit to jackkav/insomnia that referenced this pull request Jan 12, 2023
filfreire added a commit that referenced this pull request Jan 12, 2023
* use electron-build notarize config

* update electron-builder.config

* Re-add snapcraft fix done in #5555

* update electron-builder

Co-authored-by: Filipe Freire <[email protected]>
pavkout pushed a commit to pavkout/insomnia that referenced this pull request Jan 18, 2023
* Fix Kong#5531 bump snapcraft base to core22

* lint package.json
@filfreire filfreire mentioned this pull request Aug 26, 2024
@filfreire filfreire mentioned this pull request Sep 26, 2024
11 tasks
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.

2022.7.0 snap wont start on ubuntu
4 participants