Hi,

In configure:ac:

dnl add perl cccdlflags when building libraries -- this ensures that the
dnl libraries will be compiled as PIC if perl requires PIC objects
dnl -- this is needed on NetBSD and Linux, but seems to cause problems on 
atleast Solaris --
    case "${target_os}" in
        linux*|netbsd*|freebsd*|dragonfly*)
            AC_MSG_CHECKING(for perl cccdlflags needed on "${target_os}")
            eval `${PERL} -V:cccdlflags`
            PERL_CCCDLFLAGS="$cccdlflags"
            AC_SUBST(PERL_CCCDLFLAGS)
            AC_MSG_RESULT($PERL_CCCDLFLAGS)
            ;;
        *)
            AC_MSG_WARN(skipping check for perl cccdlflags on "${target_os}")
    esac


In addition to NetBSD and Linux noted in the comment, FreeBSD was added to the 
list,
and I requested addition of DragonFly BSD with Bug#3503:
 https://bugzilla.cyrusimap.org/show_bug.cgi?id=3503

From OpenBSD ports, it is also required for OpenBSD:
 
http://cvsweb.openbsd.org/cgi-bin/cvsweb/ports/mail/cyrus-imapd/patches/patch-configure?rev=1.23&content-type=text/x-cvsweb-markup

From pkgsrc bulk build failure on SmartOS (Solaris based OS), it is probably 
required on modern Solaris too.
 
http://us-east.manta.joyent.com/pkgsrc/public/reports/upstream-trunk64/20141224.2239/cyrus-imapd-2.4.17nb10/build.log

So, on what platform does it really cause problem?
I feel that it is too bad to add new platforms to the list one after the other.
How about to change the condition reversely?
  skip on "cause problems" platforms
  add perl cccdlflags by default

What do you think of the suggestion?

--
OBATA Akio / ob...@lins.jp

Reply via email to