Skip to content

build: resolve 8.3 filename format to long one on Windows - #2209

Closed
crazy-max wants to merge 1 commit into
docker:masterfrom
crazy-max:fix-win-longpath
Closed

build: resolve 8.3 filename format to long one on Windows#2209
crazy-max wants to merge 1 commit into
docker:masterfrom
crazy-max:fix-win-longpath

Conversation

@crazy-max

@crazy-maxcrazy-max commented Jan 25, 2024

Copy link
Copy Markdown
Member

related to #2206 (comment)

Converts Windows short pathnames to full pathnames (e.g. C:\Users\ADMIN~1 --> C:\Users\Administrator) so vcs:localdir:* and com.docker.image.source.entrypoint attrs are set in this case, otherwise: https://github.com/docker/buildx/actions/runs/7640538748/job/20815891223#step:5:419

Logic is taken from moby: https://github.com/moby/moby/blob/e8346c53d93d2cddf3d09910e166cdd8874070b3/integration-cli/utils_windows_test.go#L5-L27

Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
@crazy-max

Copy link
Copy Markdown
MemberAuthor

closed by #2206

@crazy-max
crazy-max deleted the fix-win-longpath branch January 26, 2024 08:10
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@crazy-max