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

Improve installation for non-depends builds #4

Open
ryanofsky opened this issue Jul 26, 2019 · 3 comments
Open

Improve installation for non-depends builds #4

ryanofsky opened this issue Jul 26, 2019 · 3 comments

Comments

@ryanofsky
Copy link
Collaborator

bitcoin/bitcoin#16367 (review) reported various problems getting multiprocess code to build outside the make depends system. I've only focused on make depends builds so far, but would like to create libmultiprocess packages and add better instructions to make installation in other contexts easier.

@ryanofsky
Copy link
Collaborator Author

@ariard can you post more details here? Main thing I'd like to know is how you installed capnp (via package manager or with manual install) and what platform or distro you used to make sure I can reproduce the problems.

If you have still have the build commands you used or any error messages / build output, it'd be great to post that as well. Thanks!

@ryanofsky
Copy link
Collaborator Author

Talked to @ariard, turns out he was doing a depends build of capnproto and libmultiprocess, but a non-depends build of bitcoin itself, due to not passing --prefix=$PWD/depends/x86_64-pc-linux-gnu to the bitcoin configure script. Passing the --prefix flag should fix all the issues encountered. To prevent this confusion in the future I think the depends README needs to be updated to clearly state that depends outputs will be ignored if prefix is not specified.

The other thing that should be be done is to update the build-osx and build-unix files to mention how to install capnproto and libmultiprocess as optional dependencies, maybe just linking to instructions at the bottom of the libmultiprocess README

ryanofsky added a commit to ryanofsky/bitcoin that referenced this issue Aug 6, 2019
If not cross compiling, it might actually be nice for it to be picked up
automatically. But for now clarify the readme to try to minimize confusion.

chaincodelabs/libmultiprocess#4 (comment)
chaincodelabs/libmultiprocess#5 (comment)
ryanofsky added a commit to ryanofsky/bitcoin that referenced this issue Aug 20, 2019
Clarify need to specify --prefix with depends

If not cross compiling, it might actually be nice for it to be picked up
automatically. But for now clarify the readme to try to minimize confusion.

chaincodelabs/libmultiprocess#4 (comment)
chaincodelabs/libmultiprocess#5 (comment)
fanquake pushed a commit to fanquake/bitcoin that referenced this issue Aug 23, 2019
If not cross compiling, it might actually be nice for it to be picked up
automatically. But for now clarify the readme to try to minimize confusion.

chaincodelabs/libmultiprocess#4 (comment)
chaincodelabs/libmultiprocess#5 (comment)
fanquake pushed a commit to fanquake/bitcoin that referenced this issue Aug 23, 2019
If not cross compiling, it might actually be nice for it to be picked up
automatically. But for now clarify the readme to try to minimize confusion.

chaincodelabs/libmultiprocess#4 (comment)
chaincodelabs/libmultiprocess#5 (comment)
sidhujag pushed a commit to syscoin/syscoin that referenced this issue Aug 23, 2019
If not cross compiling, it might actually be nice for it to be picked up
automatically. But for now clarify the readme to try to minimize confusion.

chaincodelabs/libmultiprocess#4 (comment)
chaincodelabs/libmultiprocess#5 (comment)
HashUnlimited pushed a commit to HashUnlimited/chaincoin that referenced this issue Aug 30, 2019
If not cross compiling, it might actually be nice for it to be picked up
automatically. But for now clarify the readme to try to minimize confusion.

chaincodelabs/libmultiprocess#4 (comment)
chaincodelabs/libmultiprocess#5 (comment)
barrystyle pushed a commit to zentoshi/zentoshi that referenced this issue Nov 11, 2019
If not cross compiling, it might actually be nice for it to be picked up
automatically. But for now clarify the readme to try to minimize confusion.

chaincodelabs/libmultiprocess#4 (comment)
chaincodelabs/libmultiprocess#5 (comment)
ShengguangXiao pushed a commit to DeFiCh/ain that referenced this issue Aug 28, 2020
If not cross compiling, it might actually be nice for it to be picked up
automatically. But for now clarify the readme to try to minimize confusion.

chaincodelabs/libmultiprocess#4 (comment)
chaincodelabs/libmultiprocess#5 (comment)
lyricidal added a commit to PRCYCoin/PRCYCoin that referenced this issue Mar 24, 2023
If not cross compiling, it might actually be nice for it to be picked up
automatically. But for now clarify the readme to try to minimize confusion.

chaincodelabs/libmultiprocess#4 (comment)
chaincodelabs/libmultiprocess#5 (comment)
lyricidal added a commit to PRCYCoin/PRCYCoin that referenced this issue Mar 24, 2023
If not cross compiling, it might actually be nice for it to be picked up
automatically. But for now clarify the readme to try to minimize confusion.

chaincodelabs/libmultiprocess#4 (comment)
chaincodelabs/libmultiprocess#5 (comment)
@hebasto
Copy link
Contributor

hebasto commented Mar 16, 2024

#4 (comment):

... would like to create libmultiprocess packages...

See #96.

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

No branches or pull requests

2 participants