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

Resolve environment variables case-insensitively on Windows #9438

Merged
merged 4 commits into from Aug 2, 2022

Conversation

ikedam
Copy link
Contributor

@ikedam ikedam commented May 4, 2022

What I did

This change resolves following environment variables case-insensitively on Windows:

  • Environment variables specified with services.SERVICE.build.args[].
  • Environment variables passed with --arg options in build sub command.
  • Environment variables passed with -e options in run sub command.

This doesn't fix followings. It will be fixed with compose-spec/compose-go#257 (and updating references in go.mod/go.sum)

  • Environment variables specified with services.SERVICE.environment[].

Related issue

(not mandatory) A picture of a cute animal, if possible in relation with what you did

Signed-off-by: IKEDA Yasuyuki <devld@ikedam.jp>
@ikedam
Copy link
Contributor Author

ikedam commented Jun 18, 2022

Looks conflicted with #9499

ikedam and others added 2 commits June 21, 2022 12:57
@ikedam
Copy link
Contributor Author

ikedam commented Jul 30, 2022

Oops, conflicted again (with #9636).
I'm adding a new commit to resolve.

…itiveEnv

# Conflicts:
#	pkg/compose/run.go

Signed-off-by: IKEDA Yasuyuki <devld@ikedam.jp>
@milas milas merged commit bbaaa6a into docker:v2 Aug 2, 2022
laurentsimon pushed a commit to laurentsimon/compose that referenced this pull request Aug 10, 2022
Signed-off-by: IKEDA Yasuyuki <devld@ikedam.jp>
@ikedam
Copy link
Contributor Author

ikedam commented Sep 3, 2022

Fixed in v2.10.0: https://github.com/docker/compose/releases/tag/v2.10.0

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

Successfully merging this pull request may close these issues.

None yet

2 participants