Publish artifacts under the nl.breakingit.breaktest Maven group#76
Merged
Conversation
Fork builds must not be published under Apache's org.apache.jmeter Maven namespace. The group matches the Implementation-Vendor-Id already used in the manifests. Java package names are unchanged and stay org.apache.jmeter for plugin and test-plan compatibility; artifact names (ApacheJMeter_*) are also unchanged. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Branding PR 5 — build provenance.
allprojects.groupmoves fromorg.apache.jmetertonl.breakingit.breaktest(matching theImplementation-Vendor-Idalready in the manifests), in both the main build and build-logic.Publishing fork artifacts under Apache's Maven namespace would be a trademark/provenance problem; with the source-available license this matters even more. Nothing is published publicly yet, so the old coordinates never escape.
Unchanged, deliberately:
org.apache.jmeter/org.apache.jorphan— plugin and test-plan compatibility (noted in a comment at the assignment).ApacheJMeter_*— thelib/extjar filenames are load-bearing for the self-updater and plugins; renaming them would be a separate transition like the launcher jar (Rename launcher jar to breaktest.jar (phase one: dual-name transition) #75), if ever.Verification
<groupId>nl.breakingit.breaktest</groupId>including inter-module dependency references.:src:dist:assembleand:src:core:jarbuild clean; no coordinate-style (org.apache.jmeter:) references exist anywhere in the build scripts.🤖 Generated with Claude Code