RFR: JDK-8212780: JEP 343: Packaging Tool Implementation
Roger Riggs
Roger.Riggs at oracle.com
Fri May 10 19:39:34 UTC 2019
Hi Andy,
Thanks for logging the issues.
>>
>> CLIHelp:
>> - 58, 65, 72, 80: Indentation of pLaunchOptions does not line up.
> I don't see what you mean here. Looks lined up to me
The lines with pLaunchOptions have tabs instead of spaces.
jcheck has some complaints about tabs and trailing spaces.
>>
>> IOUtils:
>> - 262: why the mix of ProcessBuilder and Runtime.exec - stick to
>> ProcessBuilder
> added to this case to JDK-8223334
>>
>> Log:
>> "JPACKAGE_DEBUG" environment variable - ? uppercase, documented?
> implemented as strictly upper case, what do we have to do to document ?
Usually java applications use system properties, not environment variables.
I can see the need for access to PATH to find the tools, but a debugging
flag seems like an outlier.
I suppose it is an undocumented implementation detail and does not need
to be documented.
Thanks, Roger
More information about the core-libs-dev
mailing list