[Android] Only always trigger stripping with component build

build/android/strip_native_libraries.gypi

Index: build/android/strip_native_libraries.gypi
diff --git a/build/android/strip_native_libraries.gypi b/build/android/strip_native_libraries.gypi
index d35a5b887ec4bcc86657a188e5d5600a72fea768..a5619ac6f9b3be8f036bb4ab11588b3a4ffecd3e 100644
--- a/build/android/strip_native_libraries.gypi
+++ b/build/android/strip_native_libraries.gypi
@@ -7,12 +7,14 @@
 #
 # To use this, create a gyp target with the following form:
 #  {
+#    'action_name': 'strip_native_libraries',
 #    'actions': [
 #      'variables': {
 #        'ordered_libraries_file': 'file generated by write_ordered_libraries'
 #        'input_paths': 'files to be added to the list of inputs'
-#        'strip_stamp': 'file to touch when the action is complete'
+#        'stamp': 'file to touch when the action is complete'
 #        'stripped_libraries_dir': 'directory to store stripped libraries',
+#        'always_run': 0, # set to 1 to force this action to always run
 #      },
 #      'includes': [ '../../build/android/strip_native_libraries.gypi' ],
 #    ],
@@ -20,7 +22,6 @@
 #
 
 {
-  'action_name': 'strip_native_libraries',

[Yaron, 2013/05/15 00:32:41]

Why did you pull this out?

[cjhopman, 2013/05/15 21:13:33]

An action_name must be unique within a target. I am sort of planning to extract
all these actions into their own .gypi files and would like to use a consistent
pattern for them, so they should all support some way of uniquifying the
action_name. The simplest way seems to be just have the includer specify it.

   'message': 'Stripping libraries for <(_target_name)',
   'variables': {
     'input_paths': [],
@@ -32,11 +33,15 @@
     '>@(input_paths)',
   ],
   'outputs': [
-    '<(strip_stamp)',
-    # Add a fake output to force the build to always re-run this step. This
-    # is required because the real inputs are not known at gyp-time and
-    # changing base.so may not trigger changes to dependent libraries.
-    '<(strip_stamp).fake',
+    '<(stamp)',
+  ],
+  'conditions': [
+    ['component == "shared_library"', {
+      # Add a fake output to force the build to always re-run this step. This
+      # is required because the real inputs are not known at gyp-time and
+      # changing base.so may not trigger changes to dependent libraries.
+      'outputs': [ '<(stamp).fake' ]
+    }],
   ],
   'action': [
     'python', '<(DEPTH)/build/android/gyp/strip_library_for_device.py',
@@ -45,6 +50,6 @@
     '--stripped-libraries-dir=<(stripped_libraries_dir)',
     '--libraries-dir=<(SHARED_LIB_DIR)',
     '--libraries-file=<(ordered_libraries_file)',
-    '--stamp=<(strip_stamp)',
+    '--stamp=<(stamp)',
   ],
 }