I prefer the makefile fix, since we don't by policy, make changes to the
imported libraries.
On Jan 23rd [1] I expressed such a tool-chain specific makefile fix
would be fine by me.
Toolchain specific means ideally it would look like what Magnus wrote [2]
Although you said GC 5.4.0 would need to be included in the logic.
If it can be shown to affect current / future versions of gcc then it
could be unqualified.
I think we've just been waiting for a webrev since then ..
-phil.
[1] http://mail.openjdk.java.net/pipermail/2d-dev/2018-January/008855.html
[2]
http://mail.openjdk.java.net/pipermail/build-dev/2018-January/020695.html
On 03/21/2018 09:53 AM, Adam Farley8 wrote:
:)
> Hi Adam,
>
> no problem. I'll open a bug and if necessary find a second reviewer.
Thanks for fixing, maybe I can stop building with warnings disabled on
our s390 machines now.
>
> ..Thomas
>
> > On Wed, Mar 21, 2018 at 5:10 PM, Andrew Leonard
<andrew_m_leon...@uk.ibm.com> wrote:
> > Hi Thomas,
> > I'm a "contributor", but not a "committer", so not on that list, didn't
even know that
> > list existed! I was sort of assuming since it was a trivial change,
and the request was
> > for a review, i'd chip in...!
> > Thanks
> > Andrew
>
> > Andrew Leonard
> > Java Runtimes Development
> > IBM Hursley
> > IBM United Kingdom Ltd
> > Phone internal: 245913, external: 01962 815913
> > internet email: andrew_m_leon...@uk.ibm.com
>
Unless stated otherwise above:
IBM United Kingdom Limited - Registered in England and Wales with
number 741598.
Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU