On 5/11/20, 10:52 AM, "Magnus Ihse Bursie" <magnus.ihse.bur...@oracle.com> wrote:
CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you can confirm the sender and know the content is safe. On 2020-05-11 18:44, Liu, Xin wrote: > Hi, Martin and Magnus, > > Thank you to fix this problem. sorry, I didn't realize that awk program is less portable. > Your script looks good to me. > > Why it works by reordering \._\- in []. Is it the only potential issue? It's an old trick. If you have a range, e.g. a-z, then "-" is interpreted as the range separator. So if you wanted to include a literal "-" in a [] regex set, you had to put it first or last, since then it could not be part of a range. The ability to escape - using \ is much newer, and apparently not supported on AIX. /Magnus Thank you to let me know that! Thanks, --lx > > For your information, the reason I added '-' because I found the suffix '-internal' when you build from openjdk source directly. > "openjdk version "15-internal" 2020-09-15" > > thanks, > --lx > > > On 5/11/20, 9:26 AM, "build-dev on behalf of Magnus Ihse Bursie" <build-dev-boun...@openjdk.java.net on behalf of magnus.ihse.bur...@oracle.com> wrote: > > CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you can confirm the sender and know the content is safe. > > > > On 2020-05-11 18:17, Doerr, Martin wrote: > > > > Hi Magnus, > > > > thank you for proposing a fix which allows building with old awk on > > AIX. Works fine. > > > > Here’s the webrev: > > > > http://cr.openjdk.java.net/~mdoerr/8244756_AIX_fix_awk_expr/webrev.00/ > > > > Would you like to be mentioned as author? > > > Whatever. :) If you want. > > /Magnus > > > > Best regards, > > > > Martin > > > >