On Fri, 9 May 2025 15:28:44 GMT, Daniel Fuchs <dfu...@openjdk.org> wrote:
>> Volkan Yazici has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - Fix license year >> - Move path resolution back to option parsing > > Sorry - I have approved too fast - aren't there some man pages to update too? > Ah - no - I see the man page just says: >> -d dir or --directory dir >> Specifies the directory to serve. Default: current directory. > > We should be good then. @dfuch, I'd checked `jwebserver.md` and also concluded no change is needed there. In particular, I'd found the following mention of _absolute path_ of interest: > If startup is successful, the server prints a message to `System.out` > listing the local address and the absolute path of the directory being > served. We still comply with this behavior. ------------- PR Comment: https://git.openjdk.org/jdk/pull/25113#issuecomment-2871267437