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

[CMake] Define MAC_OSX for cmake builds on macOS #1813

Merged
merged 1 commit into from
Aug 27, 2020

Conversation

Fuzzbawls
Copy link
Collaborator

A few places in our code rely on this precompiler definition, most
notably in the datadir and zk-params dir default values. Autotools sets
this automatically, but it is not passed into pivx-config.h as it
isn't necessary for autotools builds.

Instead of modify the autotools configure script to pass this define
into the config file, simply define it in CMakeLists.txt

A few places in our code rely on this precompiler definition, most
notably in the datadir and zk-params dir default values. Autotools sets
this automatically, but it is not passed into `pivx-config.h` as it
isn't necessary for autotools builds.

Instead of modify the autotools configure script to pass this define
into the config file, simply define it in `CMakeLists.txt`
@Fuzzbawls Fuzzbawls added Build System Trivial extremely simple issues labels Aug 20, 2020
@Fuzzbawls Fuzzbawls added this to the 5.0.0 milestone Aug 20, 2020
@Fuzzbawls Fuzzbawls self-assigned this Aug 20, 2020
Copy link

@furszy furszy left a comment

Choose a reason for hiding this comment

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

utACK bd3f39d

Copy link

@random-zebra random-zebra left a comment

Choose a reason for hiding this comment

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

utACK bd3f39d

@Fuzzbawls Fuzzbawls merged commit 259523c into PIVX-Project:master Aug 27, 2020
@random-zebra random-zebra modified the milestones: 5.0.0, 4.3.0 Sep 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build System Trivial extremely simple issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants