On 8/28/2019 11:47 AM, Alexey Semenyuk wrote:
Looks good.
Some files in the review contain no diffs though, like
http://cr.openjdk.java.net/~herrick/8229979/webrev.01/test/jdk/tools/jpackage/windows/exe/WinUpgradeUUIDTest.java.sdiff.html.
Why are they in the review?
I ran a script to remove extraneous trailing spaces and fix file
permission. The result is that the file changed, but webrev has nothing
to show. I built the webrev with a file list so all changed files are
these, even if there is no visible change.
/ANdy
- Alexey
On 8/28/2019 9:36 AM, Andy Herrick 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).
[1] https://bugs.openjdk.java.net/browse/JDK-8229979
[2] http://cr.openjdk.java.net/~herrick/8229979/
Thanks,
Andy
