> Please review this small fix. > > Configuring the same log file as `vm.log`, `$PWD/vm.log`, and `./vm.log` > creates separate outputs, splitting the messages across multiple files. > > **Fix:** > > Let each output match its own name through `matches_name()`. For file > outputs, also check `os::same_files()` so different paths to the same file > reuse the existing output. > > **Testing:** > > - Added `TestEquivalentLogFilePaths.java`. > - `gtest:LogConfiguration/server`: 21 passed, 0 failed. > - `test/hotspot/jtreg/serviceability/logging`: 7 passed, 0 failed. > - `test/hotspot/jtreg/runtime/logging`: 33 passed, 0 failed. > > --------- > - [x] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai).
Yunbo Zhang has updated the pull request incrementally with one additional commit since the last revision: 8392135: Unified logging creates extra log files ------------- Changes: - all: https://git.openjdk.org/jdk/pull/33090/files - new: https://git.openjdk.org/jdk/pull/33090/files/31956bba..96b7ec15 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=33090&range=03 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=33090&range=02-03 Stats: 1 line in 1 file changed: 0 ins; 1 del; 0 mod Patch: https://git.openjdk.org/jdk/pull/33090.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/33090/head:pull/33090 PR: https://git.openjdk.org/jdk/pull/33090
