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

bugfix for handling env vars that end in \\ #874

Merged
merged 1 commit into from
Mar 20, 2021
Merged

bugfix for handling env vars that end in \\ #874

merged 1 commit into from
Mar 20, 2021

Conversation

alexcb
Copy link
Collaborator

@alexcb alexcb commented Mar 20, 2021

  • env variables ending in \ erroneously wrapped to the second line;
    e.g.

    ENV foo \\
    RUN echo bar
    

    would end up setting foo="RUN echo bar"

Signed-off-by: Alex Couture-Beil alex@earthly.dev

@alexcb alexcb force-pushed the test-envs branch 2 times, most recently from 7a540e8 to bf134dd Compare March 20, 2021 00:08
- env variables ending in \\ erronorously wrapped to the second line;
  e.g.

    ENV foo \\
    RUN echo bar

  would end up setting foo="RUN echo bar"

Signed-off-by: Alex Couture-Beil <alex@earthly.dev>
@alexcb alexcb marked this pull request as ready for review March 20, 2021 00:20
@alexcb alexcb merged commit 9ff7970 into main Mar 20, 2021
@alexcb alexcb deleted the test-envs branch March 20, 2021 01:08
alexcb added a commit that referenced this pull request Mar 31, 2021
- env variables ending in \\ erronorously wrapped to the second line;
  e.g.

    ENV foo \\
    RUN echo bar

  would end up setting foo="RUN echo bar"

Signed-off-by: Alex Couture-Beil <alex@earthly.dev>
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.

None yet

2 participants