Merge "Revert^3 "Set the appropriate deps property for the soong generated fs modules"" into main (20c3e5ef) · Commits · e / os / android_build_soong

android/module.go

+0 −8

Original line number	Diff line number	Diff line
		@@ -113,10 +113,6 @@ type Module interface {
		VintfFragmentModuleNames(ctx ConfigurableEvaluatorContext) []string

		ConfigurableEvaluator(ctx ConfigurableEvaluatorContext) proptools.ConfigurableEvaluator

		// The usage of this method is experimental and should not be used outside of fsgen package.
		// This will be removed once product packaging migration to Soong is complete.
		DecodeMultilib(ctx ConfigContext) (string, string)
		}

		// Qualified id for a module
		@@ -2286,10 +2282,6 @@ func (m *ModuleBase) IsNativeBridgeSupported() bool {
		return proptools.Bool(m.commonProperties.Native_bridge_supported)
		}

		func (m *ModuleBase) DecodeMultilib(ctx ConfigContext) (string, string) {
		return decodeMultilib(ctx, m)
		}

		type ConfigContext interface {
		Config() Config
		}

+0 −4

Original line number	Diff line number	Diff line
		@@ -201,7 +201,3 @@ func (m ModuleProxy) VintfFragmentModuleNames(ctx ConfigurableEvaluatorContext)
		func (m ModuleProxy) ConfigurableEvaluator(ctx ConfigurableEvaluatorContext) proptools.ConfigurableEvaluator {
		panic("method is not implemented on ModuleProxy")
		}

		func (m ModuleProxy) DecodeMultilib(ctx ConfigContext) (string, string) {
		panic("method is not implemented on ModuleProxy")
		}

+0 −9

Original line number	Diff line number	Diff line
		@@ -61,7 +61,6 @@ func init() {
		AddNeverAllowRules(createProhibitHeaderOnlyRule())
		AddNeverAllowRules(createLimitNdkExportRule()...)
		AddNeverAllowRules(createLimitDirgroupRule()...)
		AddNeverAllowRules(createFilesystemIsAutoGeneratedRule())
		}

		// Add a NeverAllow rule to the set of rules to apply.
		@@ -294,14 +293,6 @@ func createLimitDirgroupRule() []Rule {
		}
		}

		func createFilesystemIsAutoGeneratedRule() Rule {
		return NeverAllow().
		NotIn("build/soong/fsgen").
		ModuleType("filesystem", "android_system_image").
		WithMatcher("is_auto_generated", isSetMatcherInstance).
		Because("is_auto_generated property is only allowed for filesystem modules in build/soong/fsgen directory")
		}

		func neverallowMutator(ctx BottomUpMutatorContext) {
		m, ok := ctx.Module().(Module)
		if !ok {

+0 −16

Original line number	Diff line number	Diff line
		@@ -359,21 +359,6 @@ var neverallowTests = []struct {
		`headers_only can only be used for generating framework-minus-apex headers for non-updatable modules`,
		},
		},
		// Test for the rule restricting use of is_auto_generated
		{
		name: `"is_auto_generated" outside allowed directory`,
		fs: map[string][]byte{
		"a/b/Android.bp": []byte(`
		filesystem {
		name: "baaz",
		is_auto_generated: true,
		}
		`),
		},
		expectedErrors: []string{
		`is_auto_generated property is only allowed for filesystem modules in build/soong/fsgen directory`,
		},
		},
		}

		var prepareForNeverAllowTest = GroupFixturePreparers(
		@@ -382,7 +367,6 @@ var prepareForNeverAllowTest = GroupFixturePreparers(
		ctx.RegisterModuleType("java_library", newMockJavaLibraryModule)
		ctx.RegisterModuleType("java_library_host", newMockJavaLibraryModule)
		ctx.RegisterModuleType("java_device_for_host", newMockJavaLibraryModule)
		ctx.RegisterModuleType("filesystem", newMockFilesystemModule)
		}),
		)

+2 −3

Original line number	Diff line number	Diff line
		@@ -2100,7 +2100,6 @@ func (p *mockLibraryModule) GenerateAndroidBuildActions(ModuleContext) {
		type mockFilesystemModuleProperties struct {
		Partition_type *string
		Deps []string
		Is_auto_generated *bool
		}

		type mockFilesystemModule struct {