RFR: 8288838: jpackage: file association additional arguments [v3]

Alexey Semenyuk asemenyuk at openjdk.org
Thu Jul 7 00:01:51 UTC 2022


On Wed, 29 Jun 2022 12:29:33 GMT, Alex Kasko <akasko at openjdk.org> wrote:

>> jpackage implementation of file association on Windows currently passes a selected filename as an only argument to associated executable.
>> 
>> It is proposed to introduce additional option in file association property file to allow optionally support additional arguments using `%*` batch wildcard.
>> 
>> Note, current implementation, while fully functional, is only a **DRAFT** one, it is not ready for integration in this form. I would appreciate any guidance on the following points:
>> 
>>  - option naming inside a properties file, currently `pass-all-args` is used
>>  - option naming in a bundler parameter implementation, it is not clear if it should introduce a new group of "file association windows specific options" next to the existing "file association mac specific options" group
>>  - test organization to cover the new option: currently it is included inside `FileAssociationTest` and piggybacks on the existing (and unrelated) `includeDescription` parameter; it is not clear whether it should be done in a separate test and whether to include runs for every parameter combination
>>  - test run implementation: currently arguments are checked when a file with associated extension is invoked from command line; it is not clear whether it would be more appropriate instead to create a desktop shortcut with the same command as a target and to invoke it with `java.awt.Desktop`
>> 
>> Also please note, that full install/uninstall run is currently enabled in `FileAssociationTest`, it is intended to be used only in a draft code during the development and to be removed (to use the same "install or unpack" logic as other tests) in a final version.
>> 
>> Testing:
>>  
>> - [x] test to cover new logic is included
>> - [x] ran jtreg:jdk/tools/jpackage with no new failures
>
> Alex Kasko has refreshed the contents of this pull request, and previous commits have been removed. Incremental views are not available.

test/jdk/tools/jpackage/share/FileAssociationsTest.java line 108:

> 106:                 .applyTo(packageTest);
> 107: 
> 108:         packageTest.run(RunnablePackageTest.Action.CREATE, RunnablePackageTest.Action.INSTALL,

FYI: the default test steps can be overridden with the value of `jpackage.test.action` system property.
Its value would be `create,install,verify_install,uninstall` for this case.

UPD: Nevermind this comment. It applied to the old version of the PR.

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

PR: https://git.openjdk.org/jdk/pull/9224


More information about the core-libs-dev mailing list