Magnus/Tim/Erik, thanks for the reviews! Change pushed.
Cheers, Mikael > On Apr 27, 2020, at 1:58 PM, Erik Joelsson <erik.joels...@oracle.com> wrote: > > Looks good. > > /Erik > > On 2020-04-27 12:30, Mikael Vidstedt wrote: >> Ah, good point/catch! I updated the title of the JBS issue. New webrev: >> >> webrev: >> http://cr.openjdk.java.net/~mikael/webrevs/8243633/webrev.01/open/webrev/ >> incremental: >> http://cr.openjdk.java.net/~mikael/webrevs/8243633/webrev.01.incr/open/webrev/ >> >> Cheers, >> Mikael >> >>> On Apr 27, 2020, at 2:06 AM, Magnus Ihse Bursie >>> <magnus.ihse.bur...@oracle.com> wrote: >>> >>> On 2020-04-27 07:49, Mikael Vidstedt wrote: >>>> As on linux-x64, cups is already in the devkit, so no need for the >>>> explicit extra dependency. >>> Actually, I realized this goes for all our cross-platform linux devkits, >>> i.e. ppc64le, s390x and arm32 as well. Can you remove cups from them too >>> while you're at it? >>> >>> /Magnus >>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8243633 >>>> webrev: >>>> http://cr.openjdk.java.net/~mikael/webrevs/8243633/webrev.00/open/webrev/ >>>> >>>> Cheers, >>>> Mikael
