bootandy/archived-dust
1
0
Fork 0
mirror of https://github.com/bootandy/dust.git synced 2026-06-08 11:29:05 +03:00
Code Issues 21 Packages Projects Releases 10 Wiki Activity

[PR #425] [MERGED] fix: 64-bit atomics for platforms with no 64-bit atomics #478

New Issue
Closed
opened 2026-06-08 11:28:39 +03:00 by zhus · 0 comments
zhus commented 2026-06-08 11:28:39 +03:00
Owner
Copy Link
Copy Source

📋 Pull Request Information

Original PR: https://github.com/bootandy/dust/pull/425
Author: @NoisyCoil
Created: 8/6/2024
Status: ✅ Merged
Merged: 8/9/2024
Merged by: @bootandy

Base: master ← Head: portable-64-bit-atomics

📝 Commits (1)

  • b37593c fix: 64-bit atomics for platforms with no 64-bit atomics

📊 Changes

2 files changed (+9 additions, -1 deletions)

View changed files

📝 Cargo.toml (+3 -0)
📝 src/progress.rs (+6 -1)

📄 Description

AtomicU64 cannot be used on platforms without 64-bit atomics, such as armel. This PR adds a dependency on the portable-atomic crate for those platforms, using the type of the same name defined in it. It introduces no changes for all other platforms (in particular, those on which dust currently builds are unchanged). I tested that this patch cross-builds to armel, and that all tests pass in an emulated armel environment.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/bootandy/dust/pull/425 **Author:** [@NoisyCoil](https://github.com/NoisyCoil) **Created:** 8/6/2024 **Status:** ✅ Merged **Merged:** 8/9/2024 **Merged by:** [@bootandy](https://github.com/bootandy) **Base:** `master` ← **Head:** `portable-64-bit-atomics` --- ### 📝 Commits (1) - [`b37593c`](https://github.com/bootandy/dust/commit/b37593c67657b37b98b712582d62f257a907628a) fix: 64-bit atomics for platforms with no 64-bit atomics ### 📊 Changes **2 files changed** (+9 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `Cargo.toml` (+3 -0) 📝 `src/progress.rs` (+6 -1) </details> ### 📄 Description `AtomicU64` cannot be used on platforms without 64-bit atomics, such as armel. This PR adds a dependency on the `portable-atomic` crate for those platforms, using the type of the same name defined in it. It introduces no changes for all other platforms (in particular, those on which dust currently builds are unchanged). I tested that this patch cross-builds to armel, and that all tests pass in an emulated armel environment. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
zhus added the pull-request label 2026-06-08 11:28:39 +03:00
zhus closed this issue 2026-06-08 11:28:40 +03:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
good first issue
help wanted
pull-request
Mirrored from GitHub Pull Request
 windows help wanted
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
zhus (Sergey Zhukov)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: bootandy/archived-dust#478
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?