Merge "respect relative_install_path for binaries in APEXes"

This commit is contained in:
Treehugger Robot
2019-03-19 21:47:18 +00:00
committed by Gerrit Code Review
2 changed files with 4 additions and 7 deletions

View File

@@ -613,9 +613,7 @@ func getCopyManifestForNativeLibrary(cc *cc.Module, handleSpecialLibs bool) (fil
} }
func getCopyManifestForExecutable(cc *cc.Module) (fileToCopy android.Path, dirInApex string) { func getCopyManifestForExecutable(cc *cc.Module) (fileToCopy android.Path, dirInApex string) {
// TODO(b/123721777) respect relative_install_path also for binaries dirInApex = filepath.Join("bin", cc.RelativeInstallPath())
// dirInApex = filepath.Join("bin", cc.RelativeInstallPath())
dirInApex = "bin"
fileToCopy = cc.OutputFile().Path() fileToCopy = cc.OutputFile().Path()
return return
} }

View File

@@ -696,10 +696,9 @@ func TestFilesInSubDir(t *testing.T) {
ensureListContains(t, dirs, "lib/foo") ensureListContains(t, dirs, "lib/foo")
ensureListContains(t, dirs, "lib/foo/bar") ensureListContains(t, dirs, "lib/foo/bar")
// TODO(b/123721777) respect relative path for binaries ensureListContains(t, dirs, "bin")
// ensureListContains(t, dirs, "bin") ensureListContains(t, dirs, "bin/foo")
// ensureListContains(t, dirs, "bin/foo") ensureListContains(t, dirs, "bin/foo/bar")
// ensureListContains(t, dirs, "bin/foo/bar")
} }
func TestUseVendor(t *testing.T) { func TestUseVendor(t *testing.T) {