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: slightly cleanup for Mac Intel; replace './main -h' with echo. #335

Merged
merged 1 commit into from
Mar 21, 2023

Conversation

mqy
Copy link
Contributor

@mqy mqy commented Mar 20, 2023

Just several minor cleanup.

  1. Mac (Intel) related:
    • $(UNAME_M) shows "x86-64".
    • shell sysctl -n hw.optional.arm64 outputs an error that should be ignored.
    • Add additional comment on -framework Accelerate.
  2. It's quite verbose to print help message per make, let's just echo a tip.

Build on macOS 13.2.1 (22D68), with Apple clang version 14.0.0 (clang-1400.0.29.202).

@mqy mqy changed the title Makefile: slightly cleanup for Mac Intel; replace 'run ./main -h' with echo. Makefile: slightly cleanup for Mac Intel; replace './main -h' with echo. Mar 20, 2023
@gjmulder gjmulder added enhancement New feature or request build Compilation issues labels Mar 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Compilation issues enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants