Skip to content

Conversation

@aglinxinyuan
Copy link
Contributor

@aglinxinyuan aglinxinyuan commented Dec 5, 2025

What changes were proposed in this PR?

Add a configuration option to automatically shorten file paths for Windows users when the original path exceeds the system’s maximum length.

After this PR, Windows users should not see this error anymore.

image

Any related issues, documentation, discussions?

Follow-up of PR #4087

How was this PR tested?

Tested manually.

Was this PR authored or co-authored using generative AI tooling?

No

@aglinxinyuan aglinxinyuan self-assigned this Dec 5, 2025
@github-actions github-actions bot removed the fix label Dec 5, 2025
@chenlica
Copy link
Contributor

chenlica commented Dec 5, 2025

Please explain how this shortening is done.

@aglinxinyuan
Copy link
Contributor Author

aglinxinyuan commented Dec 5, 2025

The shortening wasn’t done by me; this PR simply enables a configuration in IntelliJ.
IntelliJ explains how this shortening is done in this page: https://www.jetbrains.com/help/idea/run-debug-configuration-java-application.html#java

Copy link
Contributor

@Xiao-zhen-Liu Xiao-zhen-Liu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@aglinxinyuan aglinxinyuan enabled auto-merge (squash) December 6, 2025 01:54
@aglinxinyuan aglinxinyuan merged commit 6626c81 into main Dec 6, 2025
10 checks passed
@aglinxinyuan aglinxinyuan deleted the xinyuan-fix-path-length branch December 6, 2025 02:02
@chenlica
Copy link
Contributor

chenlica commented Dec 6, 2025

What if a developer is not using IntelliJ?

@aglinxinyuan
Copy link
Contributor Author

What if a developer is not using IntelliJ?

They need to determine their own approach. We officially support using intellij for development. Users working with Docker will not encounter the ‘path too long’ issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants