Created attachment 199505 [details]
svn(1) diff against net/miniupnpc
The install bits of this port attempt this: sed 's/\(.*MINIUPNPC_API_VERSION\s\+\)[0-9]\+/\117/'
There are two problems of attempting to do this with our sed:
1.) \s is a GNU extension; our sed will pass this through as an escaped \s and regex(3) will interpret it as an escaped ordinary 's'.
2.) \117 is likely expecting character represented by \117 to be inserted, but this will not behave accordingly
Fix this by plopping gsed in place of sed for this port. #1 will be explicitly disallowed in a future version of regex(3), making this ideal for now.
Created attachment 199566 [details]
Fix sed(1) usage
2.) \117 is not a byte but \1$(APIVERSION)
3.) \+ is another GNU extension not understood by BSD regex(3)
Assign to the maintainer.
With the second patch applied, do we still need gsed?
Probably not. I've tested attachment 199566 [details] against libc patch from bug 229925.
A commit references this bug:
Date: Mon Nov 26 09:43:29 UTC 2018
New revision: 485935
- fix sed(1) usage
Submitted by: Jan Beich
Reported by: Kyle Evans