Re: RFR(S): 8141290: AIX: fix build after '8140661: Rename LDFLAGS_SUFFIX to LIBS'

2015-11-04 Thread Magnus Ihse Bursie
On 2015-11-04 12:04, Volker Simonis wrote: Hi Magnus, thanks for checking, but here comes the new (and improved :) version: http://cr.openjdk.java.net/~simonis/webrevs/2015/8141290_toplevel/ http://cr.openjdk.java.net/~simonis/webrevs/2015/8141290_jdk/ Looks good. In the toplevel change I've

Re: RFR(S): 8141290: AIX: fix build after '8140661: Rename LDFLAGS_SUFFIX to LIBS'

2015-11-04 Thread Magnus Ihse Bursie
On 2015-11-03 18:20, Magnus Ihse Bursie wrote: On 2015-11-03 17:20, Volker Simonis wrote: Hi, unfortunately, change '8140661: Rename LDFLAGS_SUFFIX to LIBS' broke the AIX build. I'm sorry for that. :-( Can somebody please review this small change which fixes the build?

Re: RFR(S): 8141290: AIX: fix build after '8140661: Rename LDFLAGS_SUFFIX to LIBS'

2015-11-04 Thread Volker Simonis
On Tue, Nov 3, 2015 at 6:20 PM, Magnus Ihse Bursie wrote: > On 2015-11-03 17:20, Volker Simonis wrote: >> >> Hi, >> >> unfortunately, change '8140661: Rename LDFLAGS_SUFFIX to LIBS' broke >> the AIX build. > > > I'm sorry for that. :-( >> >> >> Can somebody please

Re: RFR(S): 8141290: AIX: fix build after '8140661: Rename LDFLAGS_SUFFIX to LIBS'

2015-11-04 Thread Volker Simonis
Hi Magnus, thanks for checking, but here comes the new (and improved :) version: http://cr.openjdk.java.net/~simonis/webrevs/2015/8141290_toplevel/ http://cr.openjdk.java.net/~simonis/webrevs/2015/8141290_jdk/ In the toplevel change I've simply removed '-liconv' from JDKLIB_LIBS on AIX. In the

Re: RFR(S): 8141290: AIX: fix build after '8140661: Rename LDFLAGS_SUFFIX to LIBS'

2015-11-03 Thread Magnus Ihse Bursie
On 2015-11-03 17:20, Volker Simonis wrote: Hi, unfortunately, change '8140661: Rename LDFLAGS_SUFFIX to LIBS' broke the AIX build. I'm sorry for that. :-( Can somebody please review this small change which fixes the build? http://cr.openjdk.java.net/~simonis/webrevs/2015/8141290/