[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Help updating swig for octave 4.0.0
From: |
Orion Poplawski |
Subject: |
Re: Help updating swig for octave 4.0.0 |
Date: |
Fri, 10 Jul 2015 12:02:16 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.0.1 |
On 07/10/2015 09:18 AM, Mike Miller wrote:
> On Thu, Jul 09, 2015 at 21:18:21 -0600, Orion Poplawski wrote:
>> Now getting:
>>
>> checking octave testcase default_arg_values
>> error: function delete_B requires at least 1 arguments
>> error: called from
>> ./default_constructor_runme.m at line 17 column 7
>>
>> and several messages like this:
>>
>> checking octave testcase refcount (with run test)
>> refcount_wrap.cxx: In static member function 'static octave_value
>> swig::traits_from_stdseq<Seq, T>::from(const sequence&)':
>> refcount_wrap.cxx:3477:30: error: 'INT_MAX' was not declared in this scope
>> if (size <= (size_type)INT_MAX) {
>> ^
>> ../../../Examples/Makefile:444: recipe for target 'octave_cpp' failed
>
> Sorry, I can't see the generated code or clear fixes for those at the
> moment.
I've made a best guess at a fix. WFM, FWIW.
>
> But I have at least cloned your PR branch now and have built swig. Looks
> like you resolved the INT_MAX issue already. I'll see if I hit the same
> errors as you and see if I can't help you resolve the remaining issues,
> maybe later today or this weekend.
>
> Aside: swig's configure unfortunately doesn't work if you are building
> against a version of Octave whose corresponding mkoctfile executable is
> not called "mkoctfile". I have a local hack to work around that, if I
> get time I may submit a proper fix for that.
>
> In your packages for Fedora do you install /usr/bin/mkoctfile-4.0.0 or
> do you only retain the unversioned name (as we do in Debian)?
We install both.
Thanks for looking at this.
--
Orion Poplawski
Technical Manager 303-415-9701 x222
NWRA, Boulder/CoRA Office FAX: 303-415-9702
3380 Mitchell Lane address@hidden
Boulder, CO 80301 http://www.nwra.com