Re: [Simulavr-devel] [bug #42595] Faulty swig version number comparison

[ThomasK]

Hi Ryan,

this is a problem (or maybe a feature) of autotools scripts, which we 
use too in our project. The script compare major and minor version (e.g. 
1 and 3) as EQUAL and only the patchlevel (here 18) is compared to equal 
or greater. For the rel-1.0 branch (which you use with version 1.0.0) 
you have to have swig in version 1.3.x where x can be equal or greater 
18. In current trunk (master branch) we check at first for 2.0.x and 
then, as fallback, for 1.3.x.

If you want, there is a quickfix: change configure.ac file and there the 
line "AC_PROG_SWIG(1.3.18)" to "AC_PROG_SWIG(3.0.0)". (and then 
./bootstrap again!) Then swig in version 3.0.x is accepted on configure.

But the next question: does it work (in best case without changes) for 
swig 3.0.2? A change in major and minor version implies normally a 
change, which could break the build against the "old" swig files. And 
the current linux distros are on 2.0.x. So we doesn't support 3.0.x in 
the moment! (which means: nobody knows, if it works or not!)

So you can try it (with the quickfix) and, if you like, report the 
result here. (use ./configure; make; make check)

cu, Thomas


Am 22.06.2014 09:00, schrieb Ryan Schmidt:
> URL:
>    <http://savannah.nongnu.org/bugs/?42595>
>
>                   Summary: Faulty swig version number comparison
>                   Project: Simulavr: an AVR simulator
>              Submitted by: ryandesign
>              Submitted on: Sun 22 Jun 2014 07:00:53 AM GMT
>                  Category: Configure
>                  Severity: 3 - Normal
>                Item Group: None
>                    Status: None
>                   Privacy: Public
>               Assigned to: None
>          Originator Email: address@hidden
>               Open/Closed: Open
>           Discussion Lock: Any
>         Component Version: simulavr
>
>      _______________________________________________________
>
> Details:
>
> The code in simulavr 1.0.0 that checks if the swig version is new enough is
> faulty, resulting in the following error
>
>
> SWIG version >= 1.3.18 is required.  You have 3.0.2.
>
>
> as reported to the MacPorts project here:
> https://trac.macports.org/ticket/44108
>
> The problem seems to be that it checks each digit of the two version numbers
> against each other without regard for the other digits. That is, it compares
> the first digits: is 3 less than 1? no? then move to the second digits: is 0
> less than 3? yes, so the swig version must be too old.
>
> The version comparison needs to be done correctly to fix this problem.
>
>
>
>
>      _______________________________________________________
>
> Reply to this item at:
>
>    <http://savannah.nongnu.org/bugs/?42595>
>
> _______________________________________________
>    Message sent via/by Savannah
>    http://savannah.nongnu.org/
>
>
> _______________________________________________
> Simulavr-devel mailing list
> address@hidden
> https://lists.nongnu.org/mailman/listinfo/simulavr-devel