diff options
| author | Kris Alder <kalder@google.com> | 2023-02-10 17:26:30 -0800 |
|---|---|---|
| committer | Aleksandr Nogikh <wp32pw@gmail.com> | 2023-02-13 10:46:51 +0100 |
| commit | 957959cbd5c40f4e7be0c363495e34454cd2fde6 (patch) | |
| tree | 1756f861b1e74a14b704698e7f1a09bde45e7cee /pkg | |
| parent | 93e26d605eebce2ad19e6f37d9e96a872ca55cce (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.
Diffstat (limited to 'pkg')
| -rw-r--r-- | pkg/build/build.go | 2 | ||||
| -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")) } |
