Skip to content

[backport camel-4.18.x] CAMEL-23828: Honor explicit --quarkus-version during export#24230

Merged
davsclaus merged 1 commit into
apache:camel-4.18.xfrom
JiriOndrusek:export-with-quarkus-version
Jun 25, 2026
Merged

[backport camel-4.18.x] CAMEL-23828: Honor explicit --quarkus-version during export#24230
davsclaus merged 1 commit into
apache:camel-4.18.xfrom
JiriOndrusek:export-with-quarkus-version

Conversation

@JiriOndrusek

Copy link
Copy Markdown
Contributor

fixes https://issues.apache.org/jira/browse/CAMEL-23828

When the user specifies --quarkus-version on the CLI, via application.properties, or via system property, the export command should use that exact version instead of resolving from the Quarkus Extension Registry. Previously the registry resolution was unconditional and would overwrite the user's version with the latest patch for that stream.

The fix removes the default value from the --quarkus-version option so the field starts as null. ExportQuarkus now only applies the build-time default and registry resolution when no explicit version was provided.

Description

Target

  • I checked that the commit is targeting the correct branch (Camel 4 uses the main branch)

Tracking

  • If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

  • I checked that each commit in the pull request has a meaningful subject line and body.
  • I have run mvn clean install -DskipTests locally from root folder and I have committed all auto-generated changes.

@davsclaus

Copy link
Copy Markdown
Contributor

How does this work on the main branch? We cannot make special code on 4.18.x if its not similar/same as the main branch

@JiriOndrusek

JiriOndrusek commented Jun 24, 2026

Copy link
Copy Markdown
Contributor Author

How does this work on the main branch? We cannot make special code on 4.18.x if its not similar/same as the main branch

In upstream, there was a big change https://issues.apache.org/jira/browse/CAMEL-23560 and part of it is QuarkusPlatformMixin, which provides the same functionality

(I mentioned the upstream fix in the ticket for 4.18.x, but I should mention it in the PR as well)

@davsclaus

Copy link
Copy Markdown
Contributor

Ah okay, then this is fine

When the user specifies --quarkus-version on the CLI, via
application.properties, or via system property, the export command
should use that exact version instead of resolving from the Quarkus
Extension Registry. Previously the registry resolution was unconditional
and would overwrite the user's version with the latest patch for that
stream.

The fix removes the default value from the --quarkus-version option so
the field starts as null. ExportQuarkus now only applies the build-time
default and registry resolution when no explicit version was provided.
@JiriOndrusek JiriOndrusek force-pushed the export-with-quarkus-version branch from 14d3116 to f1d32d0 Compare June 25, 2026 07:16
@davsclaus davsclaus merged commit 651664f into apache:camel-4.18.x Jun 25, 2026
3 checks passed
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.

2 participants