Skip to content
This repository was archived by the owner on Oct 13, 2023. It is now read-only.

Commit 2115479

Browse files
authored
Merge pull request #225 from seemethere/fix_metalinter_conversion_errors
[17.09] Fix metalinter conversion errors
2 parents be7f906 + 2b763fa commit 2115479

File tree

6 files changed

+9
-8
lines changed

6 files changed

+9
-8
lines changed

components/engine/Dockerfile.aarch64

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -173,7 +173,7 @@ RUN ./contrib/download-frozen-image-v2.sh /docker-frozen-images \
173173
# Please edit hack/dockerfile/install-binaries.sh to update them.
174174
COPY hack/dockerfile/binaries-commits /tmp/binaries-commits
175175
COPY hack/dockerfile/install-binaries.sh /tmp/install-binaries.sh
176-
RUN /tmp/install-binaries.sh tomlv vndr runc containerd tini proxy dockercli
176+
RUN /tmp/install-binaries.sh tomlv vndr runc containerd tini proxy dockercli gometalinter
177177
ENV PATH=/usr/local/cli:$PATH
178178

179179
# Wrap all commands in the "docker-in-docker" script to allow nested containers

components/engine/Dockerfile.armhf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,7 @@ RUN ./contrib/download-frozen-image-v2.sh /docker-frozen-images \
162162
# Please edit hack/dockerfile/install-binaries.sh to update them.
163163
COPY hack/dockerfile/binaries-commits /tmp/binaries-commits
164164
COPY hack/dockerfile/install-binaries.sh /tmp/install-binaries.sh
165-
RUN /tmp/install-binaries.sh tomlv vndr runc containerd tini proxy dockercli
165+
RUN /tmp/install-binaries.sh tomlv vndr runc containerd tini proxy dockercli gometalinter
166166
ENV PATH=/usr/local/cli:$PATH
167167

168168
ENTRYPOINT ["hack/dind"]

components/engine/Dockerfile.ppc64le

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,7 @@ RUN ./contrib/download-frozen-image-v2.sh /docker-frozen-images \
160160
# Please edit hack/dockerfile/install-binaries.sh to update them.
161161
COPY hack/dockerfile/binaries-commits /tmp/binaries-commits
162162
COPY hack/dockerfile/install-binaries.sh /tmp/install-binaries.sh
163-
RUN /tmp/install-binaries.sh tomlv vndr runc containerd tini proxy dockercli
163+
RUN /tmp/install-binaries.sh tomlv vndr runc containerd tini proxy dockercli gometalinter
164164
ENV PATH=/usr/local/cli:$PATH
165165

166166
# Wrap all commands in the "docker-in-docker" script to allow nested containers

components/engine/Dockerfile.s390x

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -153,7 +153,7 @@ RUN ./contrib/download-frozen-image-v2.sh /docker-frozen-images \
153153
# Please edit hack/dockerfile/install-binaries.sh to update them.
154154
COPY hack/dockerfile/binaries-commits /tmp/binaries-commits
155155
COPY hack/dockerfile/install-binaries.sh /tmp/install-binaries.sh
156-
RUN /tmp/install-binaries.sh tomlv vndr runc containerd tini proxy dockercli
156+
RUN /tmp/install-binaries.sh tomlv vndr runc containerd tini proxy dockercli gometalinter
157157
ENV PATH=/usr/local/cli:$PATH
158158

159159
# Wrap all commands in the "docker-in-docker" script to allow nested containers

components/engine/daemon/graphdriver/overlay/copy.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -157,9 +157,10 @@ func copyDir(srcDir, dstDir string, flags copyFlags) error {
157157
}
158158

159159
// system.Chtimes doesn't support a NOFOLLOW flag atm
160+
// nolint: unconvert
160161
if !isSymlink {
161-
aTime := time.Unix(stat.Atim.Sec, stat.Atim.Nsec)
162-
mTime := time.Unix(stat.Mtim.Sec, stat.Mtim.Nsec)
162+
aTime := time.Unix(int64(stat.Atim.Sec), int64(stat.Atim.Nsec))
163+
mTime := time.Unix(int64(stat.Mtim.Sec), int64(stat.Mtim.Nsec))
163164
if err := system.Chtimes(dstPath, aTime, mTime); err != nil {
164165
return err
165166
}

components/engine/pkg/archive/archive_unix.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,8 +50,8 @@ func setHeaderForSpecialDevice(hdr *tar.Header, name string, stat interface{}) (
5050
// Currently go does not fill in the major/minors
5151
if s.Mode&unix.S_IFBLK != 0 ||
5252
s.Mode&unix.S_IFCHR != 0 {
53-
hdr.Devmajor = int64(major(s.Rdev))
54-
hdr.Devminor = int64(minor(s.Rdev))
53+
hdr.Devmajor = int64(major(uint64(s.Rdev))) // nolint: unconvert
54+
hdr.Devminor = int64(minor(uint64(s.Rdev))) // nolint: unconvert
5555
}
5656
}
5757

0 commit comments

Comments
 (0)