Skip to content

Commit dda2da9

Browse files
authored
Merge pull request moby#3979 from AkihiroSuda/graduate-addgit
dockerfile: graduate `ADD <git ref>` and `ADD --checksum=<checksum>` from labs
2 parents 9bc05a1 + 11a2c6a commit dda2da9

File tree

10 files changed

+3
-39
lines changed

10 files changed

+3
-39
lines changed

.golangci.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,6 @@ run:
77

88
build-tags:
99
- dfrunsecurity
10-
- dfaddgit
11-
- dfaddchecksum
1210

1311
linters:
1412
enable:

frontend/dockerfile/dockerfile2llb/convert.go

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1055,9 +1055,6 @@ func dispatchCopy(d *dispatchState, cfg copyConfig) error {
10551055
if !cfg.isAddCommand {
10561056
return errors.New("checksum can't be specified for COPY")
10571057
}
1058-
if !addChecksumEnabled {
1059-
return errors.New("instruction 'ADD --checksum=<CHECKSUM>' requires the labs channel")
1060-
}
10611058
if len(cfg.params.SourcePaths) != 1 {
10621059
return errors.New("checksum can't be specified for multiple sources")
10631060
}
@@ -1082,9 +1079,6 @@ func dispatchCopy(d *dispatchState, cfg copyConfig) error {
10821079
if !cfg.isAddCommand {
10831080
return errors.New("source can't be a git ref for COPY")
10841081
}
1085-
if !addGitEnabled {
1086-
return errors.New("instruction ADD <git ref> requires the labs channel")
1087-
}
10881082
// TODO: print a warning (not an error) if gitRef.UnencryptedTCP is true
10891083
commit := gitRef.Commit
10901084
if gitRef.SubDir != "" {

frontend/dockerfile/dockerfile2llb/convert_addchecksum.go

Lines changed: 0 additions & 6 deletions
This file was deleted.

frontend/dockerfile/dockerfile2llb/convert_addgit.go

Lines changed: 0 additions & 6 deletions
This file was deleted.

frontend/dockerfile/dockerfile2llb/convert_noaddchecksum.go

Lines changed: 0 additions & 6 deletions
This file was deleted.

frontend/dockerfile/dockerfile2llb/convert_noaddgit.go

Lines changed: 0 additions & 6 deletions
This file was deleted.

frontend/dockerfile/dockerfile_addchecksum_test.go

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,3 @@
1-
//go:build dfaddchecksum
2-
// +build dfaddchecksum
3-
41
package dockerfile
52

63
import (

frontend/dockerfile/dockerfile_addgit_test.go

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,3 @@
1-
//go:build dfaddgit
2-
// +build dfaddgit
3-
41
package dockerfile
52

63
import (

frontend/dockerfile/docs/reference.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1298,6 +1298,7 @@ guide – Leverage build cache](https://docs.docker.com/develop/develop-images/d
12981298
> **Note**
12991299
>
13001300
> Not yet available in stable syntax, use [`docker/dockerfile:1-labs`](#syntax) version (`1.5-labs` or newer).
1301+
> Planned to be included in `docker/dockerfile:1.6`.
13011302
13021303
The checksum of a remote file can be verified with the `--checksum` flag:
13031304

@@ -1312,6 +1313,7 @@ The `--checksum` flag only supports HTTP sources currently.
13121313
> **Note**
13131314
>
13141315
> Not yet available in stable syntax, use [`docker/dockerfile:1-labs`](#syntax) version (`1.5-labs` or newer).
1316+
> Planned to be included in `docker/dockerfile:1.6`.
13151317
13161318
This form allows adding a git repository to an image directly, without using the `git` command inside the image:
13171319
```

frontend/dockerfile/release/labs/tags

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
dfrunsecurity dfaddgit dfaddchecksum
1+
dfrunsecurity

0 commit comments

Comments
 (0)