From da43504707710e6fd82cf837079f0e71e98a33b5 Mon Sep 17 00:00:00 2001 From: "evan@chromium.org" Date: Fri, 19 Aug 2011 18:45:00 +0000 Subject: Update libphonenumber.gyp to use the new shared protoc gyp rules Just a refactoring. Review URL: http://codereview.chromium.org/7671054 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@97486 0039d316-1c4b-4281-b951-d872f2087c98 --- third_party/libphonenumber/libphonenumber.gyp | 62 +++------------------------ 1 file changed, 7 insertions(+), 55 deletions(-) (limited to 'third_party') diff --git a/third_party/libphonenumber/libphonenumber.gyp b/third_party/libphonenumber/libphonenumber.gyp index d6101af..487c01e 100644 --- a/third_party/libphonenumber/libphonenumber.gyp +++ b/third_party/libphonenumber/libphonenumber.gyp @@ -3,16 +3,12 @@ # found in the LICENSE file. { - 'variables': { - 'protoc_out_dir': '<(SHARED_INTERMEDIATE_DIR)/protoc_out', - }, 'target_defaults': { 'include_dirs': [ '../..', # add it first, so src/base headers are used instead of the ones # brought with the library as cc files would be taken from the # main chrome tree as well. 'cpp/src', - '<(protoc_out_dir)', ], 'defines': [ 'USE_HASH_MAP=1', @@ -25,10 +21,8 @@ 'dependencies': [ '../icu/icu.gyp:icui18n', '../icu/icu.gyp:icuuc', - '../protobuf/protobuf.gyp:protobuf_lite', '../../base/base.gyp:base', '../../base/third_party/dynamic_annotations/dynamic_annotations.gyp:dynamic_annotations', - 'phonenumber_proto', ], 'sources': [ # 'chrome/regexp_adapter_icuregexp.cc', @@ -43,10 +37,14 @@ 'cpp/src/utf/rune.c', 'cpp/src/utf/unicodetext.cc', 'cpp/src/utf/unilib.cc', - # Generated by phonenumber_proto. - '<(protoc_out_dir)/phonemetadata.pb.cc', - '<(protoc_out_dir)/phonenumber.pb.cc', + 'resources/phonemetadata.proto', + 'resources/phonenumber.proto', ], + 'variables': { + 'proto_in_dir': 'resources', + 'proto_out_dir': 'third_party/libphonenumber', + }, + 'includes': [ '../../build/protoc.gypi' ], 'conditions': [ ['OS=="win"', { 'action': [ @@ -56,51 +54,6 @@ ], }, { - # Protobuf compiler / generate rule for the phones - 'target_name': 'phonenumber_proto', - 'type': 'none', - 'sources': [ - 'resources/phonemetadata.proto', - 'resources/phonenumber.proto', - ], - 'rules': [{ - 'rule_name': 'genproto', - 'extension': 'proto', - 'inputs': [ - '<(PRODUCT_DIR)/<(EXECUTABLE_PREFIX)protoc<(EXECUTABLE_SUFFIX)', - ], - 'variables': { - # The protoc compiler requires a proto_path argument with the - # directory containing the .proto file. - # There's no generator variable that corresponds to this, so fake it. - 'rule_input_relpath': 'resources', - }, - 'outputs': [ - '<(protoc_out_dir)/<(RULE_INPUT_ROOT).pb.h', - '<(protoc_out_dir)/<(RULE_INPUT_ROOT).pb.cc', - ], - 'action': [ - '<(PRODUCT_DIR)/<(EXECUTABLE_PREFIX)protoc<(EXECUTABLE_SUFFIX)', - '--proto_path=<(rule_input_relpath)', - '<(rule_input_relpath)/<(RULE_INPUT_ROOT)<(RULE_INPUT_EXT)', - '--cpp_out=<(protoc_out_dir)', - ], - 'message': 'Generating C++ code from <(RULE_INPUT_PATH)', - }], - 'dependencies': [ - '../protobuf/protobuf.gyp:protobuf_lite', - '../protobuf/protobuf.gyp:protoc#host', - ], - 'direct_dependent_settings': { - 'include_dirs': [ - '<(protoc_out_dir)', - ] - }, - 'export_dependent_settings': [ - '../../third_party/protobuf/protobuf.gyp:protobuf_lite', - ], - }, - { 'target_name': 'libphonenumber_unittests', 'type': 'executable', 'sources': [ @@ -113,7 +66,6 @@ 'dependencies': [ '../icu/icu.gyp:icui18n', '../icu/icu.gyp:icuuc', - '../protobuf/protobuf.gyp:protobuf_lite', '../../base/base.gyp:base', '../../base/base.gyp:test_support_base', '../../base/third_party/dynamic_annotations/dynamic_annotations.gyp:dynamic_annotations', -- cgit v1.1