Subject: pkg/25057: pkgsrc/lang/perl58 does not compile on non-Mipspro IRIX systems
To: None <gnats-bugs@gnats.NetBSD.org>
From: None <geos@epost.de>
List: pkgsrc-bugs
Date: 04/04/2004 20:57:29
>Number:         25057
>Category:       pkg
>Synopsis:       pkgsrc/lang/perl58 does not compile on non-Mipspro IRIX systems
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    pkg-manager
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Sun Apr 04 20:58:00 UTC 2004
>Closed-Date:
>Last-Modified:
>Originator:     Georg Schwarz
>Release:        pkgsrc current
>Organization:
>Environment:
IRIX lorenz 5.3 08031225 IP20 mips
>Description:
configure fails for pkgsrc/lang/perl58 on IRIX if using another compiler than Mipspro. This is because pkgsrc/lang/perl58/Makefile for IRIX sets the -n32 or -64 compiler option (depending on the ABI setting), which is specific to the Mipspro compilers.
pkgsrc/lang/perl5 should exhibit the same problems.
>How-To-Repeat:

>Fix:
Only use the Mipspro options when actually using Mipspro. The following patch should do the trick:

*** lang/perl58/Makefile.orig   Sun Apr  4 00:04:59 2004
--- lang/perl58/Makefile        Sun Apr  4 02:28:45 2004
***************
*** 82,98 ****
  
  MKPIC?=               yes
  
! .if ${OPSYS} == "IRIX"
  .  if ${ABI} == "32"
  PERL5_CC=             ${CC:T} -n32
  .  elif ${ABI} == "64"
  PERL5_CC=             ${CC:T} -64
- CONFIGURE_ARGS+=      -Duse64bitint
  .  endif
  .else
  PERL5_CC=             ${CC:T}
  .endif
  CONFIGURE_ARGS+=      -Dcc="${PERL5_CC}"
  MAKE_ENV+=            LANG=""
  
  .if ${OPSYS} == "Darwin"
--- 82,100 ----
  
  MKPIC?=               yes
  
! .if !empty(PKGSRC_COMPILER:Mmipspro*)
  .  if ${ABI} == "32"
  PERL5_CC=             ${CC:T} -n32
  .  elif ${ABI} == "64"
  PERL5_CC=             ${CC:T} -64
  .  endif
  .else
  PERL5_CC=             ${CC:T}
  .endif
  CONFIGURE_ARGS+=      -Dcc="${PERL5_CC}"
+ .if ${OPSYS} == "IRIX" && ${ABI} == "64"
+ CONFIGURE_ARGS+=      -Duse64bitint
+ .endif
  MAKE_ENV+=            LANG=""
  
  .if ${OPSYS} == "Darwin"


I hope the second .if statement does not break on non-IRIX systems which do not have set ABI.
>Release-Note:
>Audit-Trail:
>Unformatted: