← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Re: SHAPE_LINE_CHAIN in swig?

 

I will test later today both options
1. Removing VECTOR2::operator<< or renaming it to str() if it's used.
2. Upgrading to swig 3.0.10 from backports.

Hopefully first is doable and would be transparent for users.
Second one should definitely solve the issue and I feel like compared to
other hoops a user has to jump through to make KiCad compile on debian8
this would not be the worst.

Regards,
Andrew

On Tue, Jul 31, 2018 at 11:32 AM Wayne Stambaugh <stambaughw@xxxxxxxxx>
wrote:

> On 7/31/2018 1:13 PM, Seth Hillbrand wrote:
> >
> >
> > Am Di., 31. Juli 2018 um 07:31 Uhr schrieb Wayne Stambaugh
> > <stambaughw@xxxxxxxxx <mailto:stambaughw@xxxxxxxxx>>:
> >
> >     On 7/31/2018 8:33 AM, Carsten Schoenert wrote:
> >     > Am 31.07.18 um 17:50 schrieb Andrew Lutsenko:
> >     > ...
> >     >> Can swig on the qa machine be updated? Or better yet can you
> >     upgrade to
> >     >> debian 9? Debian 9 has swig 3.0.10 and compiles this just fine.
> >     >> Aside from this debian 8 is very old and should be done away with
> >     anyway
> >     >> because of security, old compilers, etc.
> >     >
> >     > Argumentation by missing security isn't a valid choice, even now
> the
> >     > ELTS team is taking care of security updates, old versions can be
> >     solved
> >     > by using backports, even swig has 3.0.10 in jessie-backports. I
> agree
> >     > that GCC wont become any version updates for Jessie.
> >     >
> >     > But there are still users out there which use Jessie based
> desktops.
> >     >
> >
> >     I'm siding with Carsten on this.  There are people who prefer stable
> >     computing platforms and I want to avoid making kicad only build on
> the
> >     latest distros.  I prefer that we keep as large of a target audience
> as
> >     possible.  How difficult would it be to change the SHAPE_LINE_CHAIN
> >     object (actually its the VECTOR2 object that causes the swig issue)
> so
> >     that older versions of swig don't choke on it?  I would be open to
> that
> >     solution.
> >
> >     Cheers,
> >
> >     Wayne
> >
> >
> > ​I'm not sure I follow the discussion.  I thought Carsten was saying
> > that jessie-backports does have SWIG 3.0.10 and so we can upgrade swig
> > on the kicad-qa​ without changing to a newer debian.
>
> I was operating under the assumption that not every user will track or
> want to track Debian backports so in this case the user would still only
> have the older version of swig.  The line of code that is causing swig
> to choke is the VECTOR2 << operator which I'm almost sure is being used
> for debugging output and therefore could easily be removed without
> issue.  I'm not sure that there are not other swig related issues in the
> SHAPE_LINE_CHAIN implementation this change may not be enough.  If we
> are going to use a version of swig that works with the current code, we
> should set the cmake find package minimum version of swig to the correct
> version.  I'm fine either way.  Others may not be fine with this.
>
> >
> > @Andrew - can you compile your changes on debian 8 using the swig from
> > backports as Carsten described?  If not, then this is moot and we'd need
> > to look at a SWIG-specific VECTOR2, an outcome that might be long-term
> > problematic.
> >
> > -S
>
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp
>

Follow ups

References