Skip to content

make cpan namespace optional - #103

Merged
petergardfjall merged 1 commit into
package-url:masterfrom
william-elastisys:cpan-optional-namespace
Aug 21, 2026
Merged

petergardfjall merged 1 commit into
package-url:masterfrom
william-elastisys:cpan-optional-namespace

Conversation

@william-elastisys

Copy link
Copy Markdown
Contributor

The purl-spec mentions on the cpan namespace that it is optional and when present it should be uppercased.

https://github.com/package-url/purl-spec/blob/ef12cbcd96cfdee33ad86f8b7fad74cf507c36c9/types/cpan-definition.json#L11

petergardfjall
petergardfjall previously approved these changes Aug 20, 2026

@petergardfjall petergardfjall left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM. Thanks.

@petergardfjall
petergardfjall force-pushed the cpan-optional-namespace branch from f2bae8f to 3ad9fa9 Compare August 20, 2026 11:03
@petergardfjall

Copy link
Copy Markdown
Collaborator

The testdata/purl-spec submodule (which holds the testsuite from the purl-spec repo) is still on an older version of that repo (package-url/purl-spec@c756cac), which is why the test is failing. Updating to the latest test spec comes with a whole lot of issues due to there having been a lot of changes to the test-spec. Need to revisit this when we have fixed the library to be able to pass the new testsuite.

@petergardfjall

Copy link
Copy Markdown
Collaborator

Updated the testdata/purl-spec submodule to the latest possible commit where it does not fail: package-url/purl-spec@9d6b901

@petergardfjall
petergardfjall merged commit c88ed7b into package-url:master Aug 21, 2026
2 checks passed
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