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

build fixes #13

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

build fixes #13

wants to merge 8 commits into from

Conversation

grimreaper
Copy link

This allows libzerocoin to build in more places. I've almost certainly missed things (e.g., I don't add rpath to LDFLAGS) but this is good enough for now.

Fix bogus use 'g++'.  On some systems 'g++' is more correctly spelled
'g++48' or the like.
boost no longer uses the 'mt' suffix.
make(1) is already chatty for you.  You can even make it more verbose.
Adding manual echo lines is pointless.
Most users expect CXXFLAGS and LDFLAGS to be used.
Without this the C++ code could not compile:

bitcoin_bignum/serialize.h:911:10: error: class member cannot be redeclared
    void insert(iterator it, std::vector<char>::const_iterator first, std::vector<char>::const_iterator last)
         ^
bitcoin_bignum/serialize.h:898:10: note: previous declaration is here
    void insert(iterator it, const_iterator first, const_iterator last)
         ^
The C++ code uses 'long long' which is a C++11 feature, so depend on it.
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.

1 participant