OK, maybe convert this to python script would be a better option. Then skip this first
On Tue, Aug 25, 2020 at 9:59 PM Eric Blake <ebl...@redhat.com> wrote: > On 8/25/20 4:11 AM, 罗勇刚(Yonggang Luo) wrote: > > Hi, works, msys2 sh compiled and run > > Top-formatting makes it harder to read, compared to interleaving your > responses. > > msys2 sh _is_ bash, so that's not answering the question Dan asked. > > > >>> +++ b/scripts/undefsym.sh > >>> @@ -1,4 +1,4 @@ > >>> -#! /usr/bin/env bash > >>> +#!/bin/sh > >> > >> Does this script actually work on non-bash shells ? If not, then this > >> change will likely break on plaforms where /bin/sh is not bash. > > A quick look at the script sees: > > comm -12 \ > <( $NM -P -g $staticlib | awk '$2!="U"{print "-Wl,-u," $1}' | sort -u) \ > <( $NM -P -g "$@" | awk '$2=="U"{print "-Wl,-u," $1}' | sort -u) > > > <() is a bash'ism, so you _are_ breaking things on platforms like Debian > where /bin/sh is dash rather than bash. > > > -- > Eric Blake, Principal Software Engineer > Red Hat, Inc. +1-919-301-3226 > Virtualization: qemu.org | libvirt.org > > -- 此致 礼 罗勇刚 Yours sincerely, Yonggang Luo