[arch-commits] Commit in electron4/trunk (PKGBUILD glslang-remove-setAllocator.patch)

[Evangelos Foutras]

    Date: Wednesday, April 29, 2020 @ 09:12:02
  Author: foutrelis
Revision: 622223

Fix build with clang 10; add workaround for large zip file

Added:
  electron4/trunk/glslang-remove-setAllocator.patch
Modified:
  electron4/trunk/PKGBUILD

-----------------------------------+
 PKGBUILD                          |    8 ++++++++
 glslang-remove-setAllocator.patch |   24 ++++++++++++++++++++++++
 2 files changed, 32 insertions(+)

Modified: PKGBUILD
===================================================================
--- PKGBUILD	2020-04-29 08:43:32 UTC (rev 622222)
+++ PKGBUILD	2020-04-29 09:12:02 UTC (rev 622223)
@@ -24,6 +24,7 @@
         'chromium-SIOCGSTAMP.patch'
         'chromium-skia-harmony.patch'
         'icu65.patch'
+        'glslang-remove-setAllocator.patch'
         'chromium-system-icu.patch'
         'fix-cfi-icall-failure-with-use_system_libjpeg-true.patch'
         'only-disable-cfi-icall-when-use_system_libjpeg-true.patch'
@@ -36,6 +37,7 @@
             '7acc4dd59b70fb64f602ceda2846ccddcb46f64a18f912658d1034965f6c1276'
             'feca54ab09ac0fc9d0626770a6b899a6ac5a12173c7d0c1005bc3964ec83e7b3'
             '1de9bdbfed482295dda45c7d4e323cee55a34e42f66b892da1c1a778682b7a41'
+            '06738b3f2d3579cd9b4d1ff876ba93d6b10a741b4deb4eab7fe3008cc577c893'
             'c4f2d1bed9034c02b8806f00c2e8165df24de467803855904bff709ceaf11af5'
             '97b421bc60a4abdf37de2d88a51b973e9f68fb44d1eccd464adfb3d9f5d71478'
             '9cae9ded6497afd15ad72d963897425ab6c7f28941bb3c3948e7996610a0d180')
@@ -135,6 +137,8 @@
   patch -Np1 -i ../chromium-SIOCGSTAMP.patch
   patch -Np4 -i ../chromium-skia-harmony.patch
   patch -Np1 -i ../icu65.patch
+  patch -Np1 -d third_party/glslang/src <../glslang-remove-setAllocator.patch
+  patch -Np1 -d third_party/angle/third_party/glslang/src <../glslang-remove-setAllocator.patch
   patch -Np1 -i ../chromium-system-icu.patch
   patch -Np1 -i ../fix-cfi-icall-failure-with-use_system_libjpeg-true.patch
   patch -Np1 -i ../only-disable-cfi-icall-when-use_system_libjpeg-true.patch
@@ -166,6 +170,10 @@
   '
   gn-m76 gen out/Release \
       --args="import(\"//electron/build/args/release.gn\") ${GN_EXTRA_ARGS}"
+  ninja -C out/Release electron
+  # Strip before zip to avoid
+  # zipfile.LargeZipFile: Filesize would require ZIP64 extensions
+  strip -s out/Release/electron
   ninja -C out/Release electron_dist_zip
   # ninja -C out/Release third_party/electron_node:headers
 }

Added: glslang-remove-setAllocator.patch
===================================================================
--- glslang-remove-setAllocator.patch	                        (rev 0)
+++ glslang-remove-setAllocator.patch	2020-04-29 09:12:02 UTC (rev 622223)
@@ -0,0 +1,24 @@
+From 24b3e8384e93f3e73b6aa14ea00a30574112f9ba Mon Sep 17 00:00:00 2001
+From: Reid Kleckner <rnk at google.com>
+Date: Wed, 4 Dec 2019 14:09:03 -0800
+Subject: [PATCH] Remove glslang::pool_allocator::setAllocator
+
+TPoolAllocator is not copy assignable, so this setter could never have
+been used. After a recent change (878a24ee2), new versions of Clang
+reject this code outright.
+---
+ glslang/Include/PoolAlloc.h | 1 -
+ 1 file changed, 1 deletion(-)
+
+diff --git a/glslang/Include/PoolAlloc.h b/glslang/Include/PoolAlloc.h
+index 0e237a6a2..b8eccb883 100644
+--- a/glslang/Include/PoolAlloc.h
++++ b/glslang/Include/PoolAlloc.h
+@@ -304,7 +304,6 @@ class pool_allocator {
+     size_type max_size() const { return static_cast<size_type>(-1) / sizeof(T); }
+     size_type max_size(int size) const { return static_cast<size_type>(-1) / size; }
+ 
+-    void setAllocator(TPoolAllocator* a) { allocator = *a; }
+     TPoolAllocator& getAllocator() const { return allocator; }
+ 
+ protected: