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

rpmalloc-sys: Update rpmalloc submodule to latest upstream branch #24

Merged
merged 1 commit into from
Mar 12, 2025

Conversation

MarijnS95
Copy link
Contributor

@MarijnS95 MarijnS95 commented Jan 13, 2025

Checklist

  • I have read the Contributor Guide
  • I have read and agree to the Code of Conduct
  • I have added a description of my changes and why I'd like them included in the section below

Description of Changes

This submodule was originally pointing to a commit somewhere between the 1.4.3 and 1.4.4 release (at the time this Rust wrapper was last committed to). Note that the upstream main branch is slightly ahead of the latest 1.4.5 release, because it also includes a just-merged fix for using Android API level 29+ compiled with LLVM 17+, which Rust apps would otherwise fail to load at runtime. The 1.4.5 release already contains a fix for ARM64 Windows, which is equally of interest to us.

Finally, also add a rerun-if-changed on the rpmalloc.c file, so that local submodule changes and experimentations actually cause the crate to be recompiled. cc-rs only emits these for the environment variables, but not for .file() arguments.

This submodule was originally pointing to a commit somewhere between
the 1.4.3 and 1.4.4 release (at the time this Rust wrapper was last
committed to).  Note that the upstream `main` branch is slightly ahead
of the latest 1.4.5 release, because it also includes a just-merged fix
for using Android API level 29+ compiled with LLVM 17+, which Rust apps
would otherwise fail to load at runtime.  The 1.4.5 release already
contains a fix for ARM64 Windows, which is equally of interest to us.

Finally, also add a `rerun-if-changed` on the `rpmalloc.c` file, so
that **local** submodule changes and experimentations actually cause the
crate to be recompiled.  `cc-rs` only emits these for the environment
variables, but not for `.file()` arguments.
Copy link
Member

@fu5ha fu5ha left a comment

Choose a reason for hiding this comment

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

Nice, thanks!

@fu5ha fu5ha merged commit 6bf1e80 into EmbarkStudios:main Mar 12, 2025
@fu5ha fu5ha mentioned this pull request Mar 12, 2025
3 tasks
@MarijnS95 MarijnS95 deleted the rpmalloc-sys-up branch March 12, 2025 10:39
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.

2 participants