Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(801)

Unified Diff: ports/libvpx/nacl.patch

Issue 1804293003: Add support for new pnacl native toolchain driver (Closed) Base URL: https://chromium.googlesource.com/webports.git@master
Patch Set: Fix according to Brad's comments Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ports/libvpx/nacl.patch
diff --git a/ports/libvpx/nacl.patch b/ports/libvpx/nacl.patch
new file mode 100644
index 0000000000000000000000000000000000000000..0218e0d5faee2ad38fe955c780c859a3e49e0b57
--- /dev/null
+++ b/ports/libvpx/nacl.patch
@@ -0,0 +1,24 @@
+diff --git a/build/make/Makefile b/build/make/Makefile
+--- a/build/make/Makefile
++++ b/build/make/Makefile
+@@ -207,6 +207,8 @@ $(BUILD_PFX)%.asm.s: %.asm
+ # If we're in debug mode, pretend we don't have GNU strip, to fall back to
+ # the copy implementation
+ HAVE_GNU_STRIP := $(if $(CONFIG_DEBUG),,$(HAVE_GNU_STRIP))
++# We don't have STRIP in new pnacl driver so fall back to using cp
++HAVE_GNU_STRIP := $(if $(filter $(NACL_ARCH), le32),,$(HAVE_GNU_STRIP))
+ ifeq ($(HAVE_GNU_STRIP),yes)
+ # Older binutils strip global symbols not needed for relocation processing
+ # when given --strip-unneeded. Using nm and awk to identify globals and
+diff --git a/build/make/configure.sh b/build/make/configure.sh
+--- a/build/make/configure.sh
++++ b/build/make/configure.sh
+@@ -428,7 +428,7 @@ NM=${NM}
+
+ CFLAGS = ${CFLAGS}
+ CXXFLAGS = ${CXXFLAGS}
+-ARFLAGS = -rus\$(if \$(quiet),c,v)
++ARFLAGS = rus\$(if \$(quiet),c,v)
+ LDFLAGS = ${LDFLAGS}
+ ASFLAGS = ${ASFLAGS}
+ extralibs = ${extralibs}
« make_all.sh ('K') | « ports/libmatroska/build.sh ('k') | ports/libyuv/build.sh » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698