RFR: 8350594: Misleading warning about install dir for DMG packaging [v2]
Alexey Semenyuk
asemenyuk at openjdk.org
Tue Feb 25 01:02:54 UTC 2025
On Mon, 24 Feb 2025 22:20:27 GMT, Alexander Matveev <almatvee at openjdk.org> wrote:
> Currently it sounds that CLI option "--install-dir" is not supported, but it will work if it points to default location.
The warning will be shown only if `--install-dir` doesn't point to the default location. This is the change from the current behavior when the warning is shown if the `--install-dir` option is present on the command line regardless of its value.
If they pass `--install-dir /Applications` they will not see the warning.
If they pass, say `--install-dir /opt` they will see the warning. Currently, it will be:
Warning: "--install-dir" is not supported by DMG and will be default to /Applications.
Do you prefer:
Warning: "--install-dir" value "/opt" is not supported by DMG and will be default to /Applications.
?
If we want to improve the warning text additionally to making it accurate for runtime packaging, I'd rather have this wording:
Warning: "--install-dir" option is ignored for DMG packaging. The installation directory will default to {0}.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23752#discussion_r1968647620
More information about the core-libs-dev
mailing list