RFR: 8335880: More troubleshooting tips around windows space in path [v3]

Magnus Ihse Bursie ihse at openjdk.org
Mon Aug 5 16:01:31 UTC 2024


On Mon, 5 Aug 2024 15:55:12 GMT, Chen Liang <liach at openjdk.org> wrote:

>> doc/building.md line 1816:
>> 
>>> 1814: The only directories required to have short paths are `Microsoft Visual Studio`
>>> 1815: and `Windows Kits`; the rest of the "contains space" warnings from `configure`
>>> 1816: can be ignored. You can choose any short name; once it is set, `configure`'s
>> 
>> What other directories are reported from configure? 
>> 
>> It does not seem correct to me to say that they can just be ignored -- either the directory is relevant for compilation, and then we need to have it without spaces, or it is not really used and then it should be removed, or made optional.
>> 
>> If  `Microsoft Visual Studio` and  `Windows Kits` are typical path-with-spaces that users can run into, we should perhaps add an additional check to configure for these. In fact, I think what we *should* do is to have configure print this help about `fsutil file setshortname` if it encounters a path with spaces that it cannot turn into a short name. That way, a complete copy/pastable command line can be generated for the user to run.
>
> configure scans `PATH` which may include other spaced directories, such as "Intellij Idea." On a user device, this list can quickly become huge and it would be unrealistic for users to generate short paths for every single one of those directories.
> 
> In fact, the only 2 directories that need short path configuration for configure to work correctly are Microsoft Visual Studio and Windows Kits.

Hm, it feels buggy that it should complain about random entries in the PATH; it should only be an issue if we try to FIXUP a path. I need to look into that.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20197#discussion_r1704339319


More information about the build-dev mailing list