Skip to content

Fix #11754 - Moved assemply to bin-war module#11761

Merged
offtherailz merged 1 commit intogeosolutions-it:masterfrom
offtherailz:fix_bin_build_2
Dec 2, 2025
Merged

Fix #11754 - Moved assemply to bin-war module#11761
offtherailz merged 1 commit intogeosolutions-it:masterfrom
offtherailz:fix_bin_build_2

Conversation

@offtherailz
Copy link
Copy Markdown
Member

Description

Because of parent-child relationship introduced recently on bin package, during a full reactor build the mapstore-binary aggregator module executed the assembly plugin before the bin-war child produced mapstore.war on the CI server.
That means the ZIP created by the assembly could miss the WAR or copy an older artifact. Locally this was less visible because of incremental artifacts in the developer workspace.

  • What I changed

  • The assembly execution was moved from binary/pom.xml into binary/bin-war/pom.xml.

  • The assembly descriptor binary/bin.xml was updated to use paths relative to bin-war (so it can include target/mapstore.war, target/jre and target/dependency).

  • The assembly output directory was adjusted so the final ZIP lands in binary/target/ just like before.

  • Why this fixes it: Putting assembly in bin-war makes the assembly run as part of the bin-war module lifecycle which is executed after packaging the WAR. This removes reactor ordering issues (parent pom executed before child modules) that previously allowed the assembly to run before the WAR existed on CI.

Please check if the PR fulfills these requirements

What kind of change does this PR introduce? (check one with "x", remove the others)

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Other... Please describe:

Issue

What is the current behavior?

Fix #11754

What is the new behavior?

Bin build should be always done after war creation.

Breaking change

Does this PR introduce a breaking change? (check one with "x", remove the other)

  • Yes, and I documented them in migration notes
  • No

@offtherailz offtherailz merged commit 23b1527 into geosolutions-it:master Dec 2, 2025
11 checks passed
github-actions bot pushed a commit that referenced this pull request Dec 2, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 2, 2025

Successfully created backport PR for 2025.02.xx:

offtherailz added a commit that referenced this pull request Dec 3, 2025
(cherry picked from commit 23b1527)

Co-authored-by: Lorenzo Natali <lorenzo.natali@geosolutionsgroup.com>
dsuren1 added a commit that referenced this pull request Dec 4, 2025
…request

* commit 'a0da62e93ec7c7f9049c3343e410d4bea1db33c1':
  Update User Guide - Itinerary plugin (#11768)
  Fix the dashboard save, legend dependency and maximize issue (#11715)
  Update User Guide - Isochrone plugin (#11769)
  #11525 - Improve Maps in Dashboards (#11625)
  Update User Guide - Cascading search filter (#11766)
  Update User Guide - Show the current time and Null values options on Chart Widget (#11764)
  Fix #11754 - Moved assemply to bin-war module (#11761)
  Update User Guide - 3D Tiles imagery layers (#11758)
  Update User Guide - Update Tags menu section (#11756)
  Fix #11748 Access Restrictions for Maps Created from Unadvertised Contexts (#11752)
mahesh-wor pushed a commit to mahesh-wor/MapStore2 that referenced this pull request Dec 17, 2025
allyoucanmap pushed a commit that referenced this pull request Mar 13, 2026
(cherry picked from commit 23b1527)

Co-authored-by: Lorenzo Natali <lorenzo.natali@geosolutionsgroup.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

QA binary does not open

3 participants