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

boards: make BOARD=native an alias for BOARD=native32 #21242

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

mguetschow
Copy link
Contributor

@mguetschow mguetschow commented Feb 24, 2025

Contribution description

(still WIP, using CI for testing)

TODO

  • make native an alias for native32 or native64 depending on the host architecture (see boards: rename native64 -> native #21100 (comment))
  • somehow (re-)add documentation for BOARD=native
  • add build system test for alias
  • think about implications of using native (or any alias, for that matter) in BOARD_WHITELIST et al.

Testing procedure

make BOARD=native and make BOARD=native32 should produce the same output for now

Issues/PRs references

Alternative to #21100

@github-actions github-actions bot added Platform: native Platform: This PR/issue effects the native platform Area: doc Area: Documentation Area: tests Area: tests and testing framework Area: core Area: RIOT kernel. Handle PRs marked with this with care! Area: build system Area: Build system Area: CI Area: Continuous Integration of RIOT components Area: tools Area: Supplementary tools Area: boards Area: Board ports Area: Kconfig Area: Kconfig integration labels Feb 24, 2025
@mguetschow mguetschow added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR CI: full build disable CI build filter CI: no fast fail don't abort PR build after first error labels Feb 24, 2025
@github-actions github-actions bot added Area: pkg Area: External package ports Area: sys Area: System labels Feb 24, 2025
@riot-ci
Copy link

riot-ci commented Feb 24, 2025

Murdock results

✔️ PASSED

c0f3555 boards/doc.txt: document native board alias for native32

Success Failures Total Runtime
148411 0 148411 01h:39m:11s

Artifacts

@github-actions github-actions bot added the Area: examples Area: Example Applications label Feb 24, 2025
@mguetschow
Copy link
Contributor Author

Hum, somehow this PR wrongly changes the output of make info-boards-supported causing a bunch of "missing periph" build failures.

Evidence: make -C tests/core/thread_float info-boards-supported | grep gba_cartridge returns nothing on master, but does so with this PR.

I can't tell why though.

@mguetschow
Copy link
Contributor Author

I can't tell why though.

I can now. It's because I need to use override in board_alias.inc.mk to be able to alter the BOARD variable if passed from the command line. override can only be overwritten by another override, so adding that in info-global.inc.mk.

Should work now. Not sure if this is the right fix though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: boards Area: Board ports Area: build system Area: Build system Area: CI Area: Continuous Integration of RIOT components Area: core Area: RIOT kernel. Handle PRs marked with this with care! Area: doc Area: Documentation Area: examples Area: Example Applications Area: Kconfig Area: Kconfig integration Area: pkg Area: External package ports Area: sys Area: System Area: tests Area: tests and testing framework Area: tools Area: Supplementary tools CI: full build disable CI build filter CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: native Platform: This PR/issue effects the native platform
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants