Skip to content

fix: use forward slash separator in Maven purl format#1530

Merged
shs96c merged 1 commit into
bazel-contrib:masterfrom
stevebarrau:fix-purl-namespace
Jan 22, 2026
Merged

fix: use forward slash separator in Maven purl format#1530
shs96c merged 1 commit into
bazel-contrib:masterfrom
stevebarrau:fix-purl-namespace

Conversation

@stevebarrau
Copy link
Copy Markdown
Contributor

Update the Maven package URL (purl) format to use forward slash (/) instead of colon (:) as the separator between group and artifact, following the correct purl specification for Maven packages.

The format now correctly generates:
pkg:maven/group/artifact@version
instead of:
pkg:maven/group:artifact@version

https://github.com/package-url/purl-spec/blob/main/types-doc/maven-definition.md

Update the Maven package URL (purl) format to use forward slash (/)
instead of colon (:) as the separator between group and artifact,
following the correct purl specification for Maven packages.

The format now correctly generates:
  pkg:maven/group/artifact@version
instead of:
  pkg:maven/group:artifact@version

https://github.com/package-url/purl-spec/blob/main/types-doc/maven-definition.md
Copy link
Copy Markdown
Collaborator

@shs96c shs96c left a comment

Choose a reason for hiding this comment

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

LGTM. Ship it!

@shs96c shs96c merged commit 2c50ea4 into bazel-contrib:master Jan 22, 2026
6 checks passed
@stevebarrau stevebarrau deleted the fix-purl-namespace branch January 22, 2026 11:15
shs96c added a commit to JonathanPerry651/rules_jvm_external that referenced this pull request Feb 3, 2026
* master: (25 commits)
  fix: use forward slash separator in Maven purl format (bazel-contrib#1530)
  Load rules from specific bzl files and add sh_test imports (bazel-contrib#1529)
  Added non-conflicting hash for install files (bazel-contrib#1454)
  Update the maven and coursier resolver tests to create a class index file. (bazel-contrib#1519)
  [ci] Drop Bazel 6 and ensure we run on Bazel 7 and 8 (bazel-contrib#1525)
  Only allow modules specified in known_contributing_modules to contribute artifacts or boms to the root module (bazel-contrib#1523)
  [gradle] Fix false resolution failures when BOM upgrades dependency version (bazel-contrib#1520)
  [gradle] Fix Gradle resolver to respect force_version and include runtime dependencies (bazel-contrib#1516)
  Correctly merge BOMs from non-root modules (bazel-contrib#1518)
  Update more lock files
  Filter test_only artifacts out of artifacts merged into root repos and print a warning when a root artifact version is overridden by a non_root bazel_dep (bazel-contrib#1511)
  Fix SHA mismatch for conflicting dependency versions (bazel-contrib#1513)
  [gradle] Plumb through the force_version attribute (bazel-contrib#1515)
  [gradle] Add dep exclusions to only that dep (bazel-contrib#1514)
  [gradle] Handle aggregating dependencies and relocation version conflicts (bazel-contrib#1512)
  BOM Fixes (bazel-contrib#1506)
  Allow an optional index of dep -> class to be created (bazel-contrib#1492)
  Put files in `ResolutionResult` (bazel-contrib#1484)
  Optimize dependency graph building with O(1) lookups (bazel-contrib#1483)
  Provide a mechanism to list all resolved direct deps for a workspace (bazel-contrib#1510)
  ...
shs96c added a commit to shs96c/rules_jvm_external that referenced this pull request Feb 3, 2026
* master:
  Add presubmit check for prebuilt jars (bazel-contrib#1486)
  Upload artifacts in parallel (address artifactorys "Maven Snapshot Version Behaviour") (bazel-contrib#1524)
  feat: Support COURSIER_SHA256 environment variable (bazel-contrib#1527)
  fix: Do not add coursier opts when run other tools (bazel-contrib#1531)
  fix: add string attributes to `amend_artifact` for explicit unset state (bazel-contrib#1499)
  fix: use forward slash separator in Maven purl format (bazel-contrib#1530)
  Load rules from specific bzl files and add sh_test imports (bazel-contrib#1529)
  Added non-conflicting hash for install files (bazel-contrib#1454)
  Update the maven and coursier resolver tests to create a class index file. (bazel-contrib#1519)
  [ci] Drop Bazel 6 and ensure we run on Bazel 7 and 8 (bazel-contrib#1525)
  Only allow modules specified in known_contributing_modules to contribute artifacts or boms to the root module (bazel-contrib#1523)
  [gradle] Fix false resolution failures when BOM upgrades dependency version (bazel-contrib#1520)
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