Opened 3 years ago
Closed 3 years ago
#2230 closed defect (fixed)
pkg-oss scripts break if 'so' included in path
Reported by: | https://stackoverflow.com/users/93534/notpeter | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | |
Component: | nginx-package | Version: | |
Keywords: | Cc: | ||
uname -a: | |||
nginx -V: | 1.20.1 |
Description
The nginx pkg-oss repo uses sed to help rename .so to -debug.so in a couple places. For example
for so in `find objs/ -maxdepth 1 -type f -name "*.so"`; do \ debugso=`echo ${so} | sed -e "s|.so|-debug.so|"` ; \ mv ${so} ${debugso} ; \ done
Since this sed regex is unarchored it sadly breaks if your path or module name includes 'so' (e.g. ngx_something, /home/alison, /home/me/source/pkg-oss/, etc).
Proposed patch attached.
Attachments (1)
Change History (2)
by , 3 years ago
Attachment: | nginx_soso.patch added |
---|
comment:1 by , 3 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
Thanks!
Fixed in http://hg.nginx.org/pkg-oss/rev/e7d65b792793.