commit 38cad0c6a9bf2903e7236d0100911f49c3fbac95
author Alexei Frolov <frolv@google.com> Thu Dec 03 12:36:24 2020 -0800
committer CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu Dec 03 21:12:21 2020 +0000
tree f5a902625d9603a8a81eb8278a79fd68de5f3c5a
parent e65ddc8b766dffdad83b74d440e836c1f7bcb2ed

pw_protobuf_compiler: Python proto follow-up fixes

- Ignore generated protobuf files in pylint
- Remove no longer needed disable build arg

Change-Id: I711e9c3be02addfc7967cf244132e494b6f0df6f
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/26701
Reviewed-by: Wyatt Hepler <hepler@google.com>
Commit-Queue: Alexei Frolov <frolv@google.com>

pw_protobuf_compiler/proto.gni

diff --git a/pw_protobuf_compiler/proto.gni b/pw_protobuf_compiler/proto.gni
index 8d0a55e..527e25b 100644
--- a/pw_protobuf_compiler/proto.gni
+++ b/pw_protobuf_compiler/proto.gni
@@ -277,12 +277,6 @@
   }
 }
 
-declare_args() {
-  # Temporary build arg to allow projects to update their proto libraries to be
-  # Python-compatible.
-  pw_protobuf_compiler_DISABLE_PYTHON = false
-}
-
 # Generates protobuf code from .proto definitions for various languages.
 # For each supported generator, creates a sub-target named:
 #
@@ -386,13 +380,11 @@
     base_target = _common.base_target
   }
 
-  if (!pw_protobuf_compiler_DISABLE_PYTHON) {
-    _pw_python_proto_library("$target_name.python") {
-      sources = invoker.sources
-      forward_variables_from(_common, "*")
-      deps = process_file_template(_deps, "{{source}}.python")
-      base_target = _common.base_target
-    }
+  _pw_python_proto_library("$target_name.python") {
+    sources = invoker.sources
+    forward_variables_from(_common, "*")
+    deps = process_file_template(_deps, "{{source}}.python")
+    base_target = _common.base_target
   }
 
   # All supported pw_protobuf generators.