Skip to content

Build the JMEOS jar the way its own script defines it - #30

Merged
estebanzimanyi merged 1 commit into
MobilityDB:mainfrom
estebanzimanyi:fix/no-skipped-tests-in-the-jmeos-jar-build
Aug 31, 2026
Merged

estebanzimanyi merged 1 commit into
MobilityDB:mainfrom
estebanzimanyi:fix/no-skipped-tests-in-the-jmeos-jar-build

Conversation

@estebanzimanyi

Copy link
Copy Markdown
Member

JMEOS carries tools/regen-from-catalog.sh, the single runnable definition of
that build: it stages the catalog and libmeos, and jmeos-core's
generate-sources emits GeneratedFunctions from the catalog.
tools/refresh-from-master.sh reaches the same script through the shared
chain, so calling it from the workflow leaves the CI path and the by-hand path
unable to drift.

The workflow named -Dmaven.test.skip=true, which skips test COMPILATION as
well as execution, so a JMEOS whose test sources do not compile against the
catalog this job derived still produced a green jar step. Naming LIBMEOS runs
the FFI suite against the libmeos the job just built — 1792 tests, 0 failures,
0 errors, 0 skipped — which is what tells a jar that merely compiles from one
whose generated surface answers.

tools/refresh-from-master.sh no longer offers --skip-tests: the chain it
forwards to carries no such option, so the line documented a flag that is
refused.

JMEOS carries `tools/regen-from-catalog.sh`, the single runnable definition of
that build: it stages the catalog and libmeos, and jmeos-core's
generate-sources emits GeneratedFunctions from the catalog.
`tools/refresh-from-master.sh` reaches the same script through the shared
chain, so calling it from the workflow leaves the CI path and the by-hand path
unable to drift.

The workflow named `-Dmaven.test.skip=true`, which skips test COMPILATION as
well as execution, so a JMEOS whose test sources do not compile against the
catalog this job derived still produced a green jar step. Naming LIBMEOS runs
the FFI suite against the libmeos the job just built — 1792 tests, 0 failures,
0 errors, 0 skipped — which is what tells a jar that merely compiles from one
whose generated surface answers.

`tools/refresh-from-master.sh` no longer offers `--skip-tests`: the chain it
forwards to carries no such option, so the line documented a flag that is
refused.
@estebanzimanyi
estebanzimanyi merged commit bcbb369 into MobilityDB:main Aug 31, 2026
1 check passed
@estebanzimanyi
estebanzimanyi deleted the fix/no-skipped-tests-in-the-jmeos-jar-build branch August 31, 2026 16:19
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