Overview
Request 508168 revoked
- suse-interpreter-python-and-env.patch:
* check that /usr/bin/env is not a valid interpreter
* warn when using /usr/bin/python
- Created by matejcik
- In state revoked
- 10 package maintainers
Loading...
Request History
matejcik created request
- suse-interpreter-python-and-env.patch:
* check that /usr/bin/env is not a valid interpreter
* warn when using /usr/bin/python
dirkmueller declined request
I'm inclined to say that this is not the right direction (see inline comment, but I'm happy to discuss it further)
matejcik revoked request
The source package 'rpmlint' has been removed
surely a error for that is overrated, given that "python" is valid for six python (e.g. 2.x and 3.x compatible at the same time).
I am not sure what the underlying reason is, is that in preparation of the python 3.x switch in tumbleweed?
I thought I made the python warning just a warning, with
printWarning
?The error is for using
/usr/bin/env
as an interpreter, which is arguably an error regardless of language. Seeing as system script behavior then depends on the user's $PATH.And yes, this is in preparation for the Python 3 switch. If this should be TW-specific, please tell me how to do that :)
But usr/bin/python is guaranteed to be python 2.x. I think it might make sense to tell people that they want to use python3 when the package name starts with python3- (or python2 when the package name starts iwth python2-) but globally this sounds wrong to me as a check.
happy to discuss that somewhere with you to see what needs to be done (and yes, I also have rpmlint for the python3 switch on my mental todo somewhere)
@aplanas, @dirkmueller, @krahmer, @lnussel, @msmeissn: review reminder