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 support for electron v35 prebuilds #1340

Merged
merged 1 commit into from
Mar 4, 2025

Conversation

m4heshd
Copy link
Contributor

@m4heshd m4heshd commented Mar 4, 2025

Stable electron v35 just released. Prebuilds were tested here.

@m4heshd m4heshd requested review from JoshuaWise and a team as code owners March 4, 2025 18:01
@m4heshd
Copy link
Contributor Author

m4heshd commented Mar 4, 2025

Ubuntu 20.04 runner is being phased out. They're failing the jobs during the deprecation period and today is apparently one of those days. 😄

@mceachen
Copy link
Member

mceachen commented Mar 4, 2025

Ubuntu 20.04 runner is being phased out. They're failing the jobs during the deprecation period and today is apparently one of those days. 😄

So... Do we drop support for Ubuntu 20? LTS support ends 31 May 2025 -- I think it's reasonable.

The alternative is to build the glibc artifacts with ubuntu 20 in a docker container, which is a PITA.

Copy link
Member

@mceachen mceachen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@mceachen mceachen merged commit 6de572a into WiseLibs:master Mar 4, 2025
21 checks passed
@mceachen
Copy link
Member

mceachen commented Mar 4, 2025

@m4heshd if you can make a PR to the build.yml to replace ubuntu-20.04 with ubuntu-22.04 I'll approve it.

(Ubuntu 22 LTS EOL's on April 2027: see https://wiki.ubuntu.com/Releases)

@m4heshd
Copy link
Contributor Author

m4heshd commented Mar 5, 2025

@mceachen Done. #1341

ArGeoph pushed a commit to ArGeoph/better-sqlite3 that referenced this pull request Mar 21, 2025

Verified

This commit was signed with the committer’s verified signature.
targos Michaël Zasso
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants