-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Remove mimalloc workaround #4606
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This unfortunately is not true.
The new mimalloc-sys crate actually breaks everything.
It also seems it has been broken for a longer time already.
So, this is something we need to look into.
The binaries do work within an Alpine environment, but not as a true static binary anymore.
Are we waiting for a new crate release or should I override it with commit purpleprotocol/mimalloc_rust@992c9da? |
I have not yet checked if the removal works for all arch's. |
70d0223
to
f40f3ba
Compare
- Updated all crates including Diesel and the new mysqlclient-sys - Updated the MSRV to v1.78 as that is what Diesel mandates - Added the mimalloc crate as a patch for now to fix armv6 static builds This probably makes dani-garcia#4606 possible
- Updated all crates including Diesel and the new mysqlclient-sys - Updated the MSRV to v1.78 as that is what Diesel mandates - Added the mimalloc crate as a patch for now to fix armv6 static builds This probably makes dani-garcia#4606 possible
- Updated all crates including Diesel and the new mysqlclient-sys - Updated the MSRV to v1.78 as that is what Diesel mandates - Added the mimalloc crate as a patch for now to fix armv6 static builds This probably makes dani-garcia#4606 possible - Updated web-vault to v2024.5.1 - Updated GitHub Actions Fixed an issue with the localhost images for extracting the musl binaries.
- Updated all crates including Diesel and the new mysqlclient-sys - Updated the MSRV to v1.78 as that is what Diesel mandates - Added the mimalloc crate as a patch for now to fix armv6 static builds This probably makes #4606 possible - Updated web-vault to v2024.5.1 - Updated GitHub Actions Fixed an issue with the localhost images for extracting the musl binaries.
f40f3ba
to
d73c993
Compare
- libatomic linking for armv6 has been fixed in purpleprotocol/mimalloc_rust@992c9da
d73c993
to
9e880cc
Compare
For reference: purpleprotocol/mimalloc_rust@446fc23