[pacman-dev] [PATCH 2/2] Use the 'configure'd PYTHON to run pactest.
allan at archlinux.org
Fri Dec 20 06:25:23 EST 2013
On 20/12/13 21:22, Allan McRae wrote:
> On 20/12/13 15:30, Jeremy Heiner wrote:
>> On Thu, Dec 19, 2013 at 11:35 PM, Andrew Gregory
>> <andrew.gregory.8 at gmail.com> wrote:
>>> On 12/19/13 at 03:31pm, Jeremy Heiner wrote:
>>>> -#! /usr/bin/python2
>>>> +# (no '#!' here because configure picks PYTHON)
>>> What is the point of removing this shebang and changing the mode? The only
>>> thing that accomplishes is making it impossible to run directly.
>> The intent was stated in the commit message as "to discourage
>> well-intentioned reversion"... I.e. without this change it is more
>> likely in the future that someone will submit a patch that says
>> "Makefile.am can be made simpler by removing the $(PYTHON) from
>> PY_LOG_COMPILER since pactest.py is executable." Besides, I wager you
>> and me are the only ones who have run pactest directly in the past 6
>> months. It's just so much more pleasant to let 'make check' run it and
>> format the output nicely.
> I'll take that wager!
> $ history | grep pactest.py | wc -l
Which is to say, there is absolutely no way I will accept removing the
shebang from the top. Running pactest.py directly is very important
when debugging a failure.
./pactest.py -v -d2 --keep-root -p ...
More information about the pacman-dev