Skip to content

ci: Build Linux in container for wider glibc support#16

Merged
timfish merged 3 commits into
mainfrom
timfish/fix/wider-glibc-support
Jul 29, 2025
Merged

ci: Build Linux in container for wider glibc support#16
timfish merged 3 commits into
mainfrom
timfish/fix/wider-glibc-support

Conversation

@timfish
Copy link
Copy Markdown
Collaborator

@timfish timfish commented Jul 28, 2025

This PR uses an Ubuntu 20.04 docker image from here to build the native module for Linux. This ensures a wider glibc version support and allows the module to be used on older versions of Debian, etc.

The image is also updated to use g++ v10 because Node v24 requires c++20.

@timfish timfish marked this pull request as ready for review July 28, 2025 21:42
@timfish timfish requested a review from AbhiPrasad July 28, 2025 21:42
Comment thread .github/workflows/build.yml Outdated
@timfish timfish merged commit 321b7d6 into main Jul 29, 2025
63 of 64 checks passed
@timfish timfish deleted the timfish/fix/wider-glibc-support branch July 29, 2025 13:55
mydea added a commit to getsentry/sentry-javascript that referenced this pull request May 8, 2026
…4.0 (#20720)

Bumps this from 2.2.0 to 2.4.0, including the following changes:

## v2.4.0

* feat: Add Node 26 support
(getsentry/sentry-javascript-profiling-node-binaries#32)

## v2.3.0

* fix: Memory leaks, integer truncation, UB, and double-stop
(getsentry/sentry-javascript-profiling-node-binaries#26)
* fix: Integer arithmetic for timestamps, emit elapsed_since_start_ns as
string
(getsentry/sentry-javascript-profiling-node-binaries#28)
* fix: Dead member, capacity checks, insert perf, napi_env by value
(getsentry/sentry-javascript-profiling-node-binaries#27)
* chore: Replace execSync with execFileSync to prevent command injection
(getsentry/sentry-javascript-profiling-node-binaries#23)
* chore: Pin GitHub Actions to full-length commit SHAs
(getsentry/sentry-javascript-profiling-node-binaries#24)
* ci: Build Linux in container for wider glibc support
(getsentry/sentry-javascript-profiling-node-binaries#16)
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.

3 participants