[NaCl SDK] Don't copy .h files to src in SDK.

native_client_sdk/src/build_tools/build_sdk.py

 #!/usr/bin/env python
 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Entry point for both build and try bots
 
 This script is invoked from XXX, usually without arguments
 to package an SDK. It automatically determines whether
 this SDK is for mac, win, linux.
@@ skipping 675 matching lines @@
     buildbot_common.Archive('sdk_tools.tgz', bucket_path, OUT_DIR,
                             step_link=False)
 
 
 def main(args):
   parser = optparse.OptionParser()
   parser.add_option('--pnacl', help='Enable pnacl build.',
       action='store_true', dest='pnacl', default=False)
   parser.add_option('--examples', help='Only build the examples.',
       action='store_true', dest='only_examples', default=False)
+  parser.add_option('--libraries', help='Only build the libraries.',

[noelallen1, 2012/07/25 20:36:03]

note:  I think this is fine but considering how little time it takes, do you
want to separate it from examples?  I didn't realize until I saw this CL that I
would first need to run --libraries then run --examples to correctly test an
example that depended on a library change.

[binji, 2012/07/25 21:13:08]

Done.

+      action='store_true', dest='only_libraries', default=False)
   parser.add_option('--update', help='Only build the updater.',
       action='store_true', dest='only_updater', default=False)
   parser.add_option('--test-examples',
       help='Run the pyauto tests for examples.', action='store_true',
       dest='test_examples', default=False)
   parser.add_option('--skip-tar', help='Skip generating a tarball.',
       action='store_true', dest='skip_tar', default=False)
   parser.add_option('--archive', help='Force the archive step.',
       action='store_true', dest='archive', default=False)
   parser.add_option('--release', help='PPAPI release version.',
@@ skipping 34 matching lines @@
   clnumber = build_utils.ChromeRevision()
   tarname = 'naclsdk_' + platform + '.tar.bz2'
   if 'pnacl' in toolchains:
     tarname = 'p' + tarname
   tarfile = os.path.join(SERVER_DIR, tarname)
 
   if options.release:
     pepper_ver = options.release
   print 'Building PEPPER %s at %s' % (pepper_ver, clnumber)
 
   if options.only_examples:

[noelallen1, 2012/07/25 20:36:03]

Should this be a superset of libraries so that we always copy and build the
libraries this depends on?  A change to make_rules.py or couple require both of
these.

[binji, 2012/07/25 21:13:08]

Done.

     BuildStepCopyExamples(pepperdir, toolchains)
     BuildStepBuildExamples(pepperdir, platform)
     if options.test_examples:
       BuildStepTestExamples(pepperdir, platform, pepper_ver)
+  elif options.only_libraries:
+    BuildStepCopyExamples(pepperdir, toolchains)
+    BuildStepBuildLibraries(pepperdir, platform)
   elif options.only_updater:
     build_updater.BuildUpdater(OUT_DIR)
   else:  # Build everything.
     BuildStepBuildToolsTests()
 
     BuildStepDownloadToolchains(platform)
     BuildStepCleanPepperDirs(pepperdir, pepperdir_old)
     BuildStepMakePepperDirs(pepperdir, ['include', 'toolchain', 'tools'])
     BuildStepCopyTextFiles(pepperdir, pepper_ver, clnumber)
     BuildStepUntarToolchains(pepperdir, platform, arch, toolchains)
@@ skipping 19 matching lines @@
       # Archive on non-trybots.
       if options.archive or buildbot_common.IsSDKBuilder():
         BuildStepArchiveBundle(pepper_ver, clnumber, tarfile)
         BuildStepArchiveSDKTools()
 
   return 0
 
 
 if __name__ == '__main__':
   sys.exit(main(sys.argv))