Skip to content

Commit

Permalink
Merge pull request #2252 from mtrmac/go1.20
Browse files Browse the repository at this point in the history
Update to Go 1.20
  • Loading branch information
rhatdan authored Apr 8, 2024
2 parents c9dc0ef + a75ed81 commit a374215
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 11 deletions.
10 changes: 3 additions & 7 deletions cmd/skopeo/list_tags.go
Original file line number Diff line number Diff line change
Expand Up @@ -78,16 +78,12 @@ See skopeo-list-tags(1) section "REPOSITORY NAMES" for the expected format
// Customized version of the alltransports.ParseImageName and docker.ParseReference that does not place a default tag in the reference
// Would really love to not have this, but needed to enforce tag-less and digest-less names
func parseDockerRepositoryReference(refString string) (types.ImageReference, error) {
if !strings.HasPrefix(refString, docker.Transport.Name()+"://") {
dockerRefString, ok := strings.CutPrefix(refString, docker.Transport.Name()+"://")
if !ok {
return nil, fmt.Errorf("docker: image reference %s does not start with %s://", refString, docker.Transport.Name())
}

_, dockerImageName, hasColon := strings.Cut(refString, ":")
if !hasColon {
return nil, fmt.Errorf(`Invalid image name "%s", expected colon-separated transport:reference`, refString)
}

ref, err := reference.ParseNormalizedNamed(strings.TrimPrefix(dockerImageName, "//"))
ref, err := reference.ParseNormalizedNamed(dockerRefString)
if err != nil {
return nil, err
}
Expand Down
2 changes: 1 addition & 1 deletion go.mod
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
module github.com/containers/skopeo

// Minimum required golang version
go 1.19 // ***** ATTENTION WARNING CAUTION DANGER ******
go 1.20 // ***** ATTENTION WARNING CAUTION DANGER ******

// Go versions 1.21 and later will AUTO-UPDATE based
// on currently running tools and the (new) `toolchain`
Expand Down
2 changes: 1 addition & 1 deletion install.md
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,7 @@ located at [https://github.com/containers/image_build/tree/main/skopeo](https://

Otherwise, read on for building and installing it from source:

To build the `skopeo` binary you need at least Go 1.19.
To build the `skopeo` binary you need at least Go 1.20.

There are two ways to build skopeo: in a container, or locally without a
container. Choose the one which better matches your needs and environment.
Expand Down
4 changes: 2 additions & 2 deletions integration/utils.go
Original file line number Diff line number Diff line change
Expand Up @@ -268,8 +268,8 @@ func decompressDir(t *testing.T, dir string) {
rawLayer["size"] = uncompressedSize
var mimeType string
getRawMapField(t, rawLayer, "mediaType", &mimeType)
if strings.HasSuffix(mimeType, ".gzip") { // This should use CutSuffix with Go ≥1.20
rawLayer["mediaType"] = strings.TrimSuffix(mimeType, ".gzip")
if uncompressedMIMEType, ok := strings.CutSuffix(mimeType, ".gzip"); ok {
rawLayer["mediaType"] = uncompressedMIMEType
}

rawLayers[i] = rawLayer
Expand Down

0 comments on commit a374215

Please sign in to comment.