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 BDB utilities #4567

Merged
merged 5 commits into from
Aug 7, 2020
Merged

Build BDB utilities #4567

merged 5 commits into from
Aug 7, 2020

Conversation

oxarbitrage
Copy link
Contributor

@oxarbitrage oxarbitrage commented Jun 11, 2020

To install the binaries we need to build with just install instead of install_lib and install_include, this will install everything.

Then the binaries will be moved to a folder in zcutil directory. We can just leave them in staging however the user might have a hard time to find them there.

Closes #4537

depends/packages/bdb.mk Outdated Show resolved Hide resolved
Copy link
Contributor

@str4d str4d left a comment

Choose a reason for hiding this comment

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

Concept ACK

depends/packages/bdb.mk Outdated Show resolved Hide resolved
depends/packages/bdb.mk Outdated Show resolved Hide resolved
@str4d str4d added the A-build Area: Build system label Jun 30, 2020
Co-authored-by: Jack Grigg <jack@z.cash>
Signed-off-by: Daira Hopwood <daira@jacaranda.org>
Signed-off-by: Daira Hopwood <daira@jacaranda.org>
Signed-off-by: Daira Hopwood <daira@jacaranda.org>
str4d
str4d previously requested changes Jul 30, 2020
Makefile.am Outdated Show resolved Hide resolved
zcutil/bin should be cleaned by `make distclean`, not `make clean`.

Co-authored-by: str4d <thestr4d@gmail.com>
Copy link
Contributor

@daira daira left a comment

Choose a reason for hiding this comment

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

Tested ACK for @str4d's change to Makefile.am. Also tested that the utilities are executable and seem to work.

@daira
Copy link
Contributor

daira commented Aug 7, 2020

@zkbot r+

@zkbot
Copy link
Contributor

zkbot commented Aug 7, 2020

📌 Commit 45b2a61 has been approved by daira

@daira daira dismissed str4d’s stale review August 7, 2020 10:23

Changes applied.

@zkbot
Copy link
Contributor

zkbot commented Aug 7, 2020

⌛ Testing commit 45b2a61 with merge 022c3a4...

@zkbot
Copy link
Contributor

zkbot commented Aug 7, 2020

☀️ Test successful - pr-merge
Approved by: daira
Pushing 022c3a4 to master...

@zkbot zkbot merged commit 022c3a4 into zcash:master Aug 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-build Area: Build system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Compile BDB utility programs as part of compiling zcashd
4 participants