Sam Varshavchik writes:
> 
> Alessandro Vesely writes:
> 
> > replacing
> > "-Wl,--whole-archive" with "-Wl,-z -Wl,allextract", and
> > "-Wl,--no-whole-archive" with "-Wl,-z -Wl,defaultextract".
> >
> > That's a job for libtool, as (not much extensively) exemplified in
> > http://www.gnu.org/software/automake/manual/html_node/Libtool-Convenience-Libraries.html
> > Doesn't that work?
> 
> No, libtool is not flexible enough to do what I want it to do here.

IMHO, it should be. At [email protected] they may have different opinions,
but it may be worth checking it out before reinventing any wheels.

> But, it shouldn't be too difficult to insert a configuration option that
> supplies the Solaris-specific linker flags.

Alternatively, one could test if ld -V prints, e.g., `*Solaris*',
similarly to what AC_PROG_LD_GNU does for setting lt_cv_prog_gnu_ld='no'.

Sometimes, including Solaris headers causes errors if CFLAGS don't
contain, say, "-D_XOPEN_SOURCE=600". If you are going to handle
Solaris specially, you should at least mention that macro in INSTALL
(and invite the reader to study `man standards' to pick up a reasonable
value for it.)

> I'm not going to touch the tarballs now.  After the release I'll quickly go
> back and add that code.

I'm sorry I didn't find the time to check it before.

> If this is your only problem then that's not really that bad.
 
I've not done testing yet. Right now I only have a few comments
of minor-to-no importance. I paste them below in case you care
flipping through them.

-----
http://www.courier-mta.org/download.php

   "You do not need to download anything else if you download this package"

is not true. Should mention authlib.

-----
courier-authlib-0.51/README

I'm not good in English, thus if I were not familiar with Courier capabilities
I'd be puzzled by

  "All Courier components that use this authentication library
  "can implement authentication using any of the above possibilities

Perhaps the following is clearer? (just an idea)

  "The Courier applications mentioned above can be configured at run
  "time to use one or more of the authentcation services compiled
  "into this library.

-----
html file names

Apache treats README* files specially when displaying directories.
That's good for README.html, but the other READMEs are not shown.

-----
configure.in

AC_CHECK_HEADERS fails on ldap.h
Solaris man page includes <ldap.h> after <lber.h>, as authldaplib.c does.
Not doing so, as the header checker does, results in a compile error
(/usr/include/ldap.h:494: field `ldctl_value' has incomplete type)
which in turn produces a multiline warning in configure.
Inverting the order the headers are checked doesn't appear to fix it.

-----
bdobj.h.in

--without-userdb defines no dbobj at all, therefore compilation fails.


-------------------------------------------------------
The SF.Net email is sponsored by: Beat the post-holiday blues
Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek.
It's fun and FREE -- well, almost....http://www.thinkgeek.com/sfshirt
_______________________________________________
courier-users mailing list
[email protected]
Unsubscribe: https://lists.sourceforge.net/lists/listinfo/courier-users

Reply via email to