On Thu, Jan 19, 2023 at 07:59:43AM +0100, Markus Armbruster wrote: > When a symbolic link points to a file that needs cleaning, the script > replaces the link with a cleaned regular file. Not wanted; skip them. > > We have a few symbolic links under subprojects/libvduse/ and > subprojects/libvhost-user/. > > Signed-off-by: Markus Armbruster <arm...@redhat.com> > --- > scripts/clean-includes | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/scripts/clean-includes b/scripts/clean-includes > index 8e8420d785..f0466a6262 100755 > --- a/scripts/clean-includes > +++ b/scripts/clean-includes > @@ -113,6 +113,10 @@ EOT > > files= > for f in "$@"; do > + if [ -L "$f" ]; then
I don't see -L used with test very often, but POSIX requires it, so it is safe for our choice of /bin/sh. Reviewed-by: Eric Blake <ebl...@redhat.com> -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3266 Virtualization: qemu.org | libvirt.org