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

Refactor: Easier support for chains with multiple genesis assets #191

Closed

Conversation

jtimon
Copy link
Contributor

@jtimon jtimon commented Jun 22, 2017

Replaces #164 .
See documentation there.

Dependencies:

@jtimon
Copy link
Contributor Author

jtimon commented Jul 19, 2017

Rebased after #187 was merged for clarity.

@jtimon
Copy link
Contributor Author

jtimon commented Aug 26, 2017

Rebased

Since the scriptPubKey for signing blocks never changes, there's no
point in repeating it with every block header.
@jtimon
Copy link
Contributor Author

jtimon commented Sep 21, 2017

Rebased to get the travis build.
But probably this will be closed after #260 has been merged.

@jtimon
Copy link
Contributor Author

jtimon commented Oct 1, 2017

@instagibbs I'm still very early in review of #260 but it seems the pegged asset is issued and can be issued further after genesis so there's no need to support several genesis assets as this PR proposed, as far as I understand, one could create a chain with 100 pegged assets (even if mutli-pegged-asset is not supported in the interpreter and the rest of the 2wp componenets yet) and there's no need to put anything in the gensis tx besides a single unit of a single genesis asset since that used once to create a second asset is enough entropy as long as you don't destroy the genesis asset and all the other assets created after it. Besides tahat destroying it is fine, but the point is it should be possible to use it only once and and destroy it.

And because of this text wall, closing, this should be no longer necessary.

@jtimon jtimon closed this Oct 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant