Skip to content

#2271: fix pgadmin installation on linux - #2388

Open
cap-juan wants to merge 1 commit into
devonfw:mainfrom
cap-juan:feature/2271-ide-install-pgadmin-fails-linux
Open

#2271: fix pgadmin installation on linux#2388
cap-juan wants to merge 1 commit into
devonfw:mainfrom
cap-juan:feature/2271-ide-install-pgadmin-fails-linux

Conversation

@cap-juan

@cap-juan cap-juan commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

This PR fixes #2271

Implemented changes:

  • Fixed ide install pgadmin on Linux when pgAdmin is installed via native packages.
  • Adjusted version resolution so Linux package managers can determine the installed version when no exact version is requested.
  • Preserved explicitly requested exact versions.
  • Detects and uses the actually installed version after native package installation.
  • Native uninstall handling is only used for tools that actually define native packages.

Testing instructions

Please add concise, understandable instructions on how a reviewer can test/verify the functionality of your contribution here:

  1. On Linux (e.g. Ubuntu or WSL), make sure pgAdmin is not installed.
  2. Build IDEasy locally over ./build-local-dev.sh
  3. Run: ide install pgadmin

Checklist for this PR

Make sure everything is checked before merging this PR. For further info please also see
our DoD.

  • When running mvn clean test locally all tests pass and build is successful
  • PR title is of the form #«issue-id»: «brief summary» (e.g. #921: fixed setup.bat and not feature/921 fixed setup.bat). If no issue ID exists, title only.
  • PR top-level comment summaries what has been done and contains link to addressed issue(s)
  • PR and issue(s) have suitable labels
  • Issue is set to In Progress and assigned to you or there is no issue (might happen for very small PRs)
  • You followed all coding conventions
  • You have added the issue implemented by your PR in CHANGELOG.adoc unless issue is labelled
    with internal
  • You have not changed any dependency in pom.xml files or otherwise if runtime dependencies changed, you have updated our LICENSE.asciidoc
  • You have formulated clear instructions on how to test your contribution under "Testing instructions"

Checklist for tool commandlets

Have you added a new «tool» as commandlet? There are the following additional checks:

  • The tool can be installed automatically (during setup via settings) or via the commandlet call
  • The tool is isolated in its IDEasy project, see Sandbox Principle
  • The new tool is added to the table of tools in LICENSE.asciidoc
  • The new commandlet is a command-wrapper for «tool»
  • Proper help texts for all supported languages are added here
  • The new commandlet installs potential dependencies automatically
  • The variables «TOOL»_VERSION and «TOOL»_EDITION are honored by your commandlet
  • The new commandlet is tested on all platforms it is available for or tested on all platforms that are in scope of the linked issue

@github-project-automation github-project-automation Bot moved this to 🆕 New in IDEasy board Aug 28, 2026
@cap-juan cap-juan added linux specific for linux OS (debian, ubunutu, suse, etc.) commandlet ide sub-command CLI IDEasy command-line-interface (parsing args, etc.) labels Aug 28, 2026
@cap-juan cap-juan moved this from 🆕 New to Team Review in IDEasy board Aug 28, 2026
@coveralls

coveralls commented Aug 28, 2026

Copy link
Copy Markdown
Collaborator

Coverage Report for CI Build 33490596893

Coverage increased (+0.006%) to 73.619%

Details

  • Coverage increased (+0.006%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 90 coverage regressions across 2 files.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

90 previously-covered lines in 2 files lost coverage.

File Lines Losing Coverage Coverage
com/devonfw/tools/ide/tool/GlobalToolCommandlet.java 51 14.01%
com/devonfw/tools/ide/tool/ToolCommandlet.java 39 76.05%

Coverage Stats

Coverage Status
Relevant Lines: 18369
Covered Lines: 14136
Line Coverage: 76.96%
Relevant Branches: 8146
Covered Branches: 5384
Branch Coverage: 66.09%
Branches in Coverage %: Yes
Coverage Strength: 3.28 hits per line

💛 - Coveralls

@cap-juan
cap-juan force-pushed the feature/2271-ide-install-pgadmin-fails-linux branch from 4c47ff0 to eddf18b Compare August 28, 2026 11:47
@JoelAdbu JoelAdbu self-assigned this Aug 31, 2026

@JoelAdbu JoelAdbu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would also suggest updating the current pgAdmin URL updater, as it does not support Linux versions yet. The pgAdmin download repository provides Linux packages (APT and YUM) in addition to the existing artifacts. You can find more information here: pgAdmin download repository.

@cap-juan

cap-juan commented Sep 1, 2026

Copy link
Copy Markdown
Contributor Author

I would also suggest updating the current pgAdmin URL updater, as it does not support Linux versions yet. The pgAdmin download repository provides Linux packages (APT and YUM) in addition to the existing artifacts. You can find more information here: pgAdmin download repository.

Thanks, I looked into this. The pgAdmin Linux repository differs from the Windows artifacts: the APT and YUM packages are distribution-specific (e.g. Noble, Bookworm/Trixie, Fedora/RHEL), while the current URL updater model only distinguishes OS and architecture and has no Linux distribution dimension. Because of that, adding a single LINUX URL would potentially generate incorrect metadata for other distributions. The runtime installation in this PR intentionally uses the native package manager/repository instead. I think proper Linux support in PgAdminUrlUpdater would require distro-aware URL metadata (URL metadata that knows whether a Linux artifact is for Ubuntu, Debian, Fedora, RHEL, openSUSE, etc) or separate handling for package repositories.

@cap-juan
cap-juan force-pushed the feature/2271-ide-install-pgadmin-fails-linux branch from eddf18b to 5a51554 Compare September 1, 2026 09:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLI IDEasy command-line-interface (parsing args, etc.) commandlet ide sub-command linux specific for linux OS (debian, ubunutu, suse, etc.)

Projects

Status: Team Review

Development

Successfully merging this pull request may close these issues.

ide install pgadmin fails on Ubuntu Linux

3 participants