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