RE: RFR(XS): 8244756: Build broken with some awk version after JDK-8244248
Thanks for the reviews! Pushed. Best regards, Martin > -Original Message- > From: Baesken, Matthias > Sent: Dienstag, 12. Mai 2020 11:01 > To: build-dev@openjdk.java.net > Cc: Doerr, Martin > Subject: RE: RFR(XS): 8244756: Build broken with some awk version after JDK- > 8244248 > > > Reviewed ! > > Best regards, Matthias > > >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.0 > 0/ > > > > >
RE: RFR(XS): 8244756: Build broken with some awk version after JDK-8244248
Reviewed ! Best regards, Matthias >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/ >
Re: RFR(XS): 8244756: Build broken with some awk version after JDK-8244248
On 5/11/20, 10:52 AM, "Magnus Ihse Bursie" 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" 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 > > > >
Re: RFR(XS): 8244756: Build broken with some awk version after JDK-8244248
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 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" 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 >
Re: RFR(XS): 8244756: Build broken with some awk version after JDK-8244248
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? 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" 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 >
Re: RFR(XS): 8244756: Build broken with some awk version after JDK-8244248
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