[Libosinfo] [PATCH] Disable static libraries by default
Daniel P. Berrange
berrange at redhat.com
Tue Apr 2 17:40:34 UTC 2013
On Tue, Apr 02, 2013 at 11:23:33AM -0600, Eric Blake wrote:
> On 04/02/2013 10:46 AM, Daniel P. Berrange wrote:
> > From: "Daniel P. Berrange" <berrange at redhat.com>
> >
> > Every source file is currently built twice by libtool, once for
> > the shared library and once for the static library. Static libs
> > are not commonly packaged by distros and slow down compilation
> > time by almost 100% compared to a shared-only build time.
> >
> > Time for non-parallel make
> >
> > shared only: 22 secs
> > shared + static: 40 secs
> >
> > Those few people who really want them, can pass --enable-static
> > to configure
> >
> > Disabling them by default requires use of LT_INIT. We don't need
> > to support older libtool, so drop use of AM_PROG_LIBTOOL entirely
> > ---
> > configure.ac | 3 +--
> > 1 file changed, 1 insertion(+), 2 deletions(-)
> > ---
> > configure.ac | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
>
> Odd double-diffstat. But looks like you are on a roll, doing the same
> cleanup throughout the virt stack :)
>
> > +++ b/configure.ac
> > @@ -5,10 +5,11 @@ AM_INIT_AUTOMAKE([1.11.1 foreign color-tests tar-ustar])
> > AC_PREREQ([2.61])
> > AC_CONFIG_SRCDIR([osinfo/osinfo_db.c])
> > AC_CONFIG_HEADERS([config.h])
> > -AC_PROG_LIBTOOL
> > AC_PROG_CC
> > AC_PROG_LN_S
> >
> > +LT_INIT([shared disable-static win32-dll])
>
> Where did win32-dll come from for this particular configure.ac? We
> weren't previously using AC_LIBTOOL_WIN32_DLL.
>
> ACK if you drop the unneeded option.
Actually, we should have had that all along - we're building DLLs as
part of the autobuild process ! Not sure why we didn't have trouble
without it though !
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|
More information about the Libosinfo
mailing list