[PATCH] Add version info to the shared libs

[Mikko Levonmaa]

Yes, true. Still to me having a meaningful version number in the actual
library files does provide value.

Also the manual does suggest that a -version-info is provided if we want to
use libtool's versioning system,
albeit they discourage the direct relation to the actual package version
name.

http://www.gnu.org/software/libtool/manual/html_node/Updating-version-info.html#Updating-version-info

Perhaps we should consider adding a WAYLAND_API_VERSION and have that be
inserted into the file name, how does
that sound?

-m



On Thu, Nov 8, 2012 at 8:24 AM, Kristian Høgsberg <hoegsberg at gmail.com>wrote:

> On Tue, Nov 06, 2012 at 03:57:47PM -0800, Mikko Levonmaa wrote:
> > Adds the wayland version information to the shared libraries.
> > libwayland-server.so.0.0.0 -> libwayland-server.so.1.0.0
>
> The libtool versioning is different from the pacakge version number.
> See:
>
>
> http://www.gnu.org/software/libtool/manual/html_node/Libtool-versioning.html
>
> Kristian
>
> > Signed-off-by: Mikko Levonmaa <mikko.levonmaa at gmail.com>
> > ---
> >  src/Makefile.am |    2 ++
> >  1 files changed, 2 insertions(+), 0 deletions(-)
> >
> > diff --git a/src/Makefile.am b/src/Makefile.am
> > index f93954e..0a799f3 100644
> > --- a/src/Makefile.am
> > +++ b/src/Makefile.am
> > @@ -19,6 +19,7 @@ libwayland_util_la_SOURCES =                        \
> >       wayland-private.h
> >
> >  libwayland_server_la_LIBADD = $(FFI_LIBS) libwayland-util.la -lrt -lm
> > +libwayland_server_la_LDFLAGS   = -version-info @WAYLAND_VERSION_MAJOR@
> :@WAYLAND_VERSION_MINOR@:@WAYLAND_VERSION_MICRO@
> >  libwayland_server_la_SOURCES =                       \
> >       wayland-protocol.c                      \
> >       wayland-server.c                        \
> > @@ -27,6 +28,7 @@ libwayland_server_la_SOURCES =                      \
> >       event-loop.c
> >
> >  libwayland_client_la_LIBADD = $(FFI_LIBS) libwayland-util.la -lrt -lm
> > +libwayland_client_la_LDFLAGS   = -version-info @WAYLAND_VERSION_MAJOR@
> :@WAYLAND_VERSION_MINOR@:@WAYLAND_VERSION_MICRO@
> >  libwayland_client_la_SOURCES =                       \
> >       wayland-protocol.c                      \
> >       wayland-client.c
> > --
> > 1.7.4.1
> >
> > _______________________________________________
> > wayland-devel mailing list
> > wayland-devel at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/wayland-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20121109/ee09ed22/attachment-0001.html>