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

Commit 69d829a1 authored by Hugo Drumond Jacob's avatar Hugo Drumond Jacob Committed by Gerrit Code Review
Browse files

Revert^2 "Use a partition packaging spec filter for android_filesystem" 

This reverts commit 02adec80.

Reason for revert: b/374239899

Bug: 374239899
Change-Id: I263c11d02d26ea150484c185745cf58253352769
parent 02adec80

filesystem/filesystem.go

+1 −1
Original line number Diff line number Diff line
@@ -264,7 +264,7 @@ func (f *filesystem) partitionName() string { 
func (f *filesystem) filterInstallablePackagingSpec(ps android.PackagingSpec) bool {

	// Filesystem module respects the installation semantic. A PackagingSpec from a module with

	// IsSkipInstall() is skipped.

	return !ps.SkipInstall()

	return !ps.SkipInstall() && (ps.Partition() == f.PartitionType())

}



var pctx = android.NewPackageContext("android/soong/filesystem")

filesystem/filesystem_test.go

+25 −0
Original line number Diff line number Diff line
@@ -584,3 +584,28 @@ func TestErofsPartition(t *testing.T) { 
	android.AssertStringDoesContain(t, "erofs fs type compress hint", buildImageConfig, "erofs_default_compress_hints=compress_hints.txt")

	android.AssertStringDoesContain(t, "erofs fs type sparse", buildImageConfig, "erofs_sparse_flag=-s")

}



// If a system_ext/ module depends on system/ module, the dependency should *not*

// be installed in system_ext/

func TestDoNotPackageCrossPartitionDependencies(t *testing.T) {

	result := fixture.RunTestWithBp(t, `

		android_filesystem {

			name: "myfilesystem",

			deps: ["binfoo"],

			partition_type: "system_ext",

		}



		cc_binary {

			name: "binfoo",

			shared_libs: ["libfoo"],

			system_ext_specific: true,

		}

		cc_library_shared {

			name: "libfoo", // installed in system/

		}

	`)



	partition := result.ModuleForTests("myfilesystem", "android_common")

	fileList := android.ContentFromFileRuleForTests(t, result.TestContext, partition.Output("fileList"))

	android.AssertDeepEquals(t, "filesystem with dependencies on different partition", "bin/binfoo\n", fileList)

}

filesystem/system_image.go

+1 −1
Original line number Diff line number Diff line
@@ -103,6 +103,6 @@ func (s *systemImage) buildLinkerConfigFile(ctx android.ModuleContext, root andr 
// partition.  Note that "apex" module installs its contents to "apex"(fake partition) as well

// for symbol lookup by imitating "activated" paths.

func (s *systemImage) filterPackagingSpec(ps android.PackagingSpec) bool {

	return s.filesystem.filterInstallablePackagingSpec(ps) &&

	return !ps.SkipInstall() &&

		(ps.Partition() == "system" || ps.Partition() == "root")

}