[prev in list] [next in list] [prev in thread] [next in thread] 

List:       cyrus-devel
Subject:    Who dislike PIC object for perl?
From:       "OBATA Akio" <obata () lins ! jp>
Date:       2014-12-26 6:23:02
Message-ID: op.xrgltgl6csw9i4 () cabos ! lins ! jp
[Download RAW message or body]

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 / obata@lins.jp


[prev in list] [next in list] [prev in thread] [next in thread]