I revised the webrev to have warning only executable name for unpack200 instead of full path on Windows, which I believe is what was intended all along, the test is also revised to take unpack200.exe on the Windows platform.
The updated version is at http://cr.openjdk.java.net/~henryjen/jdk11/8199871/1/webrev/ Cheers, Henry > On Jun 8, 2018, at 6:56 PM, Henry Jen <henry....@oracle.com> wrote: > > Hi, > > Please review this webrev[1] in which we mark Pack200 related API and tools > deprecate, and print a warning about tools to be remove in a future JDK > release. To avoid interrupt existing tools depends on the output, an option > is provided to suppress the warning message. > > The option name is following the convention of javah, not the GNU style as we > should. The rationale is that this is a temporary option and an existing > convention would make it more consistent for users. Anyhow, if we have a > strong feeling this should be changed, we can do that. > > For jar tool, the normalize option use Pack200 is also deprecated, the > suppress option only available to the new GNU style option mode. As if user > need to provide the option, the command line change is necessary, so without > compatible mode option should not be an issue. > > Cheers, > Henry > > [1] http://cr.openjdk.java.net/~henryjen/jdk11/8199871/0/webrev/
