Skip to content

Conversation

@nmarley
Copy link

@nmarley nmarley commented Sep 15, 2019

When I try and build chia_bls dependency in Alpine linux, the compiler can't find the path to the gmp.h header:

/home/alpine/dash/depends/work/build/x86_64-pc-linux-gnu/chia_bls/v20181101-7d7df22e185/contrib/relic/include/relic_types.h:39:10: fatal error: gmp.h: No such file or directory
 #include <gmp.h>
          ^~~~~~~
compilation terminated.

I am assuming this can be found in <staging-dir>/<host-triplet> but not sure why this is working in Ubuntu but not on Alpine. For some reason it's not finding the gmp header directly in dash/depends/<host-triplet>.

This patch allows the compiler to find the header files and build the chia_bls dependency.

@nmarley nmarley requested a review from codablock September 15, 2019 19:46
@UdjinM6 UdjinM6 added this to the 14.1 milestone Sep 15, 2019
@UdjinM6
Copy link

UdjinM6 commented Sep 15, 2019

Partially fixes #3090

Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

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

utACK

Copy link
Member

@PastaPastaPasta PastaPastaPasta left a comment

Choose a reason for hiding this comment

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

travis fails, seems unrelated, utACK

Copy link

@codablock codablock left a comment

Choose a reason for hiding this comment

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

utACK

@UdjinM6 UdjinM6 merged commit 822e617 into dashpay:develop Sep 22, 2019
@nmarley nmarley deleted the fix-bls-include-prefix branch September 22, 2019 21:17
nmarley added a commit to nmarley/dash that referenced this pull request Oct 4, 2019
nmarley added a commit to nmarley/dash that referenced this pull request Nov 21, 2019
nmarley added a commit to nmarley/dash that referenced this pull request Dec 7, 2019
barrystyle pushed a commit to PACGlobalOfficial/PAC that referenced this pull request Jan 14, 2020
CryptoCentric pushed a commit to absolute-community/absolute that referenced this pull request Jul 1, 2020
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.

4 participants