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

Loading...

Dirk Mueller's avatar
source maintainer target maintainer

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?


Jan Matejek's avatar

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 :)


Dirk Mueller's avatar
source maintainer target maintainer

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)


Request History
Jan Matejek's avatar

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


Dirk Mueller's avatar

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)


Jan Matejek's avatar

matejcik revoked request

The source package 'rpmlint' has been removed

openSUSE Build Service is sponsored by