On 22.12.2015 04:26, James McCoy wrote:
> On Sun, Nov 01, 2015 at 01:17:56AM -0500, James McCoy wrote:
>> On Mon, Jul 13, 2015 at 11:00:17AM +0100, Joe Orton wrote:
>>> On Mon, Jul 13, 2015 at 11:41:23AM +0200, Branko Čibej wrote:
>>>> https://ci.apache.org/builders/svn-x64-ubuntu-gcc/builds/317
>>>>
>>>> Apparently this breaks older versions of Swig ...
>>> Ah, ugh... yes I should have realised this. Sorry.
>>>
>>> I think the choices are:
>>>
>>> a) keep comments in that code & use {}, only support SWIG < 3.0
>>> b) keep comments & use %{/%}, only support SWIG > 3.0.5
>>> c) drop the comments from that code, support SWIG < 3.0 && > 3.0.5
>>>
>>> I'm guessing (c) is preferred?
> It seems like using %{/%} and adding SWIG >= 3.0.6 as supported versions
> should work. I re-applied r1690591 along with Joe's proposed patch for
> SWIG version checking and was able to build with both swig 2.0.7 & swig
> 3.0.7.
>
> The revert of r1690591 mentioned that older SWIG versions didn't work,
> but the CI build log isn't available anymore. What was the problem and
> what was the SWIG versions?
IIRC, anything newer than 3.0.2 would fail no matter what I did.
-- Brane
Received on 2015-12-22 14:45:18 CET