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.

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

Upgrade actions/download-artifact to v4; Explicitly Check PEP 440 Compliance; Fix Workflows, Makefiles, and Boost Usage #340

Merged
merged 2 commits into from
Sep 7, 2024

Conversation

rwxayheee
Copy link
Collaborator

@rwxayheee rwxayheee commented Sep 7, 2024

This is not urgent to merge. The PR is intended to provide a temporary solution for current issues with Git Actions and maybe serve as a starting point for future updates.

This PR will allow itself to pass all checks triggered by PR, generating wheel files for Linux x86_64 (wheels), source dist (sdist), and compiled binaries for Linux x86_64/aarch64, MacOS x86_64/aarch64, and Windows as artifacts.

1. upgrade actions/upload-artifact;
2. include macos-aarch64 in compile binaries;
3. exclude macos, linux-aarch from pypi-build;
4. update yum repo files as a temp fix for Mirrorlist end-of-life;
5. change of job name in compile binaries
to find boost installed by apt-get (Linux)
and homebrew (intel, Apple Silicon);
2. for newer boost versions: replace boost/filesystem/*.hpp
by boost/filesystem.hpp in source;
3. explictly check for PEP 440 compliance of version:
include packaging as build dependency,
only return compliant version from
setup.py
@rwxayheee rwxayheee changed the title Upgrade actions/download-artifact to v4; Expicitly Check PEP 440 Compliance; Fix Workflows, Makefiles, and Boost Usage Upgrade actions/download-artifact to v4; Explicitly Check PEP 440 Compliance; Fix Workflows, Makefiles, and Boost Usage Sep 7, 2024
@diogomart diogomart merged commit ee44699 into ccsb-scripps:develop Sep 7, 2024
8 checks passed
@diogomart
Copy link
Member

Amazing, thank you!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants