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

Update default python version #751

Merged
merged 23 commits into from
Apr 21, 2020
Merged

Conversation

gronke
Copy link
Member

@gronke gronke commented Mar 21, 2020

addresses #750

  • Update default Python version to 3.7
  • Document installation from Makefile for other Python versions in README

@gronke gronke added the chore label Mar 21, 2020
Copy link
Collaborator

@igalic igalic left a comment

Choose a reason for hiding this comment

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

🙋‍♀️


LATEST_PKG_PYTHON_VERSION != pkg info -g 'python3*' | cut -d'-' -f1 | sed 's/^python//' | sort -n | tail -n1 | sed -r 's/^([0-9])([0-9]+)/\1.\2/'
PYTHON ?= python${LATEST_PKG_PYTHON_VERSION}
pyver= ${PYTHON:S/^python//:S/.//:C/\([0-9]+\)/\1/}
Copy link
Collaborator

Choose a reason for hiding this comment

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

given that we already have LATEST_PYTHON_VERSION we can use this directly here

Makefile Outdated

LATEST_PKG_PYTHON_VERSION != pkg info -g 'python3*' | cut -d'-' -f1 | sed 's/^python//' | sort -n | tail -n1 | sed -r 's/^([0-9])([0-9]+)/\1.\2/'
PYTHON ?= python${LATEST_PKG_PYTHON_VERSION}
Copy link
Collaborator

Choose a reason for hiding this comment

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

this should still have a fallback, since we use this makefile on Travis, too.

Copy link
Member Author

Choose a reason for hiding this comment

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

It is a bit complicated. There are multiple layers of overrides happening here.

PYTHON_VERSION ?= $(TRAVIS_PYTHON_VERSION)
SELECTED_PYTHON_VERSION != if [ "$(PYTHON_VERSION)" != "" ]; then echo $(PYTHON_VERSION); else pkg info -g 'python3*' | cut -d'-' -f1 | sed 's/^python//' | sort -n | tail -n1 | sed -r 's/^([0-9])([0-9]+)/\1.\2/'; fi
PYTHON ?= python${SELECTED_PYTHON_VERSION}
pyver= ${PYTHON:S/^python//:S/.//:C/\([0-9]+\)/\1/}

1.) When no PYTHON_VERSION was set, use TRAVIS_PYTHON_VERSION
2.) If neither was set, guess SELECTED_PYTHON_VERSION using pkg info
3.) This all can be overwritten via PYTHON.
4.) Read numeric pyver of whatever version was selected

@gronke gronke force-pushed the chore/update-default-python-version branch from 3be5ab5 to 02b3780 Compare April 13, 2020 14:28
@gronke gronke force-pushed the chore/update-default-python-version branch from bf2575f to 2906aa4 Compare April 13, 2020 19:40
@gronke gronke force-pushed the chore/update-default-python-version branch from 4d60330 to cbb3037 Compare April 20, 2020 17:09
Copy link
Collaborator

@igalic igalic left a comment

Choose a reason for hiding this comment

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

lies and half truths.

make PYTHON=python3.8 install
```

At the current time libioc is not packaged or available in FreeBSD ports.
Copy link
Collaborator

Choose a reason for hiding this comment

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

this is not entirely true…
https://www.freshports.org/devel/py-libioc it just… still doesn't have all the right dependencies…

Copy link
Member Author

Choose a reason for hiding this comment

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

Well, let's tackle this at a different time. Would you open an Issue as a reminder? 💜

@gronke gronke force-pushed the chore/update-default-python-version branch from cbb3037 to 2ff6c52 Compare April 20, 2020 17:19
@gronke gronke force-pushed the chore/update-default-python-version branch from 2ff6c52 to 71f058e Compare April 20, 2020 17:42
Copy link
Collaborator

@igalic igalic left a comment

Choose a reason for hiding this comment

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

approving this despite the failing vnet tests
We can tackle these separately

@gronke gronke force-pushed the chore/update-default-python-version branch from 5637df9 to eba2847 Compare April 20, 2020 23:31
@gronke gronke merged commit 77f7229 into master Apr 21, 2020
@gronke gronke deleted the chore/update-default-python-version branch April 21, 2020 01:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants