Opened 4 years ago
Closed 4 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 , 4 years ago
| Attachment: | nginx_soso.patch added |
|---|
comment:1 by , 4 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.