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

fix free-bsd CI #579

Merged
merged 6 commits into from
Jul 10, 2024
Merged

fix free-bsd CI #579

merged 6 commits into from
Jul 10, 2024

Conversation

TingDaoK
Copy link
Contributor

@TingDaoK TingDaoK commented Jul 9, 2024

Issue #, if available:

  • FreeBSD 14:latest dropped python3.9, according to here

Description of changes:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@@ -232,7 +232,7 @@ jobs:
shell: bash
environment_variables: AWS_ACCESS_KEY_ID AWS_SECRET_ACCESS_KEY AWS_DEFAULT_REGION AWS_REGION
run: |
sudo pkg install -y python3 py39-urllib3 py39-pip py39-awscli cmake
sudo pkg install -y python3 devel/py-pip net/py-urllib3 devel/py-awscli cmake
Copy link
Contributor

Choose a reason for hiding this comment

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

did install python3 change from being 3.9 to 3.11?

does python3 point to whatever is "latest"? and like devel/py-pip points to whatever is "latest"?

If so, this seems fine

Copy link
Contributor Author

Choose a reason for hiding this comment

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

not sure if they just updated recently or not. but the current default version for python is 3.11 https://cgit.freebsd.org/ports/tree/Mk/bsd.default-versions.mk#n139

https://wiki.freebsd.org/Python looks like they will use the default version for all the python packages.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

freebsd/freebsd-ports@62a23ef oh, it was updated 2 month ago.

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