Fails to build on ARM

[GoogleCodeExporter]

When build 0.7.2 (and all releases going back to at least 0.5.x) on Fedora 19 
(and earlier) I get the following error when building:

gcc -DHAVE_CONFIG_H -I.    -Wall -pedantic -Wno-overlength-strings   -pthread 
-I/usr/include/PCSC   -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include   
-pthread -I/usr/include/gtk-2.0 -I/usr/lib/gtk-2.0/include 
-I/usr/include/atk-1.0 -I/usr/include/cairo -I/usr/include/gdk-pixbuf-2.0 
-I/usr/include/pango-1.0 -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include 
-I/usr/include/pixman-1 -I/usr/include/freetype2 -I/usr/include/libpng15 
-I/usr/include/libdrm -I/usr/include/harfbuzz   -O2 -g -pipe -Wall 
-Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector 
--param=ssp-buffer-size=4  -march=armv7-a -mfpu=vfpv3-d16  -mfloat-abi=hard -c 
./script.S -o dot_cardpeek.o
./script.S: Assembler messages:
./script.S:27: Error: junk at end of line, first unrecognized character is `,'
make[1]: *** [dot_cardpeek.o] Error 1

A full build log can be found here (choose the latest build as logs are only 
kept a week):
http://arm.koji.fedoraproject.org/koji/packageinfo?packageID=869

Not sure why it needs specific assembler and can't just let the compiler 
optimise.

Original issue reported on code.google.com by pbrobin...@gmail.com on 2 Apr 2013 at 12:47

[GoogleCodeExporter]

Perhaps script.S targets something that is specific to x86/AMD64, which would 
fail on ARM.

script.S is used to embed a binary within cardpeek. I will try to remove the 
use of "script.S" in the next release of cardpeek (0.8) and move to a more 
portable approach using glib-compile-resources presumably.

Original comment by L...@gmx.com on 2 Apr 2013 at 2:34

• Changed state: Started

[GoogleCodeExporter]

I have replaced script.S with a more portable approach (available in the SVN 
repository). This change will appear in the next version of card peek (0.8). I 
didn't have the opportunity to test it on ARM yet, but it should work hopefully.

Original comment by L...@gmx.com on 15 May 2013 at 10:41

• Changed state: Fixed