Re: [scripts] python: change scripts to use system python instead of env

From: Jarod Wilson
Date: Thu Jan 28 2010 - 15:21:21 EST


On Tue, Jan 26, 2010 at 9:34 PM, Américo Wang <xiyou.wangcong@xxxxxxxxx> wrote:
> On Wed, Jan 27, 2010 at 6:20 AM, Don Zickus <dzickus@xxxxxxxxxx> wrote:
>> Doh. Typo in the To: field
>>
>> ----- Forwarded message from Don Zickus <dzickus@xxxxxxxxxx> -----
>>
>> Date: Fri, 22 Jan 2010 16:32:10 -0500
>> From: Don Zickus <dzickus@xxxxxxxxxx>
>> To: lkml@xxxxxxxxxxxxxxx
>> Cc: akpm@xxxxxxxxxxxxxxxxxxxx, tglx@xxxxxxxxxxxxx, tony.luck@xxxxxxxxx,
>>        kmcmartin@xxxxxxxxxx, Don Zickus <dzickus@xxxxxxxxxx>
>> Subject: [scripts] python: change scripts to use system python instead of env
>> Message-Id: <1264195930-15815-1-git-send-email-dzickus@xxxxxxxxxx>
>>
>> Just a small change to a couple of scripts to go from
>>
>>  #!/usr/bin/env python
>>
>> to
>>
>>  #!/usr/bin/python
>>
>> This shouldn't effect anyone, unless they don't install python there.
>>
>> In preparation for python3, Fedora is doing a big push to change the scripts
>> to use the system python.  This allows developers to put the python3 in
>> their path without fear of breaking existing scripts.
>>
>> Now I am pretty sure anyone using python3 for testing purposes will probably
>> not run any of the scripts I changed, but Fedora has this automated tool
>> that checks for this stuff so I thought I would try to push it upstream.
>
>
> Hmm, so python3 will be installed in /usr/local/bin?

No, it will be /usr/bin/python3, but its possible a user would
configure their environment to have their default python interpreter
be python3, in which case env python would be using /usr/bin/python3
rather than /usr/bin/python, which can lead to things mysteriously
running differently or not running at all.

Acked-by: Jarod Wilson <jarod@xxxxxxxxxx>

--
Jarod Wilson
jarod@xxxxxxxxxxxx
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/