libtool-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Small patch for the QNX platform


From: Ralf Wildenhues
Subject: Re: Small patch for the QNX platform
Date: Sun, 14 Jun 2009 11:46:17 +0200
User-agent: Mutt/1.5.19 (2009-06-13)

Oh my, what an unfinished old thread, sorry for forgetting about it:
<http://thread.gmane.org/gmane.comp.gnu.libtool.patches/9063/focus=9172>

* Ralf Wildenhues wrote on Sun, Feb 01, 2009 at 04:24:30PM CET:
> * Mike Gorchak wrote on Fri, Jan 23, 2009 at 07:09:18AM CET:
> > This patch adds qnx as platform which supports library version information.
> >
> > --- ./libtool-2.2.6/libltdl/config/ltmain.m4sh 2008-09-07 
> > 20:54:53.000000000 +0300
> > +++ ./libtool-2.2.6-qnx/libltdl/config/ltmain.m4sh 2009-01-23  
> > 08:01:41.000000000 +0200
> > @@ -5746,7 +5746,7 @@
> >    # which has an extra 1 added just for fun
> >    #
> >    case $version_type in
> > -   darwin|linux|osf|windows|none)
> > +   darwin|linux|osf|windows|qnx|none)
> >      func_arith $number_major + $number_minor
> >      current=$func_arith_result
> >      age="$number_minor"
> 
> Thinking about this a bit more, and working on versioning.at, I gather
> the above change is wrong.  Well, at least it is inconsistent with the
> rest of the qnx version_type:
>         qnx)
>           major=".$current"
>           versuffix=".$current"
> 
> This means that your major is not CURRENT - AGE, so we should not try to
> compute current as MAJOR + MINOR.  At least that's what the comment
> before your change indicates:

As indicated, I'm applying this patch instead.  The versioning tests,
also from that old thread, will expose this issue with your version of
the patch.

Cheers,
Ralf

2009-06-14  Mike Gorchak  <address@hidden>  (tiny change)
            Ralf Wildenhues  <address@hidden>

        Fix versioning computation for QNX.
        * libltdl/config/ltmain.m4sh (func_mode_link): Set `$current'
        and `$age' properly for version_type `qnx'.

diff --git a/libltdl/config/ltmain.m4sh b/libltdl/config/ltmain.m4sh
index 02f4d21..6f44d35 100644
--- a/libltdl/config/ltmain.m4sh
+++ b/libltdl/config/ltmain.m4sh
@@ -5928,7 +5928,7 @@ func_mode_link ()
            age="$number_minor"
            revision="$number_revision"
            ;;
-         freebsd-aout|freebsd-elf|sunos)
+         freebsd-aout|freebsd-elf|qnx|sunos)
            current="$number_major"
            revision="$number_minor"
            age="0"




reply via email to

[Prev in Thread] Current Thread [Next in Thread]