RFR: JDK-8256475: Fix Behavior when Installer name differs from applicatio… [v2]

Alexey Semenyuk asemenyuk at openjdk.java.net
Thu Nov 19 17:30:13 UTC 2020


On Thu, 19 Nov 2020 16:49:23 GMT, Andy Herrick <herrick at openjdk.org> wrote:

>> …n name.
>
> Andy Herrick has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 69 additional commits since the last revision:
> 
>  - JDK-8256475: Fix Behavior when Installer name differs from application name.
>  - Merge branch 'master' into JDK-8256475
>  - 8221554: aarch64 cross-modifying code
>    
>    Reviewed-by: rehn, aph
>  - 8255978: [windows] os::release_memory may not release the full range
>    
>    Reviewed-by: iklam, minqi
>  - 8256375: AArch64: aarch64-asmtest.py may generate undefined register r18
>    
>    Reviewed-by: aph
>  - 8252304: Seed an HttpRequest.Builder from an existing HttpRequest
>    
>    Co-authored-by: Chris Hegarty <chegar at openjdk.org>
>    Reviewed-by: chegar, dfuchs, michaelm
>  - 8256499: Zero: enable Epsilon GC
>    
>    Reviewed-by: rkennke, ihse
>  - 8256552: Let ReplayCompiles set UseDebuggerErgo
>    
>    Reviewed-by: vlivanov, redestad
>  - 8256172: Clean up CDS handling of i2i_entry
>    
>    Reviewed-by: minqi, ccheung
>  - 8256178: Add RAII object for file lock
>    
>    Reviewed-by: dholmes, coleenp
>  - ... and 59 more: https://git.openjdk.java.net/jdk/compare/64b64ac9...2e044b91

test/jdk/tools/jpackage/helpers/jdk/jpackage/test/JPackageCommand.java line 218:

> 216:     }
> 217: 
> 218:     public String installerName() {

I'd suggest to insert `verifyIsOfType(PackageType.NATIVE);` call at the beginning of the function body. This will help to catch situations if installerName() is called on jpackage command building app image by accident.

test/jdk/tools/jpackage/share/MultiNameTwoPhaseTest.java line 37:

> 35: 
> 36: /*
> 37:  * @test

Without ` * @requires jpackage.test.SQETest == null` this test will be used to create test artifacts for SQE. I don't think we need any manual testing here. Especially for five test scenarios. I think automated testing should be sufficient.

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

PR: https://git.openjdk.java.net/jdk/pull/1300


More information about the core-libs-dev mailing list