-
Notifications
You must be signed in to change notification settings - Fork 2k
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/native: Remove macOS as native target #18735
Conversation
Makefile.include
Outdated
# If native, TOOLCHAIN for OSX is llvm | ||
ifeq ($(BOARD),native) | ||
ifeq ($(OS),Darwin) | ||
TOOLCHAIN ?= llvm | ||
endif | ||
endif |
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.
Maybe keep this, but add an error message instead of setting the toolchain instead to mitigate some confused forum questions?
ifeq ($(BOARD),native)
ifeq ($(OS),Darwin)
$(shell $(COLOR_ECHO) "$(COLOR_RED)"Buildin on macOS is not supported."\
"We recommend vagrant for building:$(COLOR_RESET)"\
"https://github.com/RIOT-OS/RIOT/blob/master/dist/tools/vagrant/README.md 1>&2)
endif
endif
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.
I implemented your suggestion. Still builds on my linux machine. On macOS, the error is printed but it still tries to build the application (which then fails ofc).
See e.g #14631 for the state of that. |
ea49113
to
c187f87
Compare
Makefile.include
Outdated
ifeq ($(BOARD),native) | ||
ifeq ($(OS),Darwin) | ||
TOOLCHAIN ?= llvm | ||
$(shell $(COLOR_ECHO) "$(COLOR_RED)"Buildin on macOS is not supported."\ | ||
"We recommend vagrant for building:$(COLOR_RESET)"\ |
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.
No mixing tabs and spaces ;-P
"We recommend vagrant for building:$(COLOR_RESET)"\ | |
"We recommend vagrant for building:$(COLOR_RESET)"\ |
c187f87
to
35a06c4
Compare
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.
ACK. Murdock should point out any issues.
Murdock results✔️ PASSED 35a06c4 boards/native: Remove macOS as native target
ArtifactsThis only reflects a subset of all builds from https://ci-prod.riot-os.org. Please refer to https://ci.riot-os.org for a complete build for now. |
Hi! 👋
Since building for macOS native is broken for a loooong time now and fixing it is far from trivial;
This removes building for macOS native.
It still builds for Linux on my machine. Please double check that and can somebody test the freebsd target?
How and where would one add an info message for macOS users who try to build native? Do we even want that?