|
From: Greg P. <gp...@ap...> - 2009-01-07 22:18:05
|
On Jan 7, 2009, at 1:28 PM, Greg Parker wrote: > On Jan 7, 2009, at 1:24 PM, Greg Parker wrote: >> 3. Makefile.install.am: installing libcoregrind.a and libvex.a. For >> some reason noinst_LIBRARIES is empty here, so I don't know whether >> the expressions work or not. (For whom would libcoregrind.a and >> libvex.a be installed anyway?) > > Never mind, noinst_LIBRARIES is set and #3 is failing, but it's a > syntax error in `expr` not a sed problem. Easy fix: use `expr A : B` instead of `expr match A B`. The latter appears to be a GNU-ism. --- Makefile.install.am (revision 1583) +++ Makefile.install.am (working copy) @@ -33,7 +33,7 @@ fi ; \ if [ -n "$(noinst_LIBRARIES)" ] ; then \ for f in $(noinst_LIBRARIES) expr_wont_match_me; do \ - if expr match $$f libcoregrind_ > /dev/null ; then \ + if expr $$f : libcoregrind_ > /dev/null ; then \ pU=`echo $$f | sed -e 's/libcoregrind_//g' -e 's/\.a//g'` ; \ pD=`echo $$pU | sed -e 's/_/-/g'` ; \ $(INSTALL_DATA) $$f $(DESTDIR)$(valdir)/$$pD/libcoregrind.a ; \ -- Greg Parker gp...@ap... Runtime Wrangler |