Skip to content

Update README to clarify make arch-test usage #583

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

Merged
merged 1 commit into from
Mar 4, 2025

Conversation

vestata
Copy link
Contributor

@vestata vestata commented Mar 3, 2025

Running make arch-test without explicitly setting ENABLE_ARCH_TEST=1 may fail due to missing prebuilt binaries. This issue arises because the default value of ENABLE_ARCH_TEST is 0, stopping Sail model from being fetched.

Additionally, if ENABLE_SYSTEM=1 was previously set, the build system fetches the kernel image instead of the Sail model. To ensure the correct prebuilt binary is fetched, it is recommended to run make distclean beforehand.

This serves as a temporary solution to #575.

Running make arch-test without explicitly setting ENABLE_ARCH_TEST=1
may fail due to missing prebuilt binaries. This issue arises because
the default value of ENABLE_ARCH_TEST is 0, stopping Sail model from
being fetched.

Additionally, if ENABLE_SYSTEM=1 was previously set, the build system
fetches the kernel image instead of the Sail model. To ensure the
correct prebuilt binary is fetched, it is recommended to run make
distclean beforehand.

This serves as a temporary solution to sysprog21#575.
Copy link

Code Review Agent Run Status

  • Limitations and other issues: ❌ Failure - The AI Code Review Agent skipped reviewing this change because it is configured to exclude certain pull requests based on the source/target branch or the pull request status. You can change the settings here, or contact the agent instance creator at [email protected].

@jserv jserv requested review from ChinYikMing and vacantron and removed request for ChinYikMing March 3, 2025 15:01
@jserv jserv merged commit 2ba9097 into sysprog21:master Mar 4, 2025
8 checks passed
@jserv
Copy link
Contributor

jserv commented Mar 4, 2025

Thank @vestata for contributing!

@vestata vestata deleted the fix-arch-test branch March 4, 2025 06:29
@jserv jserv added this to the release-2025.1 milestone Mar 6, 2025
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.

3 participants