Skip to content

Resolve environment variables case-insensitively on Windows#9438

Merged
milas merged 4 commits intodocker:v2from
ikedam:9431-CaseInsensitiveEnv
Aug 2, 2022
Merged

Resolve environment variables case-insensitively on Windows#9438
milas merged 4 commits intodocker:v2from
ikedam:9431-CaseInsensitiveEnv

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.

2 participants