Use unix.Uname instead of shelling out to uname on darwin/freebsd

Reuse the linux implementation based on Uname from golang.org/x/sys/unix
for darwin and freebsd.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
This commit is contained in:
Tobias Klauser 2019-09-02 15:56:46 +02:00
parent d99b0302d3
commit 7aeb3efcb4
3 changed files with 13 additions and 31 deletions

View File

@ -3,19 +3,19 @@
package operatingsystem // import "github.com/docker/docker/pkg/parsers/operatingsystem" package operatingsystem // import "github.com/docker/docker/pkg/parsers/operatingsystem"
import ( import (
"bytes"
"errors" "errors"
"os/exec"
"strings" "golang.org/x/sys/unix"
) )
// GetOperatingSystem gets the name of the current operating system. // GetOperatingSystem gets the name of the current operating system.
func GetOperatingSystem() (string, error) { func GetOperatingSystem() (string, error) {
cmd := exec.Command("uname", "-s") utsname := &unix.Utsname{}
osName, err := cmd.Output() if err := unix.Uname(utsname); err != nil {
if err != nil {
return "", err return "", err
} }
return strings.TrimSpace(string(osName)), nil return string(utsname.Machine[:bytes.IndexByte(utsname.Sysname[:], 0)]), nil
} }
// GetOperatingSystemVersion gets the version of the current operating system, as a string. // GetOperatingSystemVersion gets the version of the current operating system, as a string.

View File

@ -1,18 +0,0 @@
// Package platform provides helper function to get the runtime architecture
// for different platforms.
package platform // import "github.com/docker/docker/pkg/platform"
import (
"bytes"
"golang.org/x/sys/unix"
)
// runtimeArchitecture gets the name of the current architecture (x86, x86_64, …)
func runtimeArchitecture() (string, error) {
utsname := &unix.Utsname{}
if err := unix.Uname(utsname); err != nil {
return "", err
}
return string(utsname.Machine[:bytes.IndexByte(utsname.Machine[:], 0)]), nil
}

View File

@ -1,20 +1,20 @@
// +build freebsd darwin // +build !windows
// Package platform provides helper function to get the runtime architecture // Package platform provides helper function to get the runtime architecture
// for different platforms. // for different platforms.
package platform // import "github.com/docker/docker/pkg/platform" package platform // import "github.com/docker/docker/pkg/platform"
import ( import (
"os/exec" "bytes"
"strings"
"golang.org/x/sys/unix"
) )
// runtimeArchitecture gets the name of the current architecture (x86, x86_64, i86pc, sun4v, ...) // runtimeArchitecture gets the name of the current architecture (x86, x86_64, i86pc, sun4v, ...)
func runtimeArchitecture() (string, error) { func runtimeArchitecture() (string, error) {
cmd := exec.Command("/usr/bin/uname", "-m") utsname := &unix.Utsname{}
machine, err := cmd.Output() if err := unix.Uname(utsname); err != nil {
if err != nil {
return "", err return "", err
} }
return strings.TrimSpace(string(machine)), nil return string(utsname.Machine[:bytes.IndexByte(utsname.Machine[:], 0)]), nil
} }