[modular] Deprecate module manifest build rules (for verification)
This is in favour of the module facet -- the code for that
exists in //build/cmx/facets/module_facet_schema.json and through
//build/cmx/cmx.gni
Test=builds fine.
Change-Id: Ia02e58a990b2bf03d889353359fb83c6ce9f9bcf
diff --git a/build/module_manifest.gni b/build/module_manifest.gni
index 4f4b21b..56d63ee 100644
--- a/build/module_manifest.gni
+++ b/build/module_manifest.gni
@@ -2,49 +2,8 @@
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
-import("//build/compiled_action.gni")
import("//build/package.gni")
-module_manifest_schema = "//peridot/build/module_manifest_schema.json"
-
-# This template generates a target that will verify the supplied module
-# manifest.
-#
-# Parameters
-#
-# original_target_name (required)
-# This is the original parent-level target name for the package that calls
-# this into template. This name is used for printing out a stamp file.
-#
-# module_manifest (required)
-# Path to the module manifest that needs to be verified.
-template("verify_module_manifest") {
- assert(defined(invoker.original_target_name),
- "Must specify original_target_name.")
- assert(defined(invoker.module_manifest),
- "Must specify path to module manifest.")
-
- compiled_action(target_name) {
- tool = "//build/tools/json_validator"
- stamp_file = "$target_gen_dir/${invoker.original_target_name}.verified"
-
- inputs = [
- invoker.module_manifest,
- module_manifest_schema,
- ]
-
- outputs = [
- stamp_file,
- ]
-
- args = [
- rebase_path(module_manifest_schema),
- rebase_path(invoker.module_manifest),
- rebase_path(stamp_file),
- ]
- }
-}
-
# This is a rule used to generate an index of module packages that should be
# indexed by the on-device module resolver when it starts up. This template is
# meant to be used only once for each layer, but multiple may exist for
@@ -87,18 +46,4 @@
},
]
}
-}
-
-template("copy_module_manifest") {
- assert(defined(invoker.module_manifest), "Must specify module_manifest.")
- assert(defined(invoker.package_name), "Must specify package_name.")
-
- copy(target_name) {
- sources = [
- rebase_path(invoker.module_manifest),
- ]
- outputs = [
- "$root_build_dir/module_manifests/${invoker.package_name}",
- ]
- }
-}
+}
\ No newline at end of file
diff --git a/build/module_manifest_schema.json b/build/module_manifest_schema.json
deleted file mode 100644
index 04f59af..0000000
--- a/build/module_manifest_schema.json
+++ /dev/null
@@ -1,97 +0,0 @@
-{
- "$schema": "http://json-schema.org/schema#",
- "title": "Schema for `module_manifest.json` metadata file",
- "definitions": {
- "intentFilterArray": {
- "type": "array",
- "items": {
- "$ref": "#/definitions/intentFilter"
- },
- "additionalItems": false,
- "uniqueItems": true,
- "minItems": 1
- },
- "intentFilter": {
- "type": "object",
- "properties": {
- "action": {
- "type": "string"
- },
- "parameters": {
- "$ref": "#/definitions/parameterArray"
- }
- },
- "required": ["action", "parameters"]
- },
- "parameterArray": {
- "type": "array",
- "items": {
- "$ref": "#/definitions/parameter"
- },
- "additionalItems": false,
- "uniqueItems": true
- },
- "parameter": {
- "type": "object",
- "properties": {
- "name": {
- "$ref": "#/definitions/alphaNumString"
- },
- "type": {
- "type": "string"
- },
- "required": {
- "type": "boolean"
- }
- },
- "required": ["name", "type"],
- "additionalProperties": false
- },
- "alphaNumString": {
- "type": "string",
- "pattern": "^[a-zA-Z0-9_]+$"
- },
- "compositionPattern": {
- "type": "string",
- "enum": ["ticker", "comments-right"]
- },
- "hexColor": {
- "type": "string",
- "pattern": "^#([A-Fa-f0-9]{6})$"
- }
- },
- "type": "object",
- "properties": {
- "binary": {
- "type": "string"
- },
- "suggestion_headline": {
- "type": "string"
- },
- "intent_filters": {
- "$ref": "#/definitions/intentFilterArray"
- },
- "composition_pattern": {
- "$ref": "#/definitions/compositionPattern"
- },
- "action": {
- "type": "string"
- },
- "parameters": {
- "$ref": "#/definitions/parameterArray"
- },
- "@version": {
- "type": "integer"
- },
- "placeholder_color": {
- "$ref": "#definitions/hexColor"
- }
- },
- "required": ["binary"],
- "dependencies": {
- "intent_filters": {
- "required": ["@version"]
- }
- },
- "additionalProperties": false
-}