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

Test updating github actions #1219

Closed
wants to merge 72 commits into from
Closed

Test updating github actions #1219

wants to merge 72 commits into from

Conversation

luciansmith
Copy link

Need to rely less on node16 and move to node20.

Also need to combine python runs.

adelhpour and others added 30 commits May 1, 2024 18:14
@adelhpour
CI/CD workflow using GitHub Actions is added
3b06868
@adelhpour
stochastic test suite is given more paths to find the stochastic tests
9030a4e
@adelhpour
CI/CD using GitHub Actions is updated.
75b7b60 
- it now runs on macos machines with Apple Silicon chips

- gcc is now updated in all linux platforms

- Host architecture is no longer provided in the matrix and is extracted internally

- artifacts names now provide more info about the machine and the compiler they were built on
@luciansmith
Stop using set-output
6667cde 
Convert to the new GITHUB_OUTPUT instead.  Might not work with 'message'; could need to use 'echo' instead?
@luciansmith
Update msvc-dev-cmd
c05c35e
@luciansmith
Experiment with some other versions of things.
abfceef 
In theory, we'll need to update to node20 soon; this gets us partway there, and also sets up some python things.
@luciansmith
Revert experiment; work with that in different branch.
c9256c5
@luciansmith
Try to use upload-artifacts v3
2226aef 
Hopefully should work with node20?  Might need v4 for that.  Relative paths don't work, cf actions/upload-artifact#176
@luciansmith
Fix env.
9589e09
@luciansmith
Update checkout for non-manylinux.
8e262db
@luciansmith
First pass at collapsing python elements.
0992fb8
@luciansmith
Checkout roadrunner first.
cc208cc
@luciansmith
Windows powershell has different syntax.
6ae6fbf
@luciansmith
Fix typo.
c25d81b
@luciansmith
Maybe this is how different pythons are installed on windows?
4b12ffa
@luciansmith
Try to find python from multiple pythons on windows.
5739b35
@luciansmith
More exploration of python environment variables.
4960b0f
@luciansmith
Will work on Windows if relative paths are OK.
c4e4b85
@luciansmith
Try another format.
990b38f
@luciansmith
Try reversing slashes in directory.
519fe84
@luciansmith
Try single quotes!
483ec34
@luciansmith
Check to see if relative directories are the problem.
d386ab6
@luciansmith
Can it find requirements?
2b7808f
@luciansmith
Can we find the directory at all?
86607f0
@luciansmith
Try solution from https://stackoverflow.com/questions/66450348/need-t…
3abadc0 
…o-convert-back-slashes-to-forward-slashes-in-github-action-work-flow
@luciansmith
Fix typo.
3a95dd3
@luciansmith
More experiments.
06ce171
@luciansmith
Gah, was setting bash explicitly.
df9874e
@luciansmith
Split python install into Windows and non-.
33c3bdd
@luciansmith
Go back to trying to replace slashes, but in powershell.
c103d05
@luciansmith
Set up python_v1_exe and python_v1_dir variables.
d5c053f
@luciansmith
Fix some syntax.
8a5cd70
@luciansmith
More python directory setting.
bea7178
@luciansmith
Almost have everything set up.
591dcd4
@luciansmith
More fixes.
2396a5d
@luciansmith
Find out what actual executable filename is.
33f4d86
@luciansmith
No numbering in directory, apparently.
e2b213b
@luciansmith
Simplify python installation; start work on CMake/Python.
c098653
@luciansmith
Up one dir for manylinux python root dir.
057e625
@luciansmith
Fix the manylinux python dir a different way?
71b1f92
@luciansmith
Fix cmake options.
93869ca
@luciansmith
Get python tests working, and start working on multiple python builds.
c4c42bc
@luciansmith
Try to get new 'build' module to work.
03b32e3
@luciansmith
Fix typos.
18ae2eb
@luciansmith
Fixed wheel; attempt to make wheel artifacts directory.
9e1800a
@luciansmith
My relative filename fix might not have worked?
65d1286 
For now, switch to upload v1, which allows relative filenames.
@luciansmith
Fix syntax error.
755a8e2
@luciansmith
Try different way of running tests.
9ae7d29
@luciansmith
Try more than just -V for ctest.
59475b7 
Rename wheel files as we go.

Fix typo in filename.
@luciansmith
Fix issues with renaming wheel files.
8cc2285
@luciansmith
Fix sed part of commands.
d340943
@luciansmith
Smallish fixes.
8494ed2
@luciansmith
Only macos-latest claims 'universal'.
d279b81
@luciansmith
Remove previous install before re-building.
bb18a6c 
Problem on manylinux where the setup.py was changed but not overwritten. This is overkill, but hopefully will prevent any such similar problems.
@luciansmith
Move to macos13; don't delete old wheels.
b57477b
@luciansmith
13 is too early; use 14.
02ae195
@luciansmith
Change how to find root directory.
f1aa27b
@luciansmith
Rename files based on filenames, not OS.
ffc867c 
From https://stackoverflow.com/questions/6363441/check-if-a-file-exists-with-a-wildcard-in-a-shell-script
@luciansmith
Don't rebuild with different pythons if python is off.
037cc67 
Also, run final python tests if 'python' and 'tests' are on; don't check 'Release'.
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.
2 participants
@luciansmith @adelhpour