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

Makefile: replace echo with printf for compatibility #277

Closed
wants to merge 1 commit into from
Closed

Makefile: replace echo with printf for compatibility #277

wants to merge 1 commit into from

Conversation

kadiwa4
Copy link
Contributor

@kadiwa4 kadiwa4 commented Nov 20, 2022

I don't know why but echo -n didn't work for me. macOS supports the -n option but it doesn't work in the makefile. What it does instead is it literally writes -n to the file and then also leaves a newline at the end.

I'm using GNU Make 4.4 (gmake from Homebrew).

@avpatel
Copy link
Collaborator

avpatel commented Nov 21, 2022

We have mailing list based patch review so it would be great if you can send these patchs to OpenSBI mailing list.

You need to join OpenSBI mailing list using following link:
http://lists.infradead.org/mailman/listinfo/opensbi

Make sure you use "git send-email" to send the patches.

Regards,
Anup

I don't know why but `echo -n` didn't work for me. macOS supports the `-n` option but it doesn't work in the makefile. What it does instead is it literally writes `-n` to the file and then also
leaves a newline at the end.

I'm using GNU Make 4.4 (`gmake` from Homebrew).

Signed-off-by: KaDiWa <kalle.wachsmuth@gmail.com>
@kadiwa4 kadiwa4 closed this Dec 4, 2022
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.

None yet

2 participants