On Wed, 16 Oct 2024 18:45:38 GMT, Alexey Semenyuk <asemen...@openjdk.org> wrote:

> Make jpackage correctly handle the output directory (`--dest`) that is:
>  - a subdirectory of the input directory;
>  - the same as the input directory.
> 
> Make jpackage correctly handle temp directory (`--temp`) that is:
>  - a subdirectory of the input directory.
> 
> jpackage will produce an app image or a package in the above directory 
> configurations without running in an infinite loop.
> 
> The new test/jdk/tools/jpackage/share/InOutPathTest.java test covers all the 
> above scenarios.
> 
> `IOUtils. copyRecursive()` fixed to never run in a recursive loop by building 
> the list of non-recursive file/dir copy commands and executing them outside 
> the recursive directory tree traversal.
> 
> src/jdk.jpackage/share/classes/jdk/jpackage/internal/Arguments.java, 
> src/jdk.jpackage/share/classes/jdk/jpackage/internal/DeployParams.java, 
> src/jdk.jpackage/share/classes/jdk/jpackage/internal/StandardBundlerParam.java,
>  
> test/jdk/tools/jpackage/junit/jdk.jpackage/jdk/jpackage/internal/DeployParamsTest.java
>  files fixed to get rid of `DeployParams.input` and `DeployParams.output` 
> fields and replace them with the new `StandardBundlerParam.OUTPUT_DIR` param.
> 
> The above changes altered the logic of configuring the output directory, thus 
> added a new `BasicTest.testNoOutputDir()` test to test jpackage without 
> `--dest` parameter. This required adding new API to test helper classes.

@sashamatveev please review

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

PR Comment: https://git.openjdk.org/jdk/pull/21550#issuecomment-2418098284

Reply via email to