package mupdf
tag 590263 patch
thanks

Hi,

I've been using my unofficial mupdf package on armel since 2009. When
I noticed that mupdf is in debian I tried installing it and then noticed

mkdir -p build/release
mkdir -p build/generated
CC build/release/base_cpudep.o
cc1: error: unrecognized command line option "-mmmx"
cc1: error: unrecognized command line option "-msse"
cc1: error: unrecognized command line option "-msse2"
fitz/base_cpudep.c:1: error: bad value (k8) for -march= switch


in 
https://buildd.debian.org/fetch.cgi?pkg=mupdf;ver=0.6-1;arch=armel;stamp=1283843178

The attached patch uses these special compiler options only on amd64
and is tested to fix the build on at least armel.


best regards,
Timo Lindfors
>From b72c53a7a0af115b9a680ce1381bf32ec68fd335 Mon Sep 17 00:00:00 2001
From: Timo Juhani Lindfors <timo.lindf...@iki.fi>
Date: Sat, 11 Sep 2010 13:50:36 +0100
Subject: [PATCH] Use -mmmx -msse -msse2 -march=k8 -DARCH_X86 only on x86_64

---
 Makerules |    6 +++++-
 1 files changed, 5 insertions(+), 1 deletions(-)

diff --git a/Makerules b/Makerules
index a441176..5ff1dc8 100644
--- a/Makerules
+++ b/Makerules
@@ -2,6 +2,7 @@
 
 OS := $(shell uname)
 OS := $(OS:MINGW%=MINGW)
+MACHINE := $(shell uname -m)
 
 LIBS := -ljbig2dec -lopenjpeg -ljpeg -lz -lm
 CFLAGS := -Wall --std=gnu99 -Ifitz -Imupdf $(THIRD_INCS)
@@ -26,10 +27,13 @@ endif
 X11LIBS = -lX11 -lXext
 PDFVIEW_EXE = $(X11VIEW_EXE)
 ifeq  "$(build)" "release"
-CFLAGS += -mmmx -ftree-vectorize -msse -msse2 -march=k8 -DARCH_X86
+CFLAGS += -ftree-vectorize
+ifeq "$(MACHINE)" "x86_64"
+CFLAGS += -mmmx -msse -msse2 -march=k8 -DARCH_X86
 ARCH_SRC = archx86.c
 endif
 endif
+endif
 
 ifeq "$(OS)" "Darwin"
 CFLAGS += -I$(HOME)/include -I/usr/X11R6/include
-- 
1.7.1

Reply via email to