[eXist-commits] [eXist-db/exist] f1899f: [bugfix] Make mvn package re-runnable

[Duncan P. <no...@gi...>]

  Branch: refs/heads/develop
  Home:   https://github.com/eXist-db/exist
  Commit: f1899fbbce3e29b67922175122121ee07d2511e4
      https://github.com/eXist-db/exist/commit/f1899fbbce3e29b67922175122121ee07d2511e4
  Author: Juri Leino <gi...@li...>
  Date:   2026-08-18 (Tue, 18 Aug 2026)

  Changed paths:
    M exist-distribution/pom.xml

  Log Message:
  -----------
  [bugfix] Make mvn package re-runnable

The delete steps removed the etc/jetty a previous package had produced, and
the rename could not recreate it: its source was consumed by that same earlier
run, and the unpack which provides it is skipped on re-runs by its markers. So
the transform found no directory and the build failed.

Closes https://github.com/eXist-db/exist/issues/6601

Co-Authored-By: Claude Opus 4.8 <no...@an...>
Claude-Session: https://claude.ai/code/session_01XPEMpenW8ngcKboNFB5taH


  Commit: 8ce0aacd475dd07e7e7ecf4f175298065a60b4b9
      https://github.com/eXist-db/exist/commit/8ce0aacd475dd07e7e7ecf4f175298065a60b4b9
  Author: Juri Leino <gi...@li...>
  Date:   2026-08-18 (Tue, 18 Aug 2026)

  Changed paths:
    M exist-distribution/pom.xml

  Log Message:
  -----------
  [bugfix] Replace jetty rename with silent Ant move to drop re-run ERROR log

copy-rename-maven-plugin's RenameMojo logged an ERROR on every no-clean
re-run once its source was already consumed by a prior run (harmless,
since it never threw, but noisy and easy to mistake for a real failure).
ignoreFileNotFoundOnIncremental only suppresses this inside m2e's
incremental build context, not on a plain CLI re-run.

Swap both rename executions (appassembler and mac-bundle) for a plain
Ant <move failonerror="false" quiet="true"> under the antrun block
already used elsewhere in this pom, matching the attributes the removed
<delete> executions used. It silently no-ops when the source is already
gone instead of logging.

Addresses review feedback from #6602 (https://github.com/eXist-db/exist/pull/6602#issuecomment-5084328399).

Co-Authored-By: Claude Sonnet 5 <no...@an...>
Claude-Session: https://claude.ai/code/session_016RWTxzdzejVnDry5Muf3ju


  Commit: df02f85975eb7465119456f220a467cefe76b300
      https://github.com/eXist-db/exist/commit/df02f85975eb7465119456f220a467cefe76b300
  Author: Duncan Paterson <dun...@us...>
  Date:   2026-08-18 (Tue, 18 Aug 2026)

  Changed paths:
    M exist-distribution/pom.xml

  Log Message:
  -----------
  Merge pull request #6602 from line-o/bugfix/6601-package-not-rerunnable

[bugfix] Make mvn package re-runnable


Compare: https://github.com/eXist-db/exist/compare/cc597c3fe4a6...df02f85975eb

To unsubscribe from these emails, change your notification settings at https://github.com/eXist-db/exist/settings/notifications