RFR: JDK-8212780: JEP 343: Packaging Tool Implementation
Kevin Rushforth
kevin.rushforth at oracle.com
Tue Nov 19 20:13:13 UTC 2019
I took the "git diff" patch [5] that you uploaded yesterday, applied it,
and verified that it is the same as what is in the JDK-8200758-branch
branch of the sandbox. It builds and runs fine for me.
I ran jcheck and it found the following three files with whitespace
errors that will need to be fixed before you push:
src/jdk.incubator.jpackage/linux/classes/jdk/incubator/jpackage/internal/PackageProperty.java:49:
Trailing whitespace
src/jdk.incubator.jpackage/share/classes/jdk/incubator/jpackage/ToolProviderFactory.java:35:
Trailing whitespace
test/jdk/tools/jpackage/share/AddLauncherBase.java:137: Trailing whitespace
The second of these will go away with the fix for JDK-8234402 [6], so
you don't need to do anything there. Once the fix for JDK-8234402 is
pushed to sandbox, I presume you will update the webrev, right?
Pending the fix for JDK-8234402 and the needed white-space fixes, this
is a +1 from me (although I am not a jdk Project Reviewer, so you will
need at least one review from someone who is).
-- Kevin
[5] http://cr.openjdk.java.net/~herrick/8212780/JDK-EA-10.git.patch
[6] https://bugs.openjdk.java.net/browse/JDK-8234402
On 11/13/2019 4:23 PM, Andy Herrick wrote:
> Please review changes for [1] which is the implementation bug for
> JEP-343.
>
> The webrev at [2] is the total cumulative webrev of changes for the
> jpackage tool, currently in the JDK-8200758-branch branch of the open
> sandbox repository.
>
> The webrev at [3] shows the changes since EA-06 (Build
> 13-jpackage+1-49 ) up to the current point.
>
> The latest EA (Build 14-jpackage+1-49 ) is posted at [4].
>
> Please send feedback to core-libs-dev at openjdk.java.net
>
>
> [1] https://bugs.openjdk.java.net/browse/JDK-8212780
>
> [2] http://cr.openjdk.java.net/~herrick/8212780/webrev.EA-10/
>
> [3] http://cr.openjdk.java.net/~herrick/8212780/webrev.06-10/
>
> [4] http://jdk.java.net/jpackage/
>
More information about the core-libs-dev
mailing list