Skip to content

Conversation

@Fuzzbawls
Copy link

M1 MacOS using /opt/homebrew as a default path breaks some autotools default search locations. Further, the M1 CPU is aarch64, so detection needs to be introduced to pass relic build:

  • Add CPU Arch detection for darwin builds
  • Roll GMP_CPPFLAGS and GMP_LIBS into CPPFLAGS and LIBS
  • Include SODIUM_CFLAGS path

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.

Light ACK 544d00e.

Cannot validate it in an M1 Mac CPU but at least can confirm that nothing got broken in the other architectures.

furszy added a commit to PIVX-Project/PIVX that referenced this pull request Sep 16, 2021
df45553 CMake: Add M1 homebrew path to search paths (Fuzzbawls)
bd4fa73 Build: Add M1 homebrew path to boost search paths (Fuzzbawls)

Pull request description:

  Homebrew installs on Apple M1 based systems now use `/opt/homebrew` as the base path. This adds such path to the search paths for both autotools and cmake. PIVX-Project/bls-signatures#9 will also be needed in a future PR to compliment this.

ACKs for top commit:
  random-zebra:
    utACK df45553
  furszy:
    utACK df45553 and merging

Tree-SHA512: eff73b2cdc75245bd8a93555ebb917944f5cd574afd8b15c19c8a811f3d327d82d0f8778bbadf5eccfed61c6c3ada3872888e08dcbb94ed6f7b6287972930557
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.

ACK 544d00e

@furszy furszy merged commit 8c75dd8 into PIVX-Project:main Sep 25, 2021
furszy added a commit to furszy/bitcoin-core that referenced this pull request Sep 25, 2021
bc64f128f0 Merge PIVX-Project/bls-signatures#10: Threshold:  don't copy the entire vector just to cache the initial size.
8c75dd860d Merge PIVX-Project/bls-signatures#9: [Build] Fix autotools builds on M1 Mac CPU
46926a2216 Poly::Evaluate, don't copy the entire vector just to cache the initial vector size.
544d00ed23 [Build] Fix autotools builds on M1 Mac CPU

git-subtree-dir: src/chiabls
git-subtree-split: bc64f128f009fd8825f249090be9d64a74834093
furszy added a commit to PIVX-Project/PIVX that referenced this pull request Sep 27, 2021
ad308f2 Squashed 'src/chiabls/' changes from 676ea45e80..bc64f128f0 (furszy)

Pull request description:

  Updating the bls-sig repo to the latest version. Pulling:
  * PIVX-Project/bls-signatures#9
  * PIVX-Project/bls-signatures#10

ACKs for top commit:
  random-zebra:
    ACK ffc8e50

Tree-SHA512: 7cc8c9b9d267e9f2de1965cf17e055c724f441a9013b0557eb897fa4029d818008a3263ebe5be97a49ff4587480885caa9d101b92b24d629c6c3203f87a032bb
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.

3 participants