Re: RFR: 8326831 Clarify test harness control variables in make help

2024-03-05 Thread Magnus Ihse Bursie
On Tue, 27 Feb 2024 13:55:53 GMT, Ludvig Janiuk wrote: > Clarifying text in `make help` output on how to list variables for e.g. > JTREG='...'. Marked as reviewed by ihse (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/18028#pullrequestreview-1917514719

Re: RFR: 8326831 Clarify test harness control variables in make help

2024-02-28 Thread Guoxiong Li
On Tue, 27 Feb 2024 13:55:53 GMT, Ludvig Janiuk wrote: > Clarifying text in `make help` output on how to list variables for e.g. > JTREG='...'. > I have no idea what causes it. Nevermind. The patch looks good. - Marked as reviewed by gli (Committer). PR Review:

Re: RFR: 8326831 Clarify test harness control variables in make help

2024-02-28 Thread Ludvig Janiuk
On Tue, 27 Feb 2024 13:55:53 GMT, Ludvig Janiuk wrote: > Clarifying text in `make help` output on how to list variables for e.g. > JTREG='...'. It's interesting that the order of those two lines can change. I have no idea what causes it. - PR Comment:

Re: RFR: 8326831 Clarify test harness control variables in make help

2024-02-28 Thread Guoxiong Li
On Tue, 27 Feb 2024 13:55:53 GMT, Ludvig Janiuk wrote: > Clarifying text in `make help` output on how to list variables for e.g. > JTREG='...'. I use the command `make test-only JTREG=help` locally, the outputed message seems an error (shown below). Does it work as expected? Building target

Re: RFR: 8326831 Clarify test harness control variables in make help

2024-02-28 Thread Ludvig Janiuk
On Tue, 27 Feb 2024 13:55:53 GMT, Ludvig Janiuk wrote: > Clarifying text in `make help` output on how to list variables for e.g. > JTREG='...'. Thanks for the insight Erik! - PR Comment: https://git.openjdk.org/jdk/pull/18028#issuecomment-1968538692

Re: RFR: 8326831 Clarify test harness control variables in make help

2024-02-27 Thread Erik Joelsson
On Tue, 27 Feb 2024 13:56:56 GMT, Ludvig Janiuk wrote: > I'll note that you still need to run configure to execute make test-only > JTREG=help. Should this really be necessary? I agree that it would be convenient, in some rare usecase, to be able to run this without configure, but the added

Re: RFR: 8326831 Clarify test harness control variables in make help

2024-02-27 Thread Erik Joelsson
On Tue, 27 Feb 2024 13:55:53 GMT, Ludvig Janiuk wrote: > Clarifying text in `make help` output on how to list variables for e.g. > JTREG='...'. Marked as reviewed by erikj (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/18028#pullrequestreview-1904771315

RFR: 8326831 Clarify test harness control variables in make help

2024-02-27 Thread Ludvig Janiuk
Clarifying text in `make help` output on how to list variables for e.g. JTREG='...'. - Commit messages: - 8326831 Clarify test harness control variables in make help Changes: https://git.openjdk.org/jdk/pull/18028/files Webrev: https://webrevs.openjdk.org/?repo=jdk=18028=00

Re: RFR: 8326831 Clarify test harness control variables in make help

2024-02-27 Thread Ludvig Janiuk
On Tue, 27 Feb 2024 13:55:53 GMT, Ludvig Janiuk wrote: > Clarifying text in `make help` output on how to list variables for e.g. > JTREG='...'. I'll note that you still need to run configure to execute `make test-only JTREG=help`. Should this really be necessary? Also, while `make test-only