looks good.

/Andy

On 8/19/2019 12:56 PM, Alexey Semenyuk wrote:
Please review the jpackage fix for bug [1] at [2].

This is a fix for the JDK-8200758-branch branch of the open sandbox repository (jpackage).

- fix all permission related issues reported by lintian
- cleanup control files to address issues reported by lintian
- fix jtreg tests
- change package installation directory of Debian package from /opt/<name> to /opt/<package> to make it compliant with Debian packaging requirements

Errors reported by lintian on simple test package before the fix is [3] and after the fix is [4]

[1] https://bugs.openjdk.java.net/browse/JDK-8225447

[2] http://cr.openjdk.java.net/~asemenyuk/8225447/webrev.00/

[3] http://cr.openjdk.java.net/~asemenyuk/8225447/8225447.lintian.before_the_fix.txt

[4] http://cr.openjdk.java.net/~asemenyuk/8225447/8225447.lintian.after_the_fix.txt

Thanks,
Alexey

Reply via email to