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

configure option --enable-liquid for default liquidv1 chain #574

Merged
merged 14 commits into from
Apr 18, 2019

Conversation

instagibbs
Copy link
Collaborator

@instagibbs instagibbs commented Apr 12, 2019

The only behavior it modifies is that the default chain becomes liquidv1. Also added a Travis test for coverage and adapted a unit test, and enabled this mode for gitian descriptors.

@instagibbs
Copy link
Collaborator Author

build seems to work, needs to rebased after #573 is merged

@instagibbs
Copy link
Collaborator Author

rebased.

@instagibbs
Copy link
Collaborator Author

instagibbs commented Apr 12, 2019

Had two functional tests fail, one in two separate builds not related to this PR(mysterious failure). I'm running these two tests in a spin-loop for a bit locally, and have restarted those builds.

rare error with RPC port something something: #576

interface_bitcoin_cli: Can't replicate at all locally after hundreds of runs.

@stevenroose
Copy link
Member

Did you test the Gitian builds? @greenaddress Does this look good?

@instagibbs instagibbs merged commit ff12bf7 into ElementsProject:master Apr 18, 2019
instagibbs added a commit that referenced this pull request Apr 18, 2019
ff12bf7 QT: --enable-liquid needs to use different QAPP name for QSettings (Gregory Sanders)
f475780 QT: WalletModel should always check for balance changes once (Gregory Sanders)
93e2660 QT: dummy address needs fully valid pubkey (Gregory Sanders)
c227c65 QT: Cache the blinded destination values on transactions we create, like -cli (Gregory Sanders)
5622a83 QT: remove GetAmounts calls or put behind bitcoind mode only (Gregory Sanders)
17229bf QT: Add bracket to not reassign values on non-tx (Gregory Sanders)
88c256f Split out liquid gitian build descriptors (Gregory Sanders)
7bd0667 Change gitian defaults back to elements, leave commented out lines as switch (Gregory Sanders)
6d6ddc5 Rename binaries to liquid naming at end of gitian process (Gregory Sanders)
aa2af38 --enable-liquid: liquid.conf and liquid.pid (Gregory Sanders)
3b1ca17 --enable-liquid: change default datadir to .liquid (Gregory Sanders)
1f67fec --enable-liquid for gitian builds (Gregory Sanders)
1fdc059 Add travis build for --enable-liquid (Gregory Sanders)
7eea59a configure option --enable-liquid for default liquidv1 chain (Gregory Sanders)

Pull request description:

  The only behavior it modifies is that the default chain becomes `liquidv1`. Also added a Travis test for coverage and adapted a unit test, and enabled this mode for gitian descriptors.

Tree-SHA512: 9a19267a51ee7a8906b6a8af9c511a4838ec529c5ea2873de215eb849111d2d2380679b524e7198f53606810b47cd85ef46a8fc5816c86e78ebedd8c30de0da7
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.

2 participants