On 2020/11/23 13:43, Daniel Shahaf wrote:
> Yasuhito FUTATSUKI wrote on Sun, Nov 22, 2020 at 18:24:35 +0900:
>> On 2020/11/22 17:54, Branko Čibej wrote:
>>> On 22.11.2020 08:24, futatuki_at_apache.org wrote:
>>>> Author: futatuki
>>>> Date: Sun Nov 22 07:24:46 2020
>>>> New Revision: 1883719
>>>>
>>>> URL: http://svn.apache.org/viewvc?rev=1883719&view=rev
>>>> Log:
>>>> Reduce macro expansion, no functional change.
>>>>
>>>> * build/ac-macros/swig.m4
>>>> everywhere: Remove trailing whitespaces.
>>>> (SVN_CHECK_SWIG): Gather the use of SVN_FIND_SWIG by using cache variable.
>>>> (SVN_CHECK_SWIG, SVN_FIND_SWIG): Remove unused variable SWIG_SUITABLE.
>>>>
>>>> Modified:
>>>> subversion/trunk/build/ac-macros/swig.m4
>>>
>>>
>>> This commit causes a failure on the warnings buildbot:
>>>
>>> https://ci.apache.org/builders/svn-warnings/builds/3186/steps/Configure/logs/stdio
>>>
>>> Swig should not be "required" if it's not mentioned in configure options.
>>
>> Ah, I overlooked the usage of value 'check' for the argument 'where'.
>> I restored the usage of this in r1883722. Thank you.
>>
>> Also, I'll change this behavor to warn only if users correctly specified some
>> of --with-swig-* but swig is not found in "release mode".
>
> Edit r1883719's log message to remove the "no functional change" claim?
Thank you for point it out. That is quite right. Now I've done it.
Thanks,
--
Yasuhito FUTATSUKI <futatuki_at_yf.bsclub.org>
Received on 2020-11-23 07:00:29 CET