Skip to content

loader: normalize ./-prefixed element names (#2132)#2133

Open
kotborealis wants to merge 1 commit into
apache:masterfrom
kotborealis:normalize-paths-elements
Open

loader: normalize ./-prefixed element names (#2132)#2133
kotborealis wants to merge 1 commit into
apache:masterfrom
kotborealis:normalize-paths-elements

Conversation

@kotborealis

@kotborealis kotborealis commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Fixes #2132 by normalizing ./-prefixed names in loader, so ./a.bst and a.bst are treated exactly the same.

@juergbi juergbi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the PR. This issue also affects targets such as foo/./bar.bst, not just a ./ prefix. I think we can use os.path.normpath() to cover all these cases.

@kotborealis kotborealis force-pushed the normalize-paths-elements branch from c11b003 to c10c9e7 Compare June 12, 2026 11:18
@kotborealis kotborealis requested a review from juergbi June 12, 2026 11:19
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.

./foo.bst and foo.bst produce different element hashes

2 participants