Re: pkg/43147 (security/gnupg: no optimiztion for pentium)

[Steven Drake <sbd%NetBSD.org@localhost>]

The following reply was made to PR pkg/43147; it has been noted by GNATS.

From: Steven Drake <sbd%NetBSD.org@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: 
Subject: Re: pkg/43147 (security/gnupg: no optimiztion for pentium)
Date: Thu, 15 Apr 2010 15:58:06 +1200 (NZST)

 I think this might show the problem:
 
 $ make show-vars VARNAMES="LOWER_ARCH MACHINE_GNU_ARCH MACHINE_GNU_PLATFORM"
 i386
 i586
 i386-pc-linux
 
 It comes down to the fact that MACHINE_GNU_PLATFORM is set from LOWER_ARCH
 not MACHINE_GNU_ARCH.  LOWER_ARCH is set to MACHINE_GNU_ARCH if is not 
 already set (which is not on NetBSD, but on Linux it is!)
 
 Either set LOWER_ARCH=i586 in security/gnupg/options.mk or set 
 MACHINE_GNU_PLATFORM from MACHINE_GNU_ARCH, which makes a lot more sense!
 
 Index: bsd.prefs.mk
 ===================================================================
 RCS file: /srv/netbsd-cvs/pkgsrc/mk/bsd.prefs.mk,v
 retrieving revision 1.307
 diff -u -p -r1.307 bsd.prefs.mk
 --- bsd.prefs.mk       6 Feb 2010 11:40:22 -0000       1.307
 +++ bsd.prefs.mk       11 Apr 2010 11:30:53 -0000
 @@ -287,8 +287,8 @@ NATIVE_MACHINE_ARCH:=              ${MACHINE_ARCH}
  
  NATIVE_MACHINE_PLATFORM?=     ${OPSYS}-${OS_VERSION}-${NATIVE_MACHINE_ARCH}
  MACHINE_PLATFORM?=            ${OPSYS}-${OS_VERSION}-${MACHINE_ARCH}
 -NATIVE_MACHINE_GNU_PLATFORM?= 
${NATIVE_LOWER_ARCH}-${LOWER_VENDOR}-${LOWER_OPSYS:C/[0-9]//g}${APPEND_ELF}${LOWER_OPSYS_VERSUFFIX}
 -MACHINE_GNU_PLATFORM?=                
${LOWER_ARCH}-${LOWER_VENDOR}-${LOWER_OPSYS:C/[0-9]//g}${APPEND_ELF}${LOWER_OPSYS_VERSUFFIX}
 +NATIVE_MACHINE_GNU_PLATFORM?= 
${NATIVE_MACHINE_GNU_ARCH}-${LOWER_VENDOR}-${LOWER_OPSYS:C/[0-9]//g}${APPEND_ELF}${LOWER_OPSYS_VERSUFFIX}
 +MACHINE_GNU_PLATFORM?=                
${MACHINE_GNU_ARCH}-${LOWER_VENDOR}-${LOWER_OPSYS:C/[0-9]//g}${APPEND_ELF}${LOWER_OPSYS_VERSUFFIX}
  
  # Needed to prevent an "install:" target from being created in bsd.own.mk.
  NEED_OWN_INSTALL_TARGET=no
 
 -- 
 Steven