Skip to content
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.

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

ARG double evaluation #4194

Open
ohhmm opened this issue Dec 27, 2023 · 2 comments
Open

ARG double evaluation #4194

ohhmm opened this issue Dec 27, 2023 · 2 comments

Comments

@ohhmm
Copy link

ohhmm commented Dec 27, 2023

When evaluated ARG contains another ARG reference it expected to be fully expanded.
Example line: https:/ohhmm/openmind/blob/e7e9170d4ebb86e32112ae0b4397db51925478f6/.devcontainer/Dockerfile#L4

ARG LLVM_MAJOR_VERSION=17
ARG LLVM_FULL_VERSION=$LLVM_MAJOR_VERSION.0.6

Hovering LLVM_FULL_VERSION hints $LLVM_MAJOR_VERSION.0.6 but 17.0.6 is expected.

@dbreshears
Copy link
Member

@rcjsuen, what are your thoughts on supporting this?

@dbreshears dbreshears added this to the Future milestone Jan 3, 2024
@rcjsuen
Copy link
Contributor

rcjsuen commented Jan 23, 2024

Very interesting use case. @ohhmm I guess it knows that the . is a word separator? If you use _ I presume Docker won't interpret things quite the same way?

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

No branches or pull requests

3 participants