Re: [Bug-wget] python3

[Jérémie Courrèges-Anglas]

Ángel González <address@hidden> writes:

> On 18/12/14 09:16, Jérémie Courrèges-Anglas wrote:
>> wget-1.16.1 introduced a change wrt. python.  Just upgrading the port
>> lead to the following failure:
>>
>> checking whether /usr/local/bin/python3.4 version is>= 3.0... no
>> configure: error: Python interpreter is too old
>> *** Error 1 in . (/usr/ports/infrastructure/mk/bsd.port.mk:2725 
>> '/usr/ports/pobj/wget-1.16.1/.configure_done': @for d in 
>> /usr/ports/pobj/wge...)
>> *** Error 1 in /usr/ports/net/wget 
>> (/usr/ports/infrastructure/mk/bsd.port.mk:2473 'all')
>>
>> This is because when python 3 support is requested in our ports
>> framework, the following values are set in the ./configure script
>> environment:
>>
>>    PYTHON="/usr/local/bin/python3.4"
>>    ac_cv_prog_PYTHON="/usr/local/bin/python3.4"
>>    ac_cv_path_PYTHON="/usr/local/bin/python3.4"
> No. That's not your ports fault (well, unless you are naming python 2.7
>
> "python3.4" ;) )

Of course not.  The problem is just that we specify, via environment,
a path to a non-existent python executable.

>> I "fixed" this by:
>> 1. forcing PYTHON to the empty string, to avoid the error message
>> 2. setting am_cv_pathless_PYTHON so that tests can still pass (after
>>     installation of python3 of course)
>>
>>    
>> http://cvsweb.openbsd.org/cgi-bin/cvsweb/ports/net/wget/Makefile.diff?r1=1.65&r2=1.66
>>
>> I thought I'd let you know about this, even though I'm not sure if there
>> is a reasonable way to "solve" this issue. I feel that AM_PATH_PYTHON is
>> doing the right thing.
> I don't know which check is used there by wget, but it's broken.

I think it works as designed, but it was not designed especially for
wget and its test suite.  I think it would be fine to simply print
a warning, instead of bailing out, but I'm not sure it's possible to
tweak that in a clean manner, and IIUC that would be a job for next
automake releases, not for wget.

>> BTW, python3.4 could be added to _AM_PYTHON_INTERPRETER_LIST.

Same thing here I guess.

-- 
jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF  DDCC 0DFA 74AE 1524 E7EE