Fixes linux build with proper berkeleydb libs to have compatibility with windows wallet.dat#32
Open
tperalta82 wants to merge 2 commits intoEmberCoin:masterfrom
tperalta82:master
Open
Fixes linux build with proper berkeleydb libs to have compatibility with windows wallet.dat#32tperalta82 wants to merge 2 commits intoEmberCoin:masterfrom tperalta82:master
tperalta82 wants to merge 2 commits intoEmberCoin:masterfrom
tperalta82:master
Conversation
Owner
|
Hey, this is way too much. Since we went to supporting multiple OSes it no longer makes sense to have building of dependencies in the project anymore. Let me see if building on linux needs work and we'll go from there. |
Author
|
I agree with you, the problem is that you are using libdb 5.x (depending on the OS), on linux, and on windows you are forcing it to 6.2.x that you probably have on your devel env. This is the only reason why I added it to linux, otherwise, neither the blockchain data for the new wallet that is on your google drive, nor the windows wallet.dat works due to incompatibility. And you are also building leveldb along the way. Thanks for the input |
|
I have a patch ready for this specific issue. I should have the time to
push tomorrow.
…On Fri, Oct 13, 2017 at 8:42 AM, Tiago Peralta ***@***.***> wrote:
I agree with you, the problem is that you are using libdb 5.x (depending
on the OS), on linux, and on windows you are forcing it to 6.2.x that you
probably have on your devel env. This is the only reason why I added it to
linux, otherwise, neither the blockchain data for the new wallet that is on
your google drive, nor the windows wallet.dat works due to incompatibility.
And you are also building leveldb along the way.
Thanks for the input
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#32 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABUX-BlDO1x7PoMPL-Y0y3B-92OHNPlnks5sr1qlgaJpZM4P36Kv>
.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This may be an ugly hack, but it's working at the moment, builds and runs properly (i'm using it)
Please comment/reject/whatever if something is wrong as i'm still learning
Not sure if it will break windows build, as I can't test it atm