On Tue, 22 Apr 2025 07:41:43 GMT, Daishi Tabata <d...@openjdk.org> wrote:
>> After searching the entire JDK source code, I found that WMIC is only used >> in four files. These WMIC calls can be replaced with PowerShell for WMI. >> >> The primary challenge in this replacement is to make it work the same as >> before, even if the output format of the PowerShell command is different >> from the original WMIC output. Where necessary, I've adjusted the output >> formatting to maintain consistency. >> >> Regarding the PowerShell options `-NoLogo`, `-NoProfile`, and >> `-NonInteractive`, I've included them only when they are already used in the >> surrounding code within the affected file. >> Note: In my environment, it worked correctly even without these options. >> >> The `failure_handler` outputs powershell command execution results directly >> into HTML. While the number and order of output items may differ slightly >> after the modification, all previously output items are still included. >> Therefore, I believe this is not a problem. Specific output changes are >> located in: >> >> - `environment.html`: `windows/system/os` section >> - `process.html`: `[Process ID]/windows/native/info` section >> >> **Testing:** >> I have confirmed that all tests in `jdk/tools/jpackage` pass after these >> changes. > > Daishi Tabata has updated the pull request incrementally with one additional > commit since the last revision: > > run jcheck Marked as reviewed by asemenyuk (Reviewer). Looks good assuming jpackage tests ([test/jdk/tools/jpackage/windows/WinNoRestartTest.java](https://github.com/openjdk/jdk/blob/a6995a3d42955f1f207c14be1634daf225b5ab3f/test/jdk/tools/jpackage/windows/Win8301247Test.java#L24) and [test/jdk/tools/jpackage/windows/Win8301247Test.java](https://github.com/openjdk/jdk/blob/a6995a3d42955f1f207c14be1634daf225b5ab3f/test/jdk/tools/jpackage/windows/Win8301247Test.java#L24)) passed with this change. ------------- PR Review: https://git.openjdk.org/jdk/pull/24791#pullrequestreview-2818983754 PR Comment: https://git.openjdk.org/jdk/pull/24791#issuecomment-2855254506
