Re: RFR: 8293883: Add test/.classpath (Eclipse) [v2]

2022-09-22 Thread Andy Goryachev
> Adding a tests/.classpath file to include tests/manual/controls source folder > so Eclipse could see/execute manual tests there. > > Also, I would rather move the sources there to a specific package > (test.manual ?) instead of a default package. Any objections? > edit: won't be done as a

Re: RFR: 8293883: Add test/.classpath (Eclipse)

2022-09-22 Thread Andy Goryachev
On Thu, 15 Sep 2022 21:55:20 GMT, Andy Goryachev wrote: > Adding a tests/.classpath file to include tests/manual/controls source folder > so Eclipse could see/execute manual tests there. > > Also, I would rather move the sources there to a specific package > (test.manual ?) instead of a

Re: RFR: 8293883: Add test/.classpath (Eclipse)

2022-09-22 Thread Nir Lisker
On Thu, 15 Sep 2022 21:55:20 GMT, Andy Goryachev wrote: > Adding a tests/.classpath file to include tests/manual/controls source folder > so Eclipse could see/execute manual tests there. > > Also, I would rather move the sources there to a specific package > (test.manual ?) instead of a

Re: RFR: 8293883: Add test/.classpath (Eclipse)

2022-09-22 Thread Andy Goryachev
On Thu, 22 Sep 2022 19:54:41 GMT, Nir Lisker wrote: > There are sources under `tests/performance` and `tests/manual` that should > also be included in the classpath. I would like to limit the scope of this PR. The problem is that one thing leads to another - and the fact that all these

Re: RFR: 8293883: Add test/.classpath (Eclipse)

2022-09-22 Thread Nir Lisker
On Thu, 15 Sep 2022 21:55:20 GMT, Andy Goryachev wrote: > Adding a tests/.classpath file to include tests/manual/controls source folder > so Eclipse could see/execute manual tests there. > > Also, I would rather move the sources there to a specific package > (test.manual ?) instead of a

Re: RFR: 8293883: Add test/.classpath (Eclipse)

2022-09-22 Thread Andy Goryachev
On Thu, 15 Sep 2022 21:55:20 GMT, Andy Goryachev wrote: > Adding a tests/.classpath file to include tests/manual/controls source folder > so Eclipse could see/execute manual tests there. > > Also, I would rather move the sources there to a specific package > (test.manual ?) instead of a

Re: RFR: 8293883: Add test/.classpath (Eclipse)

2022-09-16 Thread Kevin Rushforth
On Thu, 15 Sep 2022 21:55:20 GMT, Andy Goryachev wrote: > Also, I would rather move the sources there to a specific package > (test.manual ?) instead of a default package. Any objections? That should probably be done via a separate cleanup bug, since it is outside the scope of this IDE-only

RFR: 8293883: Add test/.classpath (Eclipse)

2022-09-15 Thread Andy Goryachev
Adding a tests/.classpath file to include tests/manual/controls source folder so Eclipse could see/execute manual tests there. Also, I would rather move the sources there to a specific package (test.manual ?) instead of a default package. Any objections? - Commit messages: -