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

Replace nproc in macOS setup script #11186

Closed

Conversation

czentgr
Copy link
Collaborator

@czentgr czentgr commented Oct 7, 2024

nproc may not be found on macOS causing boost not to install. Instead, switch to use existing NPROC to determine the number of logical CPU that nproc would have returned.

@czentgr czentgr requested a review from majetideepak October 7, 2024 14:53
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Oct 7, 2024
@czentgr
Copy link
Collaborator Author

czentgr commented Oct 7, 2024

@majetideepak FYI.

Copy link

netlify bot commented Oct 7, 2024

Deploy Preview for meta-velox canceled.

Name Link
🔨 Latest commit 41c838d
🔍 Latest deploy log https://app.netlify.com/sites/meta-velox/deploys/67044b6da479050008894046

@czentgr czentgr force-pushed the cz_replace_macos_nproc branch from fec7122 to b3cc441 Compare October 7, 2024 15:32
@majetideepak majetideepak added the ready-to-merge PR that have been reviewed and are ready for merging. PRs with this tag notify the Velox Meta oncall label Oct 7, 2024
@czentgr czentgr marked this pull request as ready for review October 7, 2024 16:37
@facebook-github-bot
Copy link
Contributor

@xiaoxmeng has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

nproc may not be found on macOS causing boost not to install.
Instead, switch to use sysctl to determine the number of
logical CPU that nproc would have returned.
@czentgr czentgr force-pushed the cz_replace_macos_nproc branch from b3cc441 to 41c838d Compare October 7, 2024 20:58
@facebook-github-bot
Copy link
Contributor

@xiaoxmeng merged this pull request in b63622d.

Copy link

Conbench analyzed the 1 benchmark run on commit b63622d5.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged ready-to-merge PR that have been reviewed and are ready for merging. PRs with this tag notify the Velox Meta oncall
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants