Skip to content

Commit 5524d68

Browse files
committed
explicitly define usrLibMultiarchDir on supported platforms via build constraints
1 parent cc60ef2 commit 5524d68

File tree

3 files changed

+12
-8
lines changed

3 files changed

+12
-8
lines changed

dockerutil/image.go

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ import (
66
"encoding/json"
77
"fmt"
88
"io"
9-
goruntime "runtime"
109
"strings"
1110
"time"
1211

@@ -24,16 +23,11 @@ import (
2423

2524
const diskFullStorageDriver = "vfs"
2625

27-
var usrLibMultiarchDir = map[string]string{
28-
"arm64": "/usr/lib/aarch64-linux-gnu",
29-
"amd64": "/usr/lib/x86_64-linux-gnu",
30-
}
31-
3226
// Adapted from https://github.com/NVIDIA/libnvidia-container/blob/v1.15.0/src/nvc_container.c#L152-L165
3327
var UsrLibDirs = map[string]string{
3428
// Debian-based distros use a multi-arch directory.
35-
"debian": usrLibMultiarchDir[goruntime.GOARCH],
36-
"ubuntu": usrLibMultiarchDir[goruntime.GOARCH],
29+
"debian": usrLibMultiarchDir,
30+
"ubuntu": usrLibMultiarchDir,
3731
// Fedora and Redhat use the standard /usr/lib64.
3832
"fedora": "/usr/lib64",
3933
"rhel": "/usr/lib64",

dockerutil/image_linux_amd64.go

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
package dockerutil
2+
3+
// usrLibMultiarchDir is defined for arm64 and amd64 architectures.
4+
// Envbox is not published for other architectures.
5+
var usrLibMultiarchDir = "/usr/lib/aarch64-linux-gnu"

dockerutil/image_linux_arm64.go

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
package dockerutil
2+
3+
// usrLibMultiarchDir is defined for arm64 and amd64 architectures.
4+
// Envbox is not published for other architectures.
5+
var usrLibMultiarchDir = "/usr/lib/x86_64-linux-gnu"

0 commit comments

Comments
 (0)