aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKris Alder <kalder@google.com>2023-02-10 17:26:30 -0800
committerAleksandr Nogikh <wp32pw@gmail.com>2023-02-13 10:46:51 +0100
commit957959cbd5c40f4e7be0c363495e34454cd2fde6 (patch)
tree1756f861b1e74a14b704698e7f1a09bde45e7cee
parent93e26d605eebce2ad19e6f37d9e96a872ca55cce (diff)
pkg/build: rename android.go to cuttlefish.go
Building images for physical Android devices has different logic and options and should be in separate files. This makes it less ambiguous and frees up android.go for Android device code.
-rw-r--r--pkg/build/build.go2
-rw-r--r--pkg/build/cuttlefish.go (renamed from pkg/build/android.go)20
2 files changed, 11 insertions, 11 deletions
diff --git a/pkg/build/build.go b/pkg/build/build.go
index 9f56976de..2890894db 100644
--- a/pkg/build/build.go
+++ b/pkg/build/build.go
@@ -133,7 +133,7 @@ func getBuilder(targetOS, targetArch, vmType string) (builder, error) {
if vmType == "gvisor" {
return gvisor{}, nil
} else if vmType == "cuttlefish" {
- return android{}, nil
+ return cuttlefish{}, nil
}
}
builders := map[string]builder{
diff --git a/pkg/build/android.go b/pkg/build/cuttlefish.go
index 9d078a4ae..42ec8af26 100644
--- a/pkg/build/android.go
+++ b/pkg/build/cuttlefish.go
@@ -22,9 +22,9 @@ const (
bazelTarget = "//common-modules/virtual-device:virtual_device_x86_64_dist"
)
-type android struct{}
+type cuttlefish struct{}
-func (a android) runBuild(kernelDir, buildConfig string) error {
+func (c cuttlefish) runBuild(kernelDir, buildConfig string) error {
cmd := osutil.Command("build/build.sh")
cmd.Dir = kernelDir
cmd.Env = append(cmd.Env, "OUT_DIR=out", "DIST_DIR=dist", fmt.Sprintf("BUILD_CONFIG=%s", buildConfig))
@@ -33,14 +33,14 @@ func (a android) runBuild(kernelDir, buildConfig string) error {
return err
}
-func (a android) runBazel(kernelDir, buildConfig string) error {
+func (c cuttlefish) runBazel(kernelDir string) error {
cmd := osutil.Command("tools/bazel", "run", "--kasan", bazelTarget, "--", "--dist_dir=dist")
cmd.Dir = kernelDir
_, err := osutil.Run(time.Hour, cmd)
return err
}
-func (a android) readCompiler(archivePath string) (string, error) {
+func (c cuttlefish) readCompiler(archivePath string) (string, error) {
f, err := os.Open(archivePath)
if err != nil {
return "", err
@@ -74,7 +74,7 @@ func (a android) readCompiler(archivePath string) (string, error) {
return "", fmt.Errorf("archive %s doesn't contain include/generated/compile.h", archivePath)
}
-func (a android) build(params Params) (ImageDetails, error) {
+func (c cuttlefish) build(params Params) (ImageDetails, error) {
var details ImageDetails
if params.CmdlineFile != "" {
@@ -88,7 +88,7 @@ func (a android) build(params Params) (ImageDetails, error) {
var err error
// Default to build.sh if compiler is not specified.
if params.Compiler == "bazel" {
- if err := a.runBazel(params.KernelDir, bazelTarget); err != nil {
+ if err := c.runBazel(params.KernelDir); err != nil {
return details, fmt.Errorf("failed to build kernel: %s", err)
}
// Find the .config file; it is placed in a temporary output directory during the build.
@@ -100,10 +100,10 @@ func (a android) build(params Params) (ImageDetails, error) {
}
config = filepath.Join(params.KernelDir, strings.TrimSpace(string(configBytes)))
} else {
- if err := a.runBuild(params.KernelDir, kernelConfig); err != nil {
+ if err := c.runBuild(params.KernelDir, kernelConfig); err != nil {
return details, fmt.Errorf("failed to build kernel: %s", err)
}
- if err := a.runBuild(params.KernelDir, moduleConfig); err != nil {
+ if err := c.runBuild(params.KernelDir, moduleConfig); err != nil {
return details, fmt.Errorf("failed to build modules: %s", err)
}
config = filepath.Join(params.KernelDir, "out", "common", ".config")
@@ -114,7 +114,7 @@ func (a android) build(params Params) (ImageDetails, error) {
vmlinux := filepath.Join(buildDistDir, "vmlinux")
initramfs := filepath.Join(buildDistDir, "initramfs.img")
- details.CompilerID, err = a.readCompiler(filepath.Join(buildDistDir, "kernel-headers.tar.gz"))
+ details.CompilerID, err = c.readCompiler(filepath.Join(buildDistDir, "kernel-headers.tar.gz"))
if err != nil {
return details, err
}
@@ -155,6 +155,6 @@ func (a android) build(params Params) (ImageDetails, error) {
return details, nil
}
-func (a android) clean(kernelDir, targetArch string) error {
+func (c cuttlefish) clean(kernelDir, targetArch string) error {
return osutil.RemoveAll(filepath.Join(kernelDir, "out"))
}