Skip to content

Fix source-build prerequisites for Maven 3.9+ and Yarn 4 - #4485

Open
AshSgDe29071999 wants to merge 1 commit into
dbeaver:develfrom
AshSgDe29071999:fix/3839-build-prerequisites
Open

Fix source-build prerequisites for Maven 3.9+ and Yarn 4#4485
AshSgDe29071999 wants to merge 1 commit into
dbeaver:develfrom
AshSgDe29071999:fix/3839-build-prerequisites

Conversation

@AshSgDe29071999

@AshSgDe29071999 AshSgDe29071999 commented Jul 22, 2026

Copy link
Copy Markdown

Summary

  • Document correct Ubuntu/Debian install steps for building from source: official Maven 3.9.9+ and Yarn 4 via Corepack (apt maven/yarn break Tycho 5 / frontend builds).
  • Add deploy/check-prerequisites.sh and run it from deploy/build.sh so incompatible tool versions fail early with a clear message.
  • Allow optional DBEAVER_GIT_REF / DBEAVER_COMMON_GIT_REF when cloning platform repos for reproducible release builds.

Fixes #3839

Test plan

  • bash -n on updated shell scripts
  • ./deploy/check-prerequisites.sh reports Maven 3.8.7 (apt) as too old with a clear error
  • Confirm wiki Build and deploy Ubuntu section is updated to match (or mirror deploy/README.md)

Ubuntu apt packages often provide Maven < 3.9.9 and Yarn 1.x, which break
Tycho 5 / frontend builds. Document the correct install path and fail early
in deploy/build.sh when tool versions are incompatible.

Also allow pinning dbeaver / dbeaver-common clone refs via DBEAVER_GIT_REF
for reproducible release builds.

Fixes dbeaver#3839
@AshSgDe29071999
AshSgDe29071999 force-pushed the fix/3839-build-prerequisites branch from 5f6d8bb to 239912b Compare July 22, 2026 12:22
@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@LonwoLonwo LonwoLonwo added the external For external PRs label Jul 22, 2026
@LonwoLonwo

Copy link
Copy Markdown
Member

Hello
Thanks for your PR
We will take a look

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

external For external PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Failure to build following build-and-deploy instructions

2 participants