Skip to content

depends: non-qt bumps for 0.12#6373

Merged
laanwj merged 5 commits into
bitcoin:masterfrom
theuni:0.12bumps
Jul 15, 2015
Merged

depends: non-qt bumps for 0.12#6373
laanwj merged 5 commits into
bitcoin:masterfrom
theuni:0.12bumps

Conversation

@theuni

@theuni theuni commented Jul 3, 2015

Copy link
Copy Markdown
Member

These are all of the possible bumps for 0.12, qt and openssl excluded. I'm not advocating taking all of these, just listing them all in one place for discussion. Each is a separate commit so that the undesired ones can be easily dropped.

  • boost 1.55.0 to 1.58.0: This one's been discussed and I think we're in favor of taking it.
  • qrencode 3.4.3 to 3.4.4: changelog - I can't find specifically what changed, need to compare the sources.
  • protobuf 2.5.0 to 2.6.1: changelog1, changelog2 - This one may be risky, might be better to just leave it alone
  • ccache to 3.1.9 to 3.2.2: changelog - Looks good
  • miniupnpc 1.9.20140701 to 1.9.20150609: changelog

@theuni theuni force-pushed the 0.12bumps branch 2 times, most recently from f267bc5 to eb4f35d Compare July 3, 2015 21:20
@fanquake

fanquake commented Jul 4, 2015

Copy link
Copy Markdown
Member

This should show you the qrencode changes. fukuchi/libqrencode@055f441...3.4

@luke-jr

luke-jr commented Jul 4, 2015

Copy link
Copy Markdown
Member

Probably should wait until we're closer to rc1 before bumping, in case there's more fixes?

@laanwj

laanwj commented Jul 6, 2015

Copy link
Copy Markdown
Member

@luke-jr Bumping sooner has the advantage over doing it last-minute that at least the new versions can get some longer testing.

@laanwj

laanwj commented Jul 10, 2015

Copy link
Copy Markdown
Member

I'm fine with upgrading all of those.

Though I agree that libprotobuf is the most risky of the bunch.

@theuni

theuni commented Jul 10, 2015

Copy link
Copy Markdown
Member Author

@luke-jr Yes, I agree with @laanwj. Bumping these early means they get more testing. If obvious bugfixes come in closer to release, we can certainly still pull them in.

@jonasschnelli

Copy link
Copy Markdown
Contributor

Also agreed with @laanwj to take this in as soon as possible.
Tried a gitian build with this PR: https://builds.jonasschnelli.ch/pulls/6373/

@laanwj

laanwj commented Jul 15, 2015

Copy link
Copy Markdown
Member

ACK

@laanwj laanwj merged commit eb4f35d into bitcoin:master Jul 15, 2015
laanwj added a commit that referenced this pull request Jul 15, 2015
eb4f35d depends: bump qrencode to 3.4.4 (Cory Fields)
142ee60 depends: bump protobuf to 2.6.1 (Cory Fields)
d5b1851 depends: bump ccache to 3.2.2 (Cory Fields)
00c5a73 depends: bump miniupnpc to 1.9.20150609 (Cory Fields)
4bdad99 depends: bump boost to 1.58.0 (Cory Fields)
@Diapolo

Diapolo commented Jul 16, 2015

Copy link
Copy Markdown

The protobuf update seems to cause warnings:

protobuf-2.6.1\dist\include\google\protobuf\wire_format_lite_inl.h:383: Warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
     for (int i = 0; i < new_entries; ++i) {
                       ^

@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.

6 participants