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

syntax: add support for --link #84

Merged
merged 1 commit into from
Apr 4, 2022
Merged

Conversation

m-ildefons
Copy link
Contributor

  • Add support for parsing --link flag in COPY and ADD
    instructions.

With Dockerfile syntax version 1.4, the --link flag for the COPY and
ADD instructions become officially supported. The flag changes the way
the layers are treated during build such that caching can work more
efficient.

See also: https://github.com/moby/buildkit/blob/master/frontend/dockerfile/docs/syntax.md

This change modifies the AST and is not backwards compatible. For COPY
and ADD instructions, the arguments are now separated from the flags.

This also allows the creation of a default instance for the data
structure containing the flags.

fixes: #81

- Add support for parsing `--link` flag in `COPY` and `ADD`
  instructions.

With Dockerfile syntax version 1.4, the `--link` flag for the `COPY` and
`ADD` instructions become officially supported. The flag changes the way
the layers are treated during build such that caching can work more
efficient.

See also: https://github.com/moby/buildkit/blob/master/frontend/dockerfile/docs/syntax.md

This change modifies the AST and is not backwards compatible. For `COPY`
and `ADD` instructions, the arguments are now separated from the flags.

This also allows the creation of a default instance for the data
structure containing the flags.

fixes: hadolint#81
@lorenzo
Copy link
Member

lorenzo commented Apr 4, 2022

thanks!

@lorenzo lorenzo merged commit 79e9b7d into hadolint:master Apr 4, 2022
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.

Add --link flag to COPY/ADD instrructions
2 participants