[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[tor-commits] [tor-browser-bundle/master] Bugs 22832: Only include objects under out/Release/obj in libwebrtc-magic.a.



commit 781da76f120f849416c772cef45a9a9a103b4eef
Author: David Fifield <david@xxxxxxxxxxxxxxx>
Date:   Wed Jul 5 23:11:03 2017 -0700

    Bugs 22832: Only include objects under out/Release/obj in libwebrtc-magic.a.
    
    Object files from elsewhere under out/ should never have been included,
    because they are not part of webrtc itself, but rather side-effect build
    artifacts like gn. Including those extraneous .o files was mostly
    harmless (except for library size), because on linux they happened to be
    the same architecture as the webrtc.o files. However it won't work for
    the mac build (because libwebrtc-magic.a would include a mix of linux
    ELF and mac Mach-O objects). Additionally, build_time.o, part of the gn
    build, embeds a timestamp with month resolution, causing a failure of
    reproducibility, as found at https://bugs.torproject.org/22832.
---
 gitian/descriptors/linux/gitian-webrtc.yml | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/gitian/descriptors/linux/gitian-webrtc.yml b/gitian/descriptors/linux/gitian-webrtc.yml
index 9e7808c..8bb6129 100644
--- a/gitian/descriptors/linux/gitian-webrtc.yml
+++ b/gitian/descriptors/linux/gitian-webrtc.yml
@@ -103,9 +103,12 @@ script: |
   rm -rf out/Release
   "$GN" gen out/Release --args="$GN_ARGS"
   ninja -C out/Release webrtc field_trial metrics_default pc_test_utils
-  # https://github.com/keroserene/go-webrtc/issues/23#issuecomment-175312648
-  # libwebrtc-linux-386-magic.a(dump_syms_regtest.o): unsupported ELF file type 2
-  ar crs libwebrtc-magic.a $(find . -name '*.o' -not -name '*.main.o' -not -name 'dump_syms_regtest.o' | sort)
+  # .o files under out/Release/obj/ are the build outputs. Don't include .o
+  # files from elsewhere under out/ because they are build helpers and things
+  # like that, not necessarily of the target architecture, and anyway are not
+  # needed.
+  # https://bugs.torproject.org/22832
+  ar crs libwebrtc-magic.a $(find out/Release/obj -name '*.o' | sort)
   cd ../..
 
   # Grabbing the result

_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits