releng: 12.0.0 release#419
Conversation
Signed-off-by: Arnaud Fiorini <fiorini.arnaud@gmail.com>
📝 WalkthroughWalkthroughThis PR advances the Trace Compass build configuration from Eclipse 4.39 to 4.40 and updates the PDE API baseline from 11.3.0 to 12.0.0. Build defaults in setup and pom files are updated, the baseline target definition is renamed and repositioned to 12.0.0, and working target definitions refresh repository URLs and sequence numbers for CDT, Eclipse, and WebTools components to 2026-06 builds. ChangesTarget Platform and Baseline Upgrade
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Signed-off-by: Arnaud Fiorini <fiorini.arnaud@gmail.com>
Signed-off-by: Arnaud Fiorini <fiorini.arnaud@gmail.com>
Signed-off-by: Arnaud Fiorini <fiorini.arnaud@gmail.com>
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In
`@releng/org.eclipse.tracecompass.target/baseline/tracecompass-baseline-12.0.0.target`:
- Line 2: The baseline target file tracecompass-baseline-12.0.0.target declares
sequenceNumber="1", which may be incorrect for a new baseline; run the
PDE-target tooling or consult the project's PDE target update policy to
determine the correct starting sequence number and update the sequenceNumber
attribute accordingly (e.g., bump from "1" to the policy-required value) in the
tracecompass-baseline-12.0.0.target <target ... sequenceNumber="..."> element so
it aligns with other .target files and the project’s baseline sequencing rules.
- Line 53: The repository URL set in the target file's repository location
points to a landing page that returns HTML rather than p2 metadata, so
update/validate the p2 repository URL used by
tracecompass-baseline-12.0.0.target: confirm that the chosen URL exposes p2
metadata (content.xml or content.jar), and replace the current location value
with a URL that returns valid p2 metadata (or fallback to a known-good release
repo such as the previous release train) until the 2026-06 p2 repository is
published; ensure the change is made on the repository location attribute in the
target file so target resolution succeeds.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 89678a98-6e36-4bd7-92d1-486ec228c318
📒 Files selected for processing (4)
TraceCompass.setupreleng/org.eclipse.tracecompass.target/baseline/tracecompass-baseline-12.0.0.targetreleng/org.eclipse.tracecompass.target/tracecompass-e4.40.targetreleng/org.eclipse.tracecompass.target/tracecompass-eStaging.target
✅ Files skipped from review due to trivial changes (1)
- releng/org.eclipse.tracecompass.target/tracecompass-eStaging.target
🚧 Files skipped from review as they are similar to previous changes (2)
- releng/org.eclipse.tracecompass.target/tracecompass-e4.40.target
- TraceCompass.setup
What it does
How to test
Build Trace Compass
Follow-ups
N/A
Review checklist
Summary by CodeRabbit