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

Commit 02adec80 authored by Pechetty Sravani (xWF)'s avatar Pechetty Sravani (xWF) Committed by Gerrit Code Review
Browse files

Revert "Use a partition packaging spec filter for android_filesystem" 

This reverts commit 8a49643e.

Reason for revert: <DroidMonitor created revert due to b/374395492.Will be verified through ABTD for standard investigation>

Change-Id: I4494f15f1681bf2456439151771e84c7d1f221b7
parent 8a49643e

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() && (ps.Partition() == f.PartitionType())

	return !ps.SkipInstall()

}



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

filesystem/filesystem_test.go

+0 −25
Original line number Diff line number Diff line
@@ -584,28 +584,3 @@ 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 !ps.SkipInstall() &&

	return s.filesystem.filterInstallablePackagingSpec(ps) &&

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

}