From fbbf5175ca6bf60840d7abebab997818c3431173 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Vladim=C3=ADr=20Vondru=C5=A1?= Date: Sun, 23 Aug 2026 12:20:38 +0200 Subject: [PATCH] Platform: make Emscripten test runner work on Emscripten 6.0.1+. I'm specifically not saying "fix" because this isn't a fix, it's an awful workaround for another needless breakage in this damn thing. --- src/Magnum/Platform/WindowlessEmscriptenApplication.js | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/src/Magnum/Platform/WindowlessEmscriptenApplication.js b/src/Magnum/Platform/WindowlessEmscriptenApplication.js index 5aaaab2d7..70462fecf 100644 --- a/src/Magnum/Platform/WindowlessEmscriptenApplication.js +++ b/src/Magnum/Platform/WindowlessEmscriptenApplication.js @@ -61,6 +61,16 @@ function createMagnumModule(init) { log: document.getElementById('log'), setStatus: function(message) { + /* As of Emscripten 6.0.1 and https://github.com/emscripten-core/emscripten/pull/27121 + monitorRunDependencies() isn't called anymore for some reason + (and thus the total dependency count stays zero), and the only + way to know loading finished in order to unhide the log 
 is
+               when setStatus() is called with an empty message. No note about
+               any of this in the changelog so I suppose it's a regression, but
+               nobody cares, and I'm tired so I won't care either. */
+            if(!module.totalDependencies && !message && module.log)
+                module.log.style.display = 'block';
+
             if(module.status)
                 module.status.innerHTML = message;
         },