Skip to content

Conversation

@luke-jr
Copy link
Member

@luke-jr luke-jr commented Mar 10, 2018

The breaking changes (#12193) are already merged, so this blocks 0.17.0.

It depends on jgarzik/univalue#42 or jgarzik/univalue#50 being merged and released in UniValue 1.0.4.

@promag
Copy link
Contributor

promag commented Mar 11, 2018

Ah so this is what you meant the other day.

@maflcko maflcko added this to the 0.17.0 milestone Mar 11, 2018
@laanwj
Copy link
Member

laanwj commented Mar 12, 2018

Makes sense, utACK 8172d3a

@meshcollider
Copy link
Contributor

utACK 8172d3a

@laanwj laanwj merged commit 8172d3a into bitcoin:master Mar 14, 2018
laanwj added a commit that referenced this pull request Mar 14, 2018
8172d3a configure: UniValue 1.0.4 is required for pushKV(, bool) (Luke Dashjr)

Pull request description:

  The breaking changes (#12193) are already merged, so this blocks 0.17.0.

  It depends on jgarzik/univalue#42 or jgarzik/univalue#50 being merged and released in UniValue 1.0.4.

Tree-SHA512: 3a21bbc72d6632bd07ee60ad7780b9ee95908357bcf59b4795b693d8a5d8c88943d6451482f11916ff5417e3bdbb9916062f87d0d73e79f50eb95ddabe21f943
@Sjors
Copy link
Member

Sjors commented Mar 14, 2018

Isn't this merge premature given that 1.0.4. isn't released yet? Or should there be a ticket in the 0.17 milestone to make sure 1.0.4 is released and compatible?

PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jun 13, 2020
…(, bool)

8172d3a configure: UniValue 1.0.4 is required for pushKV(, bool) (Luke Dashjr)

Pull request description:

  The breaking changes (bitcoin#12193) are already merged, so this blocks 0.17.0.

  It depends on jgarzik/univalue#42 or jgarzik/univalue#50 being merged and released in UniValue 1.0.4.

Tree-SHA512: 3a21bbc72d6632bd07ee60ad7780b9ee95908357bcf59b4795b693d8a5d8c88943d6451482f11916ff5417e3bdbb9916062f87d0d73e79f50eb95ddabe21f943
gades pushed a commit to cosanta/cosanta-core that referenced this pull request Jun 26, 2021
…(, bool)

8172d3a configure: UniValue 1.0.4 is required for pushKV(, bool) (Luke Dashjr)

Pull request description:

  The breaking changes (bitcoin#12193) are already merged, so this blocks 0.17.0.

  It depends on jgarzik/univalue#42 or jgarzik/univalue#50 being merged and released in UniValue 1.0.4.

Tree-SHA512: 3a21bbc72d6632bd07ee60ad7780b9ee95908357bcf59b4795b693d8a5d8c88943d6451482f11916ff5417e3bdbb9916062f87d0d73e79f50eb95ddabe21f943
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants