Skip to content
This repository was archived by the owner on Feb 14, 2023. It is now read-only.

Python 3.5 for 0.22.X #49

Merged
merged 2 commits into from
Mar 4, 2020
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 15 additions & 0 deletions azure-pipelines.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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"
Expand Down Expand Up @@ -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"
Copy link
Contributor

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.

Copy link
Contributor Author

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

Copy link
Contributor

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

Copy link
Contributor

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.

py_3.6_32:
MB_PYTHON_VERSION: "3.6"
PLAT: "i686"
Expand Down Expand Up @@ -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"
Copy link
Contributor

@ogrisel ogrisel Mar 4, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not 1.12.1 as for Windows?

We should try to use the even older numpy 1.11.0 for building as noted below.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

The 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

Copy link
Contributor Author

Choose a reason for hiding this comment

The 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"
Expand Down