Skip to content
This repository was archived by the owner on Aug 30, 2024. It is now read-only.

MAINT: install gfortran on linux #45

Merged
merged 1 commit into from
Apr 2, 2019

Conversation

tylerjereddy
Copy link
Contributor

@tylerjereddy tylerjereddy commented Apr 1, 2019

Related to #22, at least for Linux--both 32 and 64-bit Docker containers seem to be missing gfortran there. Mac doesn't seem to have this problem at the moment.

If this works, all linux tests should adjust from having 500+ skips to a more reasonable number of skips, irrespective of the sporadic failure or 32/64-bit status.

I also find the comment above my changes to be confusing:

We only run the 64 bit tests as of NumPy 1.16.

Especially since I've been trying to debug the sporadic Pinv issue again on a private repo with debug / interactive Travis mode. I'll see if I can figure out what changes are associated with that comment. The complex64 failure had me thinking about m32 compile / link flags again as well.

Perhaps the comment refers to Mac, but that is confusing given the context is a more general function.

* install gfortran in the Docker container
used for Linux testing so that f2py tests
are not skipped with crytpic messages about
missing C compilers
@tylerjereddy
Copy link
Contributor Author

Linux test skip counts indeed look drastically smaller now, even in the case of the Pinv failure scenario.

@charris
Copy link
Contributor

charris commented Apr 2, 2019

We only run the 64 bit tests as of NumPy 1.16.

IIRC, that was when we stopped releasing 32 bit Mac wheels.

This looks good to me. Thanks Tyler.

@charris charris merged commit 2d5c2d3 into MacPython:master Apr 2, 2019
@tylerjereddy tylerjereddy deleted the linux_gfortran branch April 2, 2019 02:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants