Add updatesite/category.xml so the eclipse-repository build is non-empty#53
Merged
Conversation
Tycho's eclipse-repository build produced an empty p2 repo because the module only had the legacy site.xml. With a category.xml, a plain ./mvnw clean install produces a complete per-version p2 repo at updatesite/target/repository (feature + 8 bundles + category) with no Eclipse GUI or p2 publisher step. This removes the README's "remove the feature and build again" quirk from the release flow. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015krtPy49YGqBERvq4yDjCa
There was a problem hiding this comment.
Pull request overview
Adds a Tycho category.xml to the update site module so the eclipse-repository build produces a non-empty, consumable p2 repository via a standard ./mvnw clean install.
Changes:
- Introduces
category.xmldefining the feature-to-category mapping and category metadata for the update site.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+7
to
+8
| <description> A framework with some common functionality for refactoring tool development. | ||
| </description> |
This was referenced Jun 18, 2026
khatchad
added a commit
that referenced
this pull request
Jun 18, 2026
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015krtPy49YGqBERvq4yDjCa
khatchad
added a commit
that referenced
this pull request
Jun 18, 2026
…52) * Prototype: CI-published p2 update site via GitHub Pages composite repo Replaces the in-repo, cumulative, hand-assembled update site with a CI-built, Pages-published p2 composite repository (no binaries in master). See docs/RELEASING.md for the comparison. - Add updatesite/category.xml so Tycho's eclipse-repository build produces a complete per-version p2 repo (fixes the empty-build quirk; no GUI / p2 publisher / -append needed). - Add tools/p2-composite.sh to generate p2 composite indexes; verified the Eclipse p2 director loads the composite and resolves the feature. - Add .github/workflows/release.yml: manual-dispatch release pipeline (set-version, build/test, tag + GitHub Release with the p2 repo zip, publish to gh-pages as a composite, next-dev bump). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015krtPy49YGqBERvq4yDjCa * Restyle RELEASING.md: title-case headers, no-space em-dashes, no line wrapping Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015krtPy49YGqBERvq4yDjCa * Drop category.xml from prototype (merged separately via #53/#54) Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015krtPy49YGqBERvq4yDjCa * Commit the release version before tagging in release.yml The tag must point at the release-version sources, and master needs a "Release vX.Y.Z." commit, so commit the version bump and push master before tagging (mirrors release.sh). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015krtPy49YGqBERvq4yDjCa --------- Co-authored-by: Claude Opus 4.8 (1M context) <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.
Extracted from #52 as a standalone, low-risk fix.
Tycho's
eclipse-repositorybuild was producing an empty p2 repo (the README's "Remove the feature and build again" quirk) because the updatesite module only had the legacysite.xml. Adding acategory.xmlmakes a plain./mvnw clean installproduce a complete per-version p2 repo atupdatesite/target/repository(feature + 8 bundles + category) — no Eclipse GUI, no p2 publisher, no-append.This benefits the current release flow immediately (removes the GUI/quirk dependency) and is a prerequisite for the CI-published flow in #52.
site.xmland the committed update site are untouched.Verified:
clean installproduces a populatedtarget/repository;spotless:checkgreen.🤖 Generated with Claude Code