New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Smart getProperty(... was too smart with container runtime property #176

Merged

Karm merged 1 commit into master from property-regression-switching-podman

Aug 9, 2023

Owner

Karm commented Aug 8, 2023

Regression caused by fed26dd, happens for some podman settings on command line, e.g.

mvn clean verify -Ptestsuite-builder-image -Dquarkus.version=3.2.3.Final -Dquarkus.native.container-runtime=podman

must work too. Not just QUARKUS_NATIVE_CONTAINER_RUNTIME env prop.


          Smart getProperty(... was too smart with this one. -Dquarkus.native.c…

f7ae5b0

…ontainer-runtime must work too.

Karm self-assigned this

jerboaa approved these changes

View reviewed changes

Collaborator

jerboaa left a comment

LGTM, but this transformation of alternative names takes a mental toll :-/ I've once run into this weirdness myself.

Karm merged commit a1c49ae into master

36 checks passed

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

Labels

None yet