Re: RFR: 8301569: list mode of jmod and jimage cannot be used normally in turkish locale [v4]

2023-06-27 Thread Naoto Sato
On Tue, 27 Jun 2023 15:20:33 GMT, Glavo wrote: >> When the default Locale is `tr`, the jmod and jimage commands have the >> following problems: >> >> * The jmod command does not correctly recognize the `list` mode typed in >> lowercase; >> * The jimage command cannot obtain the help informatio

Re: RFR: 8301569: list mode of jmod and jimage cannot be used normally in turkish locale [v4]

2023-06-27 Thread Mandy Chung
On Tue, 27 Jun 2023 15:20:33 GMT, Glavo wrote: >> When the default Locale is `tr`, the jmod and jimage commands have the >> following problems: >> >> * The jmod command does not correctly recognize the `list` mode typed in >> lowercase; >> * The jimage command cannot obtain the help informatio

Re: RFR: 8301569: list mode of jmod and jimage cannot be used normally in turkish locale [v3]

2023-06-27 Thread Glavo
On Tue, 27 Jun 2023 07:45:51 GMT, Glavo wrote: >> When the default Locale is `tr`, the jmod and jimage commands have the >> following problems: >> >> * The jmod command does not correctly recognize the `list` mode typed in >> lowercase; >> * The jimage command cannot obtain the help informatio

Re: RFR: 8301569: list mode of jmod and jimage cannot be used normally in turkish locale [v4]

2023-06-27 Thread Glavo
> When the default Locale is `tr`, the jmod and jimage commands have the > following problems: > > * The jmod command does not correctly recognize the `list` mode typed in > lowercase; > * The jimage command cannot obtain the help information of the `list` mode. Glavo has updated the pull reque

Re: RFR: 8301569: list mode of jmod and jimage cannot be used normally in turkish locale

2023-06-27 Thread Glavo
On Wed, 7 Jun 2023 06:32:31 GMT, Jaikiran Pai wrote: > Sounds fine to me and as other have agreed `Locale.ROOT` would be more > appropriate. Would you be willing to update this PR to update the few other > occurences in some classes (`JImageTask`, `JlinkTask` and > `VersionPropsPlugin`) as not

Re: RFR: 8301569: list mode of jmod and jimage cannot be used normally in turkish locale [v3]

2023-06-27 Thread Alan Bateman
On Tue, 27 Jun 2023 07:45:51 GMT, Glavo wrote: >> When the default Locale is `tr`, the jmod and jimage commands have the >> following problems: >> >> * The jmod command does not correctly recognize the `list` mode typed in >> lowercase; >> * The jimage command cannot obtain the help informatio

Re: RFR: 8301569: list mode of jmod and jimage cannot be used normally in turkish locale [v3]

2023-06-27 Thread Glavo
> When the default Locale is `tr`, the jmod and jimage commands have the > following problems: > > * The jmod command does not correctly recognize the `list` mode typed in > lowercase; > * The jimage command cannot obtain the help information of the `list` mode. Glavo has updated the pull reque

Re: RFR: 8301569: list mode of jmod and jimage cannot be used normally in turkish locale [v2]

2023-06-26 Thread Glavo
> When the default Locale is `tr`, the jmod and jimage commands have the > following problems: > > * The jmod command does not correctly recognize the `list` mode typed in > lowercase; > * The jimage command cannot obtain the help information of the `list` mode. Glavo has updated the pull reque

Re: RFR: 8301569: list mode of jmod and jimage cannot be used normally in turkish locale

2023-06-06 Thread Jaikiran Pai
On Wed, 31 May 2023 19:46:57 GMT, Glavo wrote: > > Hello @Glavo, I see you reopened this PR, so I'm guessing you are still > > interested in pursuing this further. Are you considering updating this PR > > to implement Alan's suggestion to do similar changes in `JImageTask.java` > > and `JlinkT

Re: RFR: 8301569: list mode of jmod and jimage cannot be used normally in turkish locale

2023-05-31 Thread Glavo
On Sun, 29 Jan 2023 23:54:21 GMT, Glavo wrote: >> When the default Locale is `tr`, the jmod and jimage commands have the >> following problems: >> >> * The jmod command does not correctly recognize the `list` mode typed in >> lowercase; >> * The jimage command cannot obtain the help informatio

Re: RFR: 8301569: list mode of jmod and jimage cannot be used normally in turkish locale

2023-05-31 Thread Mandy Chung
On Sun, 29 Jan 2023 15:37:28 GMT, Glavo wrote: > When the default Locale is `tr`, the jmod and jimage commands have the > following problems: > > * The jmod command does not correctly recognize the `list` mode typed in > lowercase; > * The jimage command cannot obtain the help information of t

Re: RFR: 8301569: list mode of jmod and jimage cannot be used normally in turkish locale

2023-05-04 Thread Jaikiran Pai
On Sun, 29 Jan 2023 23:54:21 GMT, Glavo wrote: >> When the default Locale is `tr`, the jmod and jimage commands have the >> following problems: >> >> * The jmod command does not correctly recognize the `list` mode typed in >> lowercase; >> * The jimage command cannot obtain the help informatio

Re: RFR: 8301569: list mode of jmod and jimage cannot be used normally in turkish locale

2023-02-01 Thread Alan Bateman
On Wed, 1 Feb 2023 09:26:17 GMT, Glavo wrote: >> src/jdk.jlink/share/classes/jdk/tools/jmod/JmodTask.java line 1474: >> >>> 1472: String verb = words.get(0); >>> 1473: try { >>> 1474: options.mode = Enum.valueOf(Mode.class, >>> verb.toUpperCase(Locale.ROO

Re: RFR: 8301569: list mode of jmod and jimage cannot be used normally in turkish locale

2023-02-01 Thread Glavo
On Wed, 1 Feb 2023 07:20:17 GMT, Alan Bateman wrote: > This is the lookup of the main.usage.list resource (`jimage list --help`) > rather than the `list` option but good to fix this anyway. Sorry, I found the problem with text search, but didn't look at them carefully. - PR: https

Re: RFR: 8301569: list mode of jmod and jimage cannot be used normally in turkish locale

2023-02-01 Thread Glavo
On Wed, 1 Feb 2023 07:20:13 GMT, Alan Bateman wrote: > The equivalent in jlink and jimage are using Locale.ENGLISH, I guess we > should try to keep them consistent. This class is using `Locale.ROOT` to convert modifiers to lowercase, and there is no reference to `Locale.ENGLISH`.

Re: RFR: 8301569: list mode of jmod and jimage cannot be used normally in turkish locale

2023-01-31 Thread Alan Bateman
On Sun, 29 Jan 2023 15:37:28 GMT, Glavo wrote: > When the default Locale is `tr`, the jmod and jimage commands have the > following problems: > > * The jmod command does not correctly recognize the `list` mode typed in > lowercase; > * The jimage command cannot obtain the help information of t

Re: RFR: 8301569: list mode of jmod and jimage cannot be used normally in turkish locale

2023-01-31 Thread Glavo
On Sun, 29 Jan 2023 15:37:28 GMT, Glavo wrote: > When the default Locale is `tr`, the jmod and jimage commands have the > following problems: > > * The jmod command does not correctly recognize the `list` mode typed in > lowercase; > * The jimage command cannot obtain the help information of t

RFR: 8301569: list mode of jmod and jimage cannot be used normally in turkish locale

2023-01-31 Thread Glavo
When the default Locale is `tr`, the jmod and jimage commands have the following problems: * The jmod command does not correctly recognize the `list` mode typed in lowercase; * The jimage command cannot obtain the help information of the `list` mode. - Commit messages: - list mode