RFR(XXS): 8185333: [s390] Fix build without cds

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

RFR(XXS): 8185333: [s390] Fix build without cds

Lindenmaier, Goetz
Hi,

Could I please get reviews for this tiny fix?
http://cr.openjdk.java.net/~goetz/wr17/8185333-fixCDSs390/webrev.01/

This is required to build s390 without cds support.

Best regards,
  Goetz.
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: RFR(XXS): 8185333: [s390] Fix build without cds

Andrew Haley
On 26/07/17 10:33, Lindenmaier, Goetz wrote:
> Could I please get reviews for this tiny fix?
> http://cr.openjdk.java.net/~goetz/wr17/8185333-fixCDSs390/webrev.01/
>
> This is required to build s390 without cds support.

OK.

--
Andrew Haley
Java Platform Lead Engineer
Red Hat UK Ltd. <https://www.redhat.com>
EAC8 43EB D3EF DB98 CC77 2FAD A5CD 6035 332F A671
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

RE: RFR(XXS): 8185333: [s390] Fix build without cds

Lindenmaier, Goetz
Hi Andrew,

this was quick, thanks!

Best regards,
  Goetz.

> -----Original Message-----
> From: Andrew Haley [mailto:[hidden email]]
> Sent: Mittwoch, 26. Juli 2017 11:40
> To: Lindenmaier, Goetz <[hidden email]>; hotspot-runtime-
> [hidden email]
> Subject: Re: RFR(XXS): 8185333: [s390] Fix build without cds
>
> On 26/07/17 10:33, Lindenmaier, Goetz wrote:
> > Could I please get reviews for this tiny fix?
> > http://cr.openjdk.java.net/~goetz/wr17/8185333-fixCDSs390/webrev.01/
> >
> > This is required to build s390 without cds support.
>
> OK.
>
> --
> Andrew Haley
> Java Platform Lead Engineer
> Red Hat UK Ltd. <https://www.redhat.com>
> EAC8 43EB D3EF DB98 CC77 2FAD A5CD 6035 332F A671
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: RFR(XXS): 8185333: [s390] Fix build without cds

Ioi Lam
In reply to this post by Lindenmaier, Goetz
Looks good!

Thanks
Ioi

> On Jul 26, 2017, at 2:33 AM, Lindenmaier, Goetz <[hidden email]> wrote:
>
> Hi,
>
> Could I please get reviews for this tiny fix?
> http://cr.openjdk.java.net/~goetz/wr17/8185333-fixCDSs390/webrev.01/
>
> This is required to build s390 without cds support.
>
> Best regards,
>  Goetz.

Loading...