Code review comment for lp://qastaging/~intrigeri/apparmor/utils-keep-shebang

Revision history for this message
Steve Beattie (sbeattie) wrote :

Oh, I take what I said slightly back. If the PYTHON environment variable is set, it will always overwrite it (but just for aa-easyprof, as it's the only util covered by the setup-tools script, because the rest get installed in ${PREFIX}/sbin/). Given that we've deprecated python 2, going forward the only reason to use this would be to specify a specific version of python3 (or a hypothetical future python4) because the code was incompatible with the default python3. I don't think it's worth supporting that at this time, and still feel dropping the dead code is the preferred option here.

« Back to merge proposal