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

Fixed Github Actions workflows #385

Merged
merged 10 commits into from
Feb 4, 2025
Merged

Conversation

joanimato
Copy link
Collaborator

Fixed the workflows in the github actions folder. Now the code should automatically compile and upload to pypi on release. This included:

  • Updating the URL used to download boost
  • Updating the cibuildwheel version used for building
  • Updating the macos release MAKEFILE to C++14 standard (otherwise latest boost wasn't working)
  • Remove pyproject.toml. It doesn't really add anything right now. Perhaps a future release could upgrade to using a toml file instead of setup.py.

@@ -75,24 +79,32 @@ jobs:
sed -i s/^mirrorlist=http/#mirrorlist=http/g /etc/yum.repos.d/*.repo
Copy link
Collaborator

Choose a reason for hiding this comment

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

These sed commands were added to enforce a working config when we had runners still using CentOS 7 after it reached end of life. Feel free to discard if they're causing problems in the future. Thanks again for the work done

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

That workflow still uses Centos and it's a pain. It probably has to be rewritten.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Is there a way to request Ubuntu?

Copy link
Collaborator

Choose a reason for hiding this comment

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

No worries at the moment. If it's currently working we can keep it as it is

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, it is possible. The job below uses Ubuntu (I'm guessing it was written later?). But yeah, we can upgrade later.

Copy link
Member

@diogomart diogomart left a comment

Choose a reason for hiding this comment

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

Thank you Joani

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.

4 participants