[gst-cvs] gstreamer: libgstcheck: Don't use character classes in sed expressions

Jan Schmidt thaytan at kemper.freedesktop.org
Wed Sep 30 09:02:21 PDT 2009


Module: gstreamer
Branch: master
Commit: 274d9854729b37956a0f3916fa0e43fbd97155b3
URL:    http://cgit.freedesktop.org/gstreamer/gstreamer/commit/?id=274d9854729b37956a0f3916fa0e43fbd97155b3

Author: Jan Schmidt <thaytan at noraisin.net>
Date:   Wed Sep 30 15:52:33 2009 +0100

libgstcheck: Don't use character classes in sed expressions

Apparently the sed that ships on Solaris 10 doesn't support character
classes like [:alnum:], so don't use them. We don't need them for the
symbol names that are being extracted anyway.

Also, use $(SED) instead of 'sed'

Fixes: #596877

---

 libs/gst/check/Makefile.am |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/libs/gst/check/Makefile.am b/libs/gst/check/Makefile.am
index 8d39c95..70b2f5f 100644
--- a/libs/gst/check/Makefile.am
+++ b/libs/gst/check/Makefile.am
@@ -90,13 +90,13 @@ LIBGSTCHECK_EXPORTED_SYMBOLS = \
 	$(LIBGSTCHECK_EXPORTED_FUNCS)
 
 print-check-symbols:
-	CHECK_SYMS=`sed -n -e 's/^..*CK_EXPORT[[:space:]][[:space:]]*\([[:alnum:]_][[:alnum:]_]*\)..*$$/\1/p' @top_srcdir@/libs/gst/check/libcheck/check.h.in` ; \
+	CHECK_SYMS=`$(SED) -n -e 's/^..*CK_EXPORT[ ]\{1,\}\([_0-9A-Za-z]\{1,\}\)..*$$/\1/p' @top_srcdir@/libs/gst/check/libcheck/check.h.in` ; \
 	for s in $$CHECK_SYMS; do \
 	  echo $$s; \
 	done;
 
 $(SYMBOLS_FILE):
-	CHECK_SYMS=`sed -n -e 's/^..*CK_EXPORT[[:space:]][[:space:]]*\([[:alnum:]_][[:alnum:]_]*\)..*$$/\1/p' @top_srcdir@/libs/gst/check/libcheck/check.h.in` ; \
+	CHECK_SYMS=`$(SED) -n -e 's/^..*CK_EXPORT[ ]\{1,\}\([_0-9A-Za-z]\{1,\}\)..*$$/\1/p' @top_srcdir@/libs/gst/check/libcheck/check.h.in` ; \
 	rm -f $(SYMBOLS_FILE) 2>/dev/null; \
 	for s in $(LIBGSTCHECK_EXPORTED_SYMBOLS) $$CHECK_SYMS; do \
 	  echo $$s >> $(SYMBOLS_FILE); \





More information about the Gstreamer-commits mailing list