Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add a cygwin build to the list of PR actions #12

Open
wants to merge 31 commits into
base: master
Choose a base branch
from
Open

Add a cygwin build to the list of PR actions #12

wants to merge 31 commits into from

Conversation

DWesl
Copy link
Owner

@DWesl DWesl commented Feb 3, 2021

We'll see if this works.

Description

Motivation and Context

Test the cygwin build environment

How Has This Been Tested?

This PR is the test.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • I give permission for my code to be distributed under the existing license

DWesl added 15 commits February 3, 2021 11:04
@DWesl
Add a cygwin build to the PR list.
ed0d827 
We'll see if this works.
@DWesl
Checkout the repository in the Cygwin build.
f417501
@DWesl
Add more requirements to the cygwin setup.
4ca3555
@DWesl
Try to cache the pip cache in the GitHub Actions cache.
017f1fd 
This should speed things up a bit.

Also caches the Cygwin download cache.  I don't want to try skipping
setup.exe entirely, but this should help.
@DWesl
YAML syntax fix.
47c00f3
@DWesl
Move cache settings to with: block where they belong.
9b54080
@DWesl
Fix syntax so config is actually in with: block.
d1c3214
@DWesl
Make pip config directory.
f556932
@DWesl
Wrap everything in "bash -c".
34f61c5 
Try to ensure everything actually runs in Cygwin.
@DWesl
Tell SciPy which BLAS and LAPACK to use.
273a684 
BLAS will redirect to openblas at runtime.
@DWesl
Condense requirements files.
057d0e6
@DWesl
Update benchmark dependency versions.
d5f078d
@DWesl
Note that the cython implementation makes the wheels non-universal.
ab7d422
@DWesl
Don't count linter helper files in coverage stats.
f5a7fac 
They aren't going to be imported or used, and they aren't supposed to
be.
@DWesl
Separate out wheel build and install steps, and check DLLs.
5d3a170 
I think this is also a problem here.  We'll see.
@DWesl
Add the Cygwin bin and fallback lapack directories to PATH
9a319ea 
Being able to find the linear algebra libraries is nice.
@DWesl
Stop trying to use variables for install path
58ad57a 
Just try to remember to install it.
DWesl added 9 commits July 20, 2021 16:52
@DWesl
Add C++ compiler to required packages
ec4357b 
SciPy uses C++ now.
@DWesl
CI: Update Cygwin build action
a31f115 
Specify shell as bash, wrap lines, consistently use python 3.8, and change to "official" Cygwin install action.
@DWesl
CI: Update tested python versions.
5159099 
Drop 3.5, add 3.8-3.10.
@DWesl
CI: Specify python versions as strings.
26b8e1f 
Hopefully this lets install-python try for 3.10 instead of 3.1
@DWesl
CI: Specify SciPy version to install.
eb8ca16 
Unpatched SciPy 1.7+ doesn't install on Cygwin
@DWesl
CI: Specify build dependencies for install
9622f5c 
SciPy install fails for some reason.
@DWesl
CI: Check that NumPy imports
4c48b60
@DWesl
CI: Check NumPy DLLs.
6f6d198
@DWesl
CI: Update path to include current Cygwin dir
19f80b7 
I left it at the defaults for the old Cygwin directory
@DWesl
Copy link
Owner Author

DWesl commented May 14, 2022

SciPy install fails, apparently due to missing NumPy configuration file. I've never encountered that file before, and it doesn't cause problems on local builds.

DWesl added 5 commits May 20, 2022 14:44
@DWesl
CI: Update SciPy version on Cygwin
7ac59a0 
See if this fixes the weird compile error.
@DWesl
CI: Revert back to using SciPy 1.6
fe2e10e 
Should update to 1.9 once that's available.
@DWesl
CI: Try installing SciPy 1.5
1f94444 
This used to work.  I should check what the versions were back when I was testing this with tox.
@DWesl
CI: Try adding Cygwin job with SciPy <1.0
27315e7
@DWesl
CI: Try using SciPy 1.2
450dd86
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@DWesl