build_nexe.py: Allow --strip-debug for archives

build/build_nexe.py

Index: build/build_nexe.py
diff --git a/build/build_nexe.py b/build/build_nexe.py
index 7c068f78efa90ab199a93d9ed6ceb36313f9d107..e2dcb35bd4b7588b185324d5dae6bd5af84a7ff7 100644
--- a/build/build_nexe.py
+++ b/build/build_nexe.py
@@ -495,7 +495,10 @@ class Builder(object):
       if self.strip_all or self.strip_debug:
         self.Strip(out)
     elif self.outtype in ['nlib', 'plib']:
-      self.Archive(srcs)
+      out = self.Archive(srcs)
+      # Also valid to strip archives, but strip_all seems dangerous.

[Roland McGrath, 2012/12/13 20:51:39]

Indeed, --strip-all makes .a or .o files unusable.
--strip-debug is fine for them.

+      if self.strip_debug:

[Sam Clegg, 2012/12/13 20:55:58]

This seems a little confusing to me.

If I specify strip-all on a archive I would expect it to at least strip debug
info, rather than do nothing.  But stripping everything and having it fail
wouldn't be so bad either since it would force me to do the right thing.

[jvoung - send to chromium..., 2012/12/13 21:23:23]

Yeah, it shouldn't do less that strip-debug...
Ok I'll pass strip-all straight through so that people can do the right thing in
the gyp files.

+        self.Strip(out)
     else:
       ErrOut('FAILED: Unknown outtype %s:\n' % (self.outtype))