Skip to content

Conversation

@Timeless0911
Copy link
Contributor

@Timeless0911 Timeless0911 commented May 14, 2025

Summary

In v0.7.0, we bump @ast-grep/napi to v0.38 and due to breaking changes in @ast-grep/napi, Ubuntu 20 is not supported now if user enable dts generation.

This is a bit too radical. This PR pin to a locked version to ensure support for Ubuntu 20.

image

Related Links

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@netlify
Copy link

netlify bot commented May 14, 2025

Deploy Preview for rslib ready!

Name Link
🔨 Latest commit 8a0330b
🔍 Latest deploy log https://app.netlify.com/sites/rslib/deploys/68248403a2fc9a0008399711
😎 Deploy Preview https://deploy-preview-992--rslib.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@Timeless0911 Timeless0911 enabled auto-merge (squash) May 14, 2025 11:56
@Timeless0911
Copy link
Contributor Author

@HerringtonDarkholme Can we keep support for ubuntu 20? 🫡

@Timeless0911 Timeless0911 merged commit ea1e11d into main May 14, 2025
16 checks passed
@Timeless0911 Timeless0911 deleted the chore/ast branch May 14, 2025 11:59
@HerringtonDarkholme
Copy link
Contributor

unfortunately ast-grep is built by github action runner. GH recently deprecated ubuntu 20.

@Timeless0911
Copy link
Contributor Author

@HerringtonDarkholme
Copy link
Contributor

Is rspack or rslib depending on glibc? If so, I can borrow your solution to ast-grep. Otherwise, the best solution now is to pin the version at 0.37.0 for a while, and later bumping to a newer version.

@Timeless0911
Copy link
Contributor Author

Cloudide in our company are still using Ubuntu 20 yet. And a glibc error will be throwed when the binding is loaded. And I'm still not clear about the situation of other docker images.
The same error in netlify with ubuntu 20 but we can update docker in Netlify.

@Timeless0911
Copy link
Contributor Author

img_v3_02m8_1420d220-2ff8-4a0f-99de-74a9741db36g

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.

4 participants