Skip to content
This repository has been archived by the owner on Sep 1, 2023. It is now read-only.

Update dependencies #1318

Closed
wants to merge 4 commits into from
Closed

Conversation

breznak
Copy link
Member

@breznak breznak commented May 23, 2017

Update most dependencies to the latest versions:

Fixes #1308
Fixes #1293

@breznak
Copy link
Member Author

breznak commented May 23, 2017

Commit 30def6c updating yaml-cpp is known to fail, I'll have to revert it. We are using the old API version 0.3.0, while upstream is it 0.5.3 - would it be possible for us to easily transition?
Otherwise this PR passes OK locally, so the rest of the commits are ready for review/merge.

@oxtopus @vitaly-krugl

@breznak
Copy link
Member Author

breznak commented May 23, 2017

Windows swig build fails
https://ci.appveyor.com/project/numenta-ci/nupic-core/build/0.3.0.2101/job/4h33kp4iamn7nt4h#L1289
is it related to the expected) #1292 (review) (part 1, win)? Or another issue? If the former, we can try to install the special win binary as suggested
http://prdownloads.sourceforge.net/swig/swigwin-3.0.12.zip

@vitaly-krugl
Copy link
Member

Commit 30def6c updating yaml-cpp is known to fail, I'll have to revert it. We are using the old API version 0.3.0, while upstream is it 0.5.3 - would it be possible for us to easily transition?
Otherwise this PR passes OK locally, so the rest of the commits are ready for review/merge.

@breznak, I have not had the opportunity/time to look into the new Yaml-cpp API, so can't comment on how easily we might transition to it.

@vitaly-krugl
Copy link
Member

Windows swig build fails
https://ci.appveyor.com/project/numenta-ci/nupic-core/build/0.3.0.2101/job/4h33kp4iamn7nt4h#L1289
is it related to the expected) #1292 (review) (part 1, win)? Or another issue? If the former, we can try to install the special win binary as suggested
http://prdownloads.sourceforge.net/swig/swigwin-3.0.12.zip

cc @scottpurdy

@breznak, it would be reasonable to follow SWIG's recommendation per @scottpurdy's review feedback. That said, this pull request might be attempting too many things. Perhaps SWIG upgrade could be done via a separate PR out of practical considerations.

@breznak
Copy link
Member Author

breznak commented May 24, 2017

Closing as suggested, will provide smaller, separate PRs instead.

@breznak breznak closed this May 24, 2017
@breznak breznak deleted the update_dependencies branch May 24, 2017 08:18
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.

2 participants