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)

nginx_soso.patch (3.0 KB ) - added by https://stackoverflow.com/users/93534/notpeter 3 years ago.

Download all attachments as: .zip

Change History (2)

by https://stackoverflow.com/users/93534/notpeter, 3 years ago

Attachment: nginx_soso.patch added

comment:1 by thresh, 3 years ago

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.