RFR: 7902924: Add build script for jtreg plugin [v4]

Bradford Wetmore wetmore at openjdk.java.net
Wed May 5 20:55:58 UTC 2021


On Wed, 5 May 2021 20:43:31 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:

>> plugins/idea/README.md line 60:
>> 
>>> 58: ### Installing the plugin
>>> 59: 
>>> 60: To install the plugin in your IDE, first you need to build a plugin module file (a `.jar` file), as described in the previous section.
>> 
>> Several comments here:
>> 
>> 1.  The addition in the filename "-gradle" is a little confusing, is it really necessary?
>> 
>> 2.  There are two jar files in my resulting build directory, you have to compare them to know they are the same.  Just adds a speedbump for someone trying for the first time.
>> 
>> ./build/libs/jtreg-plugin-gradle-1.12.jar
>> ./build/idea-sandbox/plugins/jtreg-plugin-gradle/lib/jtreg-plugin-gradle-1.12.jar
>> 
>> 3.  IntelliJ can also install a .zip file, but you said to install the jar file.  Is it possible to not build the .zip file?
>> 
>> ./build/distributions/jtreg-plugin-gradle-1.12.zip
>> 
>> 4.  Lastly, I built the plugin with my default JDK 16 (60.0), but apparently Intellij IDEA 2021.1 only recognized up to JDK 11 (55.0) and hung after throwing up a popup.  You might want to -target 11?
>
>> Several comments here:
>> 
>>     1. The addition in the filename "-gradle" is a little confusing, is it really necessary?
> 
> I didn't do anything to add that - maybe it can be removed, I'll check
>> 
>>     2. There are two jar files in my resulting build directory, you have to compare them to know they are the same.  Just adds a speedbump for someone trying for the first time.
> 
> The README talks about `build/distributions`. In there there is just _one_ file:
> 
> build/distributions/jtreg-plugin-gradle-1.12.zip
> 
> That is the plugin that should be installed.
> 
>> 
>> 
>> ./build/libs/jtreg-plugin-gradle-1.12.jar
>> ./build/idea-sandbox/plugins/jtreg-plugin-gradle/lib/jtreg-plugin-gradle-1.12.jar
>> 
>>     1. IntelliJ can also install a .zip file, but you said to install the jar file.  Is it possible to not build the .zip file?
> 
> The zip file is the one to install. I'll drop references to `jar` and replace with `zip`
> 
>> 
>> 
>> ./build/distributions/jtreg-plugin-gradle-1.12.zip
>> 
>>     1. Lastly, I built the plugin with my default JDK 16 (60.0), but apparently Intellij IDEA 2021.1 only recognized up to JDK 11 (55.0) and hung after throwing up a popup.  You might want to -target 11?
> 
> I'll check that.

Missed the zip, I probably just read it as jar since you talked about building the jar in the para above.

-------------

PR: https://git.openjdk.java.net/jtreg/pull/9


More information about the jtreg-dev mailing list