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

Commit c1fd399c authored by Inseob Kim's avatar Inseob Kim
Browse files

Use installExecutable for cc binaries 

Normally InstallFile is enough because linker grants +x permission to
its output. Just to guarantee +x permission even when the source file
doesn't have one.

Test: build
Change-Id: I01082df11d414804e1a73336fc637a6a2208021f
parent cb301ccd

cc/binary.go

+1 −1
Original line number Diff line number Diff line
@@ -512,7 +512,7 @@ func (binary *binaryDecorator) install(ctx ModuleContext, file android.Path) { 
		}

		binary.baseInstaller.subDir = "bootstrap"

	}

	binary.baseInstaller.install(ctx, file)

	binary.baseInstaller.installExecutable(ctx, file)



	var preferredArchSymlinkPath android.OptionalPath

	for _, symlink := range binary.symlinks {

cc/installer.go

+4 −0
Original line number Diff line number Diff line
@@ -100,6 +100,10 @@ func (installer *baseInstaller) install(ctx ModuleContext, file android.Path) { 
	installer.path = ctx.InstallFile(installer.installDir(ctx), file.Base(), file)

}



func (installer *baseInstaller) installExecutable(ctx ModuleContext, file android.Path) {

	installer.path = ctx.InstallExecutable(installer.installDir(ctx), file.Base(), file)

}



func (installer *baseInstaller) everInstallable() bool {

	// Most cc modules are installable.

	return true