[Octave-bug-tracker] [bug #60413] build: sed expression problems when prefix is empty or slash

[Markus Mützel]

Follow-up Comment #7, bug #60413 (project octave):

Along the lines that Mike suggested in comment #0, maybe, we could change the
sed command to something like this:

if test -n "$prefix"; then echo "$var" | sed -E "s%(-L|-I| )${prefix}([/\\
]|$)%\1\$\{prefix\}\2%g"; else echo "$var"; fi


IIUC, that would only replace anything in the respective variable ($var in the
example) if the prefix is non-empty and the prefix is preceded by "-I", "-L"
or " " in the variable and it is followed by "/", "\" or " " or ends with the
prefix.

An example in subst-config-vals.in.h could maybe look like:

if test -n "@prefix@"; then echo '@CPPFLAGS@' | sed -E "s%(-L|-I|
)@prefix@([/\\ ]|$)%\1\$\{prefix\}\2%g"; else echo '@CPPFLAGS@'; fi


Does "sed -E" work on all platforms we'd like to support (GNU, BSD, Mac, ...)?
Fwiw, "sed -r" seems to work similarly for me with GNU sed 4.7 if that should
be better supported cross-platform.

    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?60413>

_______________________________________________
  Message sent via Savannah
  https://savannah.gnu.org/