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

fix(Makefile): set the default REF to Nemu #528

Merged
merged 1 commit into from
Dec 29, 2024
Merged

Conversation

poemonsense
Copy link
Member

This commits add the default REF variable with value of Nemu.

We intended to record the REF_HOME at compile-time with the REF_HOME macro passed as a C++ flag. However, this detection is wrapped in the ifneq ($(REF),) checker and fails to capture NEMU_HOME if the user does not explicitly specify REF=Nemu.

We also remove the default REF_PROXY variable in config.h.

This commits add the default REF variable with value of Nemu.

We intended to record the REF_HOME at compile-time with the REF_HOME
macro passed as a C++ flag. However, this detection is wrapped in
the `ifneq ($(REF),)` checker and fails to capture NEMU_HOME if the
user does not explicitly specify `REF=Nemu`.

We also remove the default REF_PROXY variable in config.h.
@poemonsense poemonsense requested a review from klin02 December 29, 2024 12:05
@poemonsense poemonsense merged commit 8a4a73e into master Dec 29, 2024
5 checks passed
@poemonsense poemonsense deleted the fix-default-ref-home branch December 29, 2024 13:03
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