boards: set native variant depending on the host's CPU architecture#21291
Closed
OlegHahm wants to merge 7 commits intoRIOT-OS:masterfrom
Closed
boards: set native variant depending on the host's CPU architecture#21291OlegHahm wants to merge 7 commits intoRIOT-OS:masterfrom
OlegHahm wants to merge 7 commits intoRIOT-OS:masterfrom
Conversation
OlegHahm
requested review from
MichelRottleuthner,
MrKevinWeiss,
aabadie,
jia200x,
kaspar030,
leandrolanzieri and
miri64
as code owners
github-actions
bot
added
Platform: native
OlegHahm
added
the
CI: ready for build
OlegHahm
force-pushed
the
pr/native_host_arch_dependent
branch
from
495d7a8 to
e91ca21
Compare
Moving the common code from the former boards/native into a common directory.
OlegHahm
force-pushed
the
pr/native_host_arch_dependent
branch
from
75ed8e4 to
c20e9a5
Compare
Reflect the fact that the 64 bit version of native is the new default now.
Contributor
|
Oh, I've been working on #21242 with a more generic alias approach. Not fully ready yet, sorry for not tagging you there. |
Contributor
|
But I do like the idea of moving common |
Member
Author
|
Oh, I missed that one indeed. What are we gonna do? Do you want to try to cherry-pick the native-common stuff and I'm gonna close this one? |
Contributor
Yes, sounds good, I'll give it a try. |
4 tasks
Contributor
|
Integrated in and discontinued in favor of #21242 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Contribution description
This patch renames the current native board into native32 and creates an de facto alias via the Makesystem dependent on the host system's architecture (kudos to @benpicco).
The changeset is mostly renaming directories and replacing native64 with native32 within application Makefiles (unless native64) was not supported.
It's mostly untested so far. Let's see what the CI comes up with.
Testing procedure
Use your favorite RIOT application on native.
Issues/PRs references
Alternativ variant to #21100