On Thu, May 17, 2012, at 01:21 AM, Дилян Палаузов wrote:
> Hello,
>
> I have integrated the comments about libtool/shared libraries and at the
> end the changes are:
>
> -- in libcyrus_min:libconfig:config_read one parameter is added for
> config_need_data, so that this variable does not have to be global and
> presented in programs, not using config_read, update all invocations of
> the function to use the new API
> -- in libimap:global:cyrus_init add one parameter (config_need_data),
> so that its value can be passed to config_read; update all invocations
> of cyrus_init;
> -- create by default only shared libraries, this can be further tuned
> with ./configure --enable-static / --disable-shared
> -- rename libsieve and libimap to libcyrus_sieve and libcyrus_sieve
> and install them in $(libdir) (e.g. /usr/lib)
> -- make the perl/{imap,sieve/managesieve} modules use the shared
> libraries libcyrys_min and libcyrus. This might lead to a problem, when
> the tests in perl/imap/t/ are run, before the shared libraries are
> installed, as the tests depend on the shared libraries, but the tests
> are not instructed to where to find thw libraries.
> -- when there is no system-wide libcom_err, compile the bundled one
> as libcyrus_com_err and eventually install it in $(libdir)
commit "Makefile.am: remove LD_(BASIC,SERVER,SIEVE,UTILITY)_FLAGS"
commit "Makefile.am: remove LD_SIEVE_FLAGS"
commit "timsieved/scripttest.c: mark internal functions with "static""
commit "Makefile.am: build doc/text/htmlstrip only on "make dist""
This all looks good
commit "Makefile.am: move the defintion of COM_ERR_LIB here"
I don't think there's any point having COM_ERR_LIB and COM_ERR_LIBS,
just one should do, but we can tweak that later.
commit "libcyrus_min:libconfig:config_read: add parameter
config_need_data"
commit "libimap:global:cyrus_init add parameter config_need_data"
I think this could be done more cleanly, but it does solve the immediate
problem.
commit "notifyd/notifytest: add a function fatal"
commit "configure.ac, Makefile.am: Add libtool support"
commit "/.gitignore: update to ingore .la, .lo and .libs/ files"
commit "convert lib/libcyrus_min.a to libtool archive"
commit "convert lib/libcyrus.a to libtool archive"
commit "convert com_err/et/libcom_err.a to a libtool archive"
commit "convert sieve/libsieve.a to a libtool archive"
commit "convert imap/libimap.a to a libtool archivedev"
Looks good, except that the Perl build is broken. I get this:
make[2]: Entering directory `/home/gnb/software/cyrus/imapd/perl/imap'
make[2]: *** No rule to make target `-lcyrus', needed by `subdirs'.
Stop.
make[2]: *** Waiting for unfinished jobs....info
I've been wrestling with this most of the day but libtool + MakeMaker =
one enormous headache :(
Also, is there a way of getting rid of these warnings:
/bin/bash ./libtool --mode=install /usr/bin/install -c imtest/imtest
'/home/gnb/software/cyrus/inst/usr/cyrus/bin'
libtool: install: warning: `lib/libcyrus.la' has not been installed in
`/usr/cyrus/lib'
libtool: install: warning: `lib/libcyrus_min.la' has not been installed
in `/usr/cyrus/lib'
because there's quite a lot of them and they're confusing the script I
use to find compile time warnings.
> I hope with uploading now the separate branch "dev/libtool" on
> git.cyrusimapd.org I will not mess up the things again.
I think you should go ahead and merge it in once we can get the Perl
code building . There's a few problems remaining after that but nothing
we can't handle once we know about it. In other news I've tweaked
Cassandane to set up $LD_LIBRARY_PATH so it will be able to run Cyrus
binaries built for shared libraries.
--
Greg.
