Skip to content

build: declare central-publishing-maven-plugin explicitly - #184

Merged
marevol merged 1 commit into
masterfrom
feat/declare-central-publishing
Aug 6, 2026
Merged

build: declare central-publishing-maven-plugin explicitly#184
marevol merged 1 commit into
masterfrom
feat/declare-central-publishing

Conversation

@marevol

@marevol marevol commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Summary

Companion change for codelibs/fess-parent#61. Maven Central introduced an upload limit that
makes it impractical to keep releasing the Fess plugin artifacts there, so fess-parent moved
central-publishing-maven-plugin from build/plugins to pluginManagement and the plugin
repositories now deploy to maven.codelibs.org.

fess-crawler stays on Maven Central, so it declares the plugin itself. The version and the
publishingServerId configuration are still inherited from fess-parent.

The declaration is inheritable (no inherited=false), so fess-crawler,
fess-crawler-lasta and fess-crawler-opensearch are all published exactly as before.

Verification

  • Before the change: mvn validate no longer logged Installing Central Publishing features
    (i.e. the Maven Central path had been lost)
  • After the change: Inspecting build with total of 4 modules and
    Installing Central Publishing features are logged again
  • mvn package -DskipTests succeeds for all four modules

Depends on

codelibs/fess-parent#61 (must be merged first)

fess-parent moved central-publishing-maven-plugin from build/plugins to
pluginManagement so that the Fess plugin repositories deploy to
maven.codelibs.org instead of Maven Central. fess-crawler stays on Maven
Central, so it now declares the plugin itself. The version and the
publishingServerId configuration are still inherited from fess-parent.

The declaration is inheritable so that all three modules are published the
same way as before.
@marevol
marevol force-pushed the feat/declare-central-publishing branch from a43358d to 5f8898a Compare August 6, 2026 07:38
@marevol
marevol merged commit 4878c22 into master Aug 6, 2026
1 check 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.

1 participant