Skip to content

Conversation

@viceice
Copy link
Contributor

@viceice viceice commented Jul 2, 2024

This will downgrade the used GLIBC to v2.31, so it's again compatibe with ubuntu 20.04.

@viceice viceice requested review from a team and JoshuaWise as code owners July 2, 2024 10:35
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.

Excellent, thank you!

I looked into using buster instead of bullseye, but LTS for buster ended on June 30, 2024.

A gotcha that might bite us in the future due to this: GHA tests may pass with recent Ubuntu docker images, but running with prebuilds may have some spurious errors due to some GCC change between v2.31 and current. Mentioning @JoshuaWise for visibility.

Also: we should consider replacing the linux prebuild step with something akin to the prebuild-linux-arm step that uses docker (so we can get arbitrary glibc prebuilds for amd64 as well).

@mceachen mceachen merged commit 50ace3b into WiseLibs:master Jul 2, 2024
@rathboma
Copy link
Contributor

rathboma commented Jul 2, 2024

Omg, arbitrary glibc prebuilds would be fantastic.

2.28 seems very old now, but in particular the Ubuntu Extended Security Maintenance program (which is free for individuals) supports Ubuntu 18.04 until 2028, so a significant number of folks are going to be on 18.04 for a while.

Seeing as using an older version of glibc isn't a problem (just using a newer one), this would be fantastic.

@mceachen Would you be open to a PR that set up an environment with glibc 2.28+ as the compile target using the prebuild step you suggest above?

@mceachen
Copy link
Member

mceachen commented Jul 3, 2024

@rathboma if the test suite can validate the prebuild works (so we'd use a docker image just like what this PR does), I don't see the downside to supporting Ubuntu 18 until it's really EOL (unless somehow it breaks newer more common builds, but I don't see that happening).

@viceice viceice deleted the patch-1 branch July 3, 2024 06:15
patricebender referenced this pull request in cap-js/cds-dbs Jul 3, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [better-sqlite3](https://togithub.com/WiseLibs/better-sqlite3) |
[`11.1.1` ->
`11.1.2`](https://renovatebot.com/diffs/npm/better-sqlite3/11.1.1/11.1.2)
|
[![age](https://developer.mend.io/api/mc/badges/age/npm/better-sqlite3/11.1.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/better-sqlite3/11.1.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/better-sqlite3/11.1.1/11.1.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/better-sqlite3/11.1.1/11.1.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>WiseLibs/better-sqlite3 (better-sqlite3)</summary>

###
[`v11.1.2`](https://togithub.com/WiseLibs/better-sqlite3/releases/tag/v11.1.2)

[Compare
Source](https://togithub.com/WiseLibs/better-sqlite3/compare/v11.1.1...v11.1.2)

#### What's Changed

- Use node bullseye to build arm images by
[@&#8203;viceice](https://togithub.com/viceice) in
[https://github.com/WiseLibs/better-sqlite3/pull/1216](https://togithub.com/WiseLibs/better-sqlite3/pull/1216)

#### New Contributors

- [@&#8203;viceice](https://togithub.com/viceice) made their first
contribution in
[https://github.com/WiseLibs/better-sqlite3/pull/1216](https://togithub.com/WiseLibs/better-sqlite3/pull/1216)

**Full Changelog**:
WiseLibs/better-sqlite3@v11.1.1...v11.1.2

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/cap-js/cds-dbs).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy40MjEuOSIsInVwZGF0ZWRJblZlciI6IjM3LjQyMS45IiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6W119-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
ArGeoph pushed a commit to ArGeoph/better-sqlite3 that referenced this pull request Mar 21, 2025
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.

3 participants