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

allow compilation on stable Rust #7

Merged
merged 5 commits into from
Feb 22, 2024
Merged

allow compilation on stable Rust #7

merged 5 commits into from
Feb 22, 2024

Conversation

jounathaen
Copy link
Member

My goal is to use the VirtAddr and PhysAddr in other crates which are stable. With this PR this should be possible:

[target.'cfg(target_arch = "aarch64")'.dependencies]
aarch64 = { path = "../../rust-aarch64" , default-features = false}

Not 100% sure if this is all correct, as it is difficult to test without an aarch64 machine.

@jounathaen
Copy link
Member Author

Oh, this is BTW heavily inspired by https://github.com/rust-osdev/x86_64

mkroening and others added 5 commits February 21, 2024 14:39
Co-authored-by: Jonathan Klimt <jonathan.klimt@eonerc.rwth-aachen.de>
Signed-off-by: Martin Kröning <martin.kroening@eonerc.rwth-aachen.de>
Co-authored-by: Jonathan Klimt <jonathan.klimt@eonerc.rwth-aachen.de>
Signed-off-by: Martin Kröning <martin.kroening@eonerc.rwth-aachen.de>
Co-authored-by: Jonathan Klimt <jonathan.klimt@eonerc.rwth-aachen.de>
Signed-off-by: Martin Kröning <martin.kroening@eonerc.rwth-aachen.de>
Co-authored-by: Jonathan Klimt <jonathan.klimt@eonerc.rwth-aachen.de>
Signed-off-by: Martin Kröning <martin.kroening@eonerc.rwth-aachen.de>
Co-authored-by: Jonathan Klimt <jonathan.klimt@eonerc.rwth-aachen.de>
Signed-off-by: Martin Kröning <martin.kroening@eonerc.rwth-aachen.de>
@mkroening mkroening changed the title add feature gate for nightly and no_std allow compilation on stable Rust Feb 22, 2024
@mkroening mkroening merged commit 786d41b into master Feb 22, 2024
3 checks passed
@mkroening mkroening deleted the stable_support branch February 22, 2024 12:49
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