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 NEWLIB_MULTILIB_NAMES in Makefile #1588

Merged
merged 1 commit into from
Oct 22, 2024

Conversation

MetaLambda
Copy link
Contributor

The NEWLIB_MULTILIB_NAMES generated in makefile for MULTILIB_GEN may contain space at the end, which cause the generate_target_board script to crash.

@MetaLambda
Copy link
Contributor Author

see #1513

@cmuellner
Copy link
Collaborator

So it looks like we generate useless space characters and fix this later.
Could we do something on the other end to prevent the useless space characters from being generated?

@MetaLambda
Copy link
Contributor Author

ok, I'll try a fix in Makefile.in

@MetaLambda
Copy link
Contributor Author

@cmuellner I modify the Makefile.in to eliminate the useless space at the end of NEWLIB_MULTILIB_NAMES

see riscv-collab#1513
The NEWLIB_MULTILIB_NAMES generated in makefile for MULTILIB_GEN may contain space at the end, which cause the generate_target_board script to crash

Signed-off-by: Bin XIE <[email protected]>
@MetaLambda MetaLambda changed the title fix arch_abi_list in scripts/generate_target_board fix NEWLIB_MULTILIB_NAMES in Makefile Oct 22, 2024
Copy link
Collaborator

@cmuellner cmuellner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@cmuellner cmuellner merged commit 935b263 into riscv-collab:master Oct 22, 2024
26 checks passed
@cmuellner
Copy link
Collaborator

Merged after the CI showed that all pipelines were fine.

Thanks!

@TommyMurphyTM1234
Copy link
Collaborator

Thanks a lot @MetaLambda! 👍

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