From c53ff47046412b53d85d2761baf6272071692bc1 Mon Sep 17 00:00:00 2001 From: Hood Chatham Date: Thu, 23 May 2024 17:42:31 -0400 Subject: [PATCH] BLD Clean up unused Makefile sed (#4785) This stuff is no longer needed since the update to Emscripten 3.1.58. --- Makefile | 6 ------ 1 file changed, 6 deletions(-) diff --git a/Makefile b/Makefile index 71593cd19..6189b17ab 100644 --- a/Makefile +++ b/Makefile @@ -134,12 +134,6 @@ dist/pyodide.asm.js: \ cd dist && npx prettier -w pyodide.asm.js ; \ fi - # Strip out C++ symbols which all start __Z. - # There are 4821 of these and they have VERY VERY long names. - # To show some stats on the symbols you can use the following: - # cat dist/pyodide.asm.js | grep -ohE 'var _{0,5}.' | sort | uniq -c | sort -nr | head -n 20 - sed -i -E 's/var __Z[^;]*;//g' dist/pyodide.asm.js - sed -i '1i "use strict";' dist/pyodide.asm.js # Remove last 4 lines of pyodide.asm.js, see issue #2282 # Hopefully we will remove this after emscripten fixes it, upstream issue # emscripten-core/emscripten#16518