>Number:         33769
>Category:       pkg
>Synopsis:       multimedia/mjpegtools fails to build under amd64 (includes fix)
>Confidential:   no
>Severity:       serious
>Priority:       high
>Responsible:    pkg-manager
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Mon Jun 19 19:55:00 +0000 2006
>Originator:     martijnb@atlas.ipv6.stack.nl
>Release:        NetBSD 3.99.21
>Organization:
	
>Environment:
	
	
System: NetBSD atlas.ipv6.stack.nl 3.99.21 NetBSD 3.99.21 (ATLAS) #0: Tue Jun 13 23:23:19 CEST 2006 martijnb@atlas.ipv6.stack.nl:/usr/obj/sys/arch/amd64/compile/ATLAS amd64
Architecture: x86_64
Machine: amd64
>Description:
	multimedia/mjpegtools fails to build under current/amd64 because of 
assembler errors due to a wrong CPU detection. mjpegtools assumes that on an
amd64 platform __LP64___ is defined, which turns out not to be the case. 
There appears to be some confusion as to whether code should check for
__LP64__ or _LP64.
 
>How-To-Repeat:
	try to build multimedia/mjpegtools on a (current ?) amd64
>Fix:
The pragmatic approach is to check for either __LP64__ or _LP64, hoping that
at least one of these is set.

$NetBSD$

--- utils/cpu_accel.c.orig	2005-03-31 23:20:51.000000000 +0200
+++ utils/cpu_accel.c
@@ -117,7 +117,7 @@ static int x86_accel (void)
 	/* Slightly weirdified cpuid that preserves the ebx and edi required
 	   by gcc for PIC offset table and frame pointer */
 
-#ifdef __LP64__
+#if defined(__LP64__) || defined(_LP64)
 #  define REG_b "rbx"
 #  define REG_S "rsi"
 #else

>Unformatted: