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
Add Python 3.11 to GHA #590
Conversation
Same problem, but looks like flake8-quotes here. Maybe pinning flake8 is the best option. Line 13 in 2784660
|
|
Signed-off-by: Gábor Lipták <gliptak@gmail.com>
# $ make install PYTHON=python3.7 | ||
|
||
PYTHON = python3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be PYTHON ?= python3
?
https://www.gnu.org/software/make/manual/html_node/Setting.html
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Linked doc says:
If you’d like a variable to be set to a value only if it’s not already set, then you can use the shorthand operator ‘?=’ instead of ‘=’.
PYTHON is already set, so there's no need to use ?=
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ahh, true. I just did a test with this Makefile
:
PYTHON = python3
MYTHON ?= python3
.PHONY: test
test:
@echo "PYTHON is set to $(PYTHON)"
@echo "MYTHON is set to $(MYTHON)"
which gives these results:
$ make test PYTHON=python3.8 MYTHON=python3.8
PYTHON is set to python3.8
MYTHON is set to python3.8
$ PYTHON=python3.8 MYTHON=python3.8 make test
PYTHON is set to python3
MYTHON is set to python3.8
so this was just a gap in my understanding of Makefile variables. Sorry!
Merging. Thanks. |
Signed-off-by: Gábor Lipták gliptak@gmail.com