Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit adf591a5 authored by Ramy Medhat's avatar Ramy Medhat Committed by Kousik Kumar
Browse files

[DO NOT MERGE] Add USE_RBE support to soong. 

Test: Built aosp_arm-user with and without USE_RBE. USE_RBE uses
a proxy script in place of rewrapper.

Bug: b/166182389
Change-Id: I5bf008a940513872d70b5b215bd6209f759826ae
Merged-In: I5bf008a940513872d70b5b215bd6209f759826ae
parent 81aa8a42

android/config.go

+4 −0
Original line number Diff line number Diff line
@@ -714,6 +714,10 @@ func (c *config) UseGoma() bool { 
	return Bool(c.productVariables.UseGoma)

}



func (c *config) UseRBE() bool {

	return Bool(c.productVariables.UseRBE)

}



func (c *config) RunErrorProne() bool {

	return c.IsEnvTrue("RUN_ERROR_PRONE")

}

android/variable.go

+1 −0
Original line number Diff line number Diff line
@@ -192,6 +192,7 @@ type productVariables struct { 
	HostStaticBinaries               *bool `json:",omitempty"`

	Binder32bit                      *bool `json:",omitempty"`

	UseGoma                          *bool `json:",omitempty"`

	UseRBE                           *bool `json:",omitempty"`

	Debuggable                       *bool `json:",omitempty"`

	Eng                              *bool `json:",omitempty"`

	Treble_linker_namespaces         *bool `json:",omitempty"`

ui/build/Android.bp

+1 −0
Original line number Diff line number Diff line
@@ -52,6 +52,7 @@ bootstrap_go_package { 
        "ninja.go",

        "path.go",

        "proc_sync.go",

        "rbe.go",

        "signal.go",

        "soong.go",

        "test_build.go",

ui/build/build.go

+5 −0
Original line number Diff line number Diff line
@@ -161,6 +161,11 @@ func Build(ctx Context, config Config, what int) { 
		startGoma(ctx, config)

	}



	if config.StartRBE() {

		// Ensure RBE proxy is started

		startRBE(ctx, config)

	}



	if what&BuildProductConfig != 0 {

		// Run make for product config

		runMakeProductConfig(ctx, config)

ui/build/config.go

+24 −0
Original line number Diff line number Diff line
@@ -493,6 +493,30 @@ func (c *configImpl) StartGoma() bool { 
	return true

}



func (c *configImpl) UseRBE() bool {

	if v, ok := c.environ.Get("USE_RBE"); ok {

		v = strings.TrimSpace(v)

		if v != "" && v != "false" {

			return true

		}

	}

	return false

}



func (c *configImpl) StartRBE() bool {

	if !c.UseRBE() {

		return false

	}



	if v, ok := c.environ.Get("NOSTART_RBE"); ok {

		v = strings.TrimSpace(v)

		if v != "" && v != "false" {

			return false

		}

	}

	return true

}



// RemoteParallel controls how many remote jobs (i.e., commands which contain

// gomacc) are run in parallel.  Note the parallelism of all other jobs is

// still limited by Parallel()