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

8.0 Fix PXB-3210: PXB fails to build on macOS since 8.0.33-28 due to FIND_PRO… #1532

Merged
merged 1 commit into from
Dec 22, 2023

Conversation

aybek
Copy link
Contributor

@aybek aybek commented Dec 21, 2023

…CPS()

https://jira.percona.com/browse/PXB-3210

When predicting required memory for prepare (PXB-2710) sys/sysctl is used by macOS and proc/sysinfo by other systems for memory calculation. As part of the PXB-2993 fix to make the memory calculation logic compatible with procps-4 procps.cmake file was introduced to detect the version during cmake, but cmake macro is not accounting for macOS and and failing to find procps on macOS. Now we make sure that cmake does not search for procps on macOS.

…CPS()

https://jira.percona.com/browse/PXB-3210

When predicting required memory for prepare (PXB-2710) `sys/sysctl` is used by macOS and `proc/sysinfo` by other systems for memory calculation.
As part of the PXB-2993 fix to make the memory calculation logic compatible with procps-4 procps.cmake file was introduced to detect the version during cmake,
but cmake macro is not accounting for macOS and and failing to find procps on macOS.
Now we make sure that cmake does not search for procps on macOS.
@aybek aybek requested a review from satya-bodapati December 21, 2023 06:03
@aybek aybek changed the title PXB-3210: PXB fails to build on macOS since 8.0.33-28 due to FIND_PRO… 8.0 Fix PXB-3210: PXB fails to build on macOS since 8.0.33-28 due to FIND_PRO… Dec 21, 2023
@aybek aybek requested a review from satya-bodapati December 22, 2023 04:36
Copy link
Contributor

@satya-bodapati satya-bodapati left a comment

Choose a reason for hiding this comment

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

LGTM!

@aybek aybek merged commit 23fcf12 into percona:8.0 Dec 22, 2023
2 checks passed
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.

2 participants