Yes - sorry - thought I had replied.

looks good.

/Andy

On 5/15/2020 3:36 AM, Nick Gasson wrote:
On 05/15/20 03:39 am, Alexey Semenyuk wrote:
Looks good.

- Alexey
Thanks Alexey. Can I get another reviewer to look at it before pushing?


Nick

Hi,

Bug: https://bugs.openjdk.java.net/browse/JDK-8244981
Webrev: http://cr.openjdk.java.net/~ngasson/8244981/webrev.0/

Several jpackage jtreg tests fail on Debian/Ubuntu with this error:

      Running [fakeroot, dpkg-deb, --verbose, -b, <..snip..>]
      dpkg-deb: error: parsing file '/tmp/<..snip..>/images/DEBIAN/control' 
near line 9 package 'appcategorytest':
       end of file during value of field 'Installed-Size' (missing final 
newline)

dpkg-deb requires each field in the control file to be terminated with a
newline. This patch adds a trailing newline to all resource files, but I
think that shouldn't cause a problem.

Tested with jdk/tools/jpackage jtreg tests.

--
Thanks,
Nick

Reply via email to