Hi Alexey,

Looks good.

Thanks,
Alexander

On 9/18/2019 8:23 AM, 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).

This fix:

- Move common deb and rpm packaging code in the new class (jdk.jpackage.internal.LinuxPackageBundler).
- Implement Java runtime packaging on Linux.
- Fix of [3], [4] bugs as a side effect of code refactoring.
- Add shell scripts to run jpackage jtreg tests.
- Add tests for application desktop integration on Linux.
- Add uninstall script to clean leftovers after file associations of the app are removed. Uninstalling Linux app with file associations will produce the following console output:
---
Running: sudo dpkg -r fileassociationstest
(Reading database ... 250166 files and directories currently installed.)
Removing fileassociationstest (1.0-1) ...
Remove fileassociationstest-FileassociationsTest.desktop default handler for application/x-jpackage-jptest1 mime type from /usr/share/applications/defaults.list file Remove fileassociationstest-FileassociationsTest.desktop default handler for application/x-jpackage-jptest2 mime type from /usr/share/applications/defaults.list file
/usr/share/applications/defaults.list file updated
---
- Clean some unused code.

/Andy


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

[2] http://cr.openjdk.java.net/~asemenyuk/8225249/webrev.02/

[3] https://bugs.openjdk.java.net/browse/JDK-8230899

[4] https://bugs.openjdk.java.net/browse/JDK-8230917

Reply via email to