Skip to content

Normalize paths in flatten#214

Merged
thesayyn merged 1 commit intobazel-contrib:mainfrom
loosebazooka:fix_flatten
Apr 15, 2026
Merged

Normalize paths in flatten#214
thesayyn merged 1 commit intobazel-contrib:mainfrom
loosebazooka:fix_flatten

Conversation

@loosebazooka
Copy link
Copy Markdown
Contributor

This normalizes all paths before flattening.

When flatting tars with equivalent paths ./etc/duck vs etc/duck/ flatten would just let them all in the flattened result. loading those images into a local container runtime would result in an error with repeated paths error. This instead normalizes them to etc/duck to prevent this.

This doesn't normalize absolute paths like /etc/duck

Signed-off-by: Appu Goundan <[email protected]>
@thesayyn
Copy link
Copy Markdown
Contributor

Wouldn't it be better to normalize them to ./? Its much more readable.

@loosebazooka
Copy link
Copy Markdown
Contributor Author

I guess I have no preference. I can switch it

@thesayyn thesayyn merged commit 07be8d6 into bazel-contrib:main Apr 15, 2026
1 check passed
@loosebazooka
Copy link
Copy Markdown
Contributor Author

@thesayyn I have a followup for this that will just preserve the seen path while deduplicating

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.

2 participants