Looks good to me. On Fri, Aug 3, 2018 at 5:58 AM, Alan Bateman <alan.bate...@oracle.com> wrote:
> > > On 03/08/2018 00:21, David Holmes wrote: > >> On 3/08/2018 4:59 PM, Baesken, Matthias wrote: >> >>> Thank you David , can the change be pushed , or do I need a second >>> review for an XS change ? >>> (any way a second review would be good 😊 ) >>> >> >> Need a review from official build team member :) >> > I'm not in the build group but just to confirm that USE_MMAP is specific > to libzip and dates back to when the entire zip file (not just the CEN) was > memory mapped. I don't know how it got into make file to build libjdwp or > libdt_socket. So the change looks right to me too. > > -Alan. >