Fix iOS builds with assembly.
authorTimothy B. Terriberry <tterribe@xiph.org>
Wed, 26 Mar 2014 19:58:04 +0000 (12:58 -0700)
committerTimothy B. Terriberry <tterribe@xiph.org>
Wed, 26 Mar 2014 19:58:04 +0000 (12:58 -0700)
The patch in 76e831d917ff got us most of the way there, but
 out-of-tree builds required a second Makefile.am rule, which was
 missing @ARM2GNU_PARAMS@.
Also, the arm2gnu.pl was terminating argument processing on any
 argument beginning with --, rather than an argument that was just
 -- by itself (as is the normal convention in GNU programs).
That meant it never saw the --apple flag even when it was passed.

Thanks to Jonathan Lennox for the report and for testing.

Makefile.am
celt/arm/arm2gnu.pl

index e76b204..fac7651 100644 (file)
@@ -228,4 +228,4 @@ $(CELT_SOURCES_ARM_ASM:%.s=%-gnu.S): $(top_srcdir)/celt/arm/arm2gnu.pl
        $(top_srcdir)/celt/arm/arm2gnu.pl @ARM2GNU_PARAMS@ < $< > $@
 # For autoconf-modified sources (e.g., armopts.s)
 %-gnu.S: %.s
-       $(top_srcdir)/celt/arm/arm2gnu.pl < $< > $@
+       $(top_srcdir)/celt/arm/arm2gnu.pl @ARM2GNU_PARAMS@ < $< > $@
index 54f7129..6c922ac 100755 (executable)
@@ -36,12 +36,12 @@ eval 'exec /usr/local/bin/perl -S $0 ${1+"$@"}'
 
 while ($ARGV[0] =~ /^-/) {
     $_ = shift;
-  last if /^--/;
-    if (/^-n/) {
+  last if /^--$/;
+    if (/^-n$/) {
     $nflag++;
     next;
     }
-    if (/^--apple/) {
+    if (/^--apple$/) {
         $apple = 1;
         $symprefix = "_";
         next;