RFR: 8244533: Configure should abort on missing short names in Windows

Erik Joelsson erikj at openjdk.org
Wed May 7 15:26:15 UTC 2025


On Mon, 7 Apr 2025 09:18:08 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:

> Windows can create short names for directories that contain spaces in the name.
> 
> However, where short name creation was previously enabled by default, it no longer seems to be in recent version of Windows (the default setting is instead 'per volume', and the default for each volume is 'off').
> 
> We should check that the output paths of these utility macros are actually space-free, and emit an error otherwise. We should also add additional documentation about short names, and the short name creation setting to doc/building.md.
> 
> When testing this in several different cases with path including spaces that either had or had not a short name, it became apparent that the current handling of such cases were not ... optimal. Hence some bug fixes were needed to properly handle such cases.

This is a great change. Just a minor grammar nit in the docs.

doc/building.md line 117:

> 115:       upper and lower case letters.
> 116: 
> 117:     Failure to follow this procedure might result in hard-to-debug build

We are listing 2 bullets for Cygwin here so I think plural sounds better.
Suggestion:

    Failure to follow these procedures might result in hard-to-debug build

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

PR Review: https://git.openjdk.org/jdk/pull/24483#pullrequestreview-2822221921
PR Review Comment: https://git.openjdk.org/jdk/pull/24483#discussion_r2077896621


More information about the build-dev mailing list