-
Notifications
You must be signed in to change notification settings - Fork 18
Python 3.5 for 0.22.X #49
Changes from 1 commit
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -13,6 +13,13 @@ jobs: | |
name: windows | ||
vmImage: vs2017-win2016 | ||
matrix: | ||
py_3.5_32: | ||
PYTHON_VERSION: "3.5.x" | ||
PYTHON_ARCH: "x86" | ||
NP_BUILD_DEP: "1.12.1" | ||
py_3.5_64: | ||
PYTHON_VERSION: "3.5.x" | ||
NP_BUILD_DEP: "1.12.1" | ||
py_3.6_32: | ||
PYTHON_VERSION: "3.6.x" | ||
PYTHON_ARCH: "x86" | ||
|
@@ -46,6 +53,11 @@ jobs: | |
name: linux | ||
vmImage: ubuntu-16.04 | ||
matrix: | ||
py_3.5_32: | ||
MB_PYTHON_VERSION: "3.5" | ||
PLAT: "i686" | ||
py_3.5_64: | ||
MB_PYTHON_VERSION: "3.5" | ||
py_3.6_32: | ||
MB_PYTHON_VERSION: "3.6" | ||
PLAT: "i686" | ||
|
@@ -75,6 +87,9 @@ jobs: | |
name: macOS | ||
vmImage: xcode9-macos10.13 | ||
matrix: | ||
py_3.5_64: | ||
MB_PYTHON_VERSION: "3.5" | ||
NP_BUILD_DEP: "numpy==1.13.3" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
We should try to use the even older numpy 1.11.0 for building as noted below. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. They were different for py36, I just used the same as py36 for each platform. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is also consistent with https://pypi.org/project/oldest-supported-numpy/ BTW There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. yes for macos, but not for windows. And we don't force the version for linux. I don't know why |
||
py_3.6_64: | ||
MB_PYTHON_VERSION: "3.6" | ||
NP_BUILD_DEP: "numpy==1.13.3" | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should force the build against the oldest supported of numpy for Python 3.5.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes but the existing ones are not following oldest supported numpy. I thought that was intentional
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's probably because I used the version numbers for 0.23 and not 0.22.X
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Anyway that should be fine as it is.