{"push_id":2910405516,"size":5,"distinct_size":1,"ref":"refs/heads/test","head":"aa71e741a88d63635f5868f62e210ff8407f670b","before":"be15ada3dfcabd6e461aa17eccee29541520443c","commits":[{"sha":"9161e72964e934236b331ed409fb5ceb0be75e8a","author":{"name":"Sam Clegg","email":"4504a65e6a180f18042ae97f363f5ae49cc5c461@chromium.org"},"message":"Fix more flake8 fixes (#7180)\n\nThis changes from the last of the \"import *\" statements from\r\nemscripten so the dependency graph is now clear.","distinct":false,"url":"https://api.github.com/repos/aheejin/emscripten/commits/9161e72964e934236b331ed409fb5ceb0be75e8a"},{"sha":"ab8337c30ae1c54088e8debca170f31bd663ea19","author":{"name":"Sam Clegg","email":"4504a65e6a180f18042ae97f363f5ae49cc5c461@chromium.org"},"message":"Remove shared.check_execute (#7183)\n\nThis function was basically the same as check_call except that it\r\ncaptures it stdout.  It was only used in test code and in both\r\nplaces a more suitable method was available.","distinct":false,"url":"https://api.github.com/repos/aheejin/emscripten/commits/ab8337c30ae1c54088e8debca170f31bd663ea19"},{"sha":"fd06e7e47c8e2d26e2194bcf6d45991c06c7ff4e","author":{"name":"Sam Clegg","email":"4504a65e6a180f18042ae97f363f5ae49cc5c461@chromium.org"},"message":"Remove explicit dependency on pthread_mutux functions from library_fetch.js (#7195)\n\nThese depedencies are currently causing warning in all builds with\r\nFETCH=1+USE_PTHTREADS=1 as far as I can tell.  This is because when\r\npthreads are enabled these function are implemented in native code\r\nin musl and are named with a leading `__`.\r\n\r\nIn fact, I far can tell the fetch library doesn't actually depend\r\non these functions so simply removing the dependency fixes the\r\nissue.","distinct":false,"url":"https://api.github.com/repos/aheejin/emscripten/commits/fd06e7e47c8e2d26e2194bcf6d45991c06c7ff4e"},{"sha":"1f773bf3534e3c36cff041b4144c12cd459b79c0","author":{"name":"Sam Clegg","email":"4504a65e6a180f18042ae97f363f5ae49cc5c461@chromium.org"},"message":"Create separate code path for testing JS vs running JS tools (#7184)\n\n* Create separate process for testing and JS and running JS tools\r\n\r\nThe `jsrun.run_js()` method has a lot of extra complexity that we\r\ndon't need when running, for example, the js compiler.\r\n\r\nThis change also makes the output much nicer when the JS compiler\r\nfails.\r\n\r\nBefore:\r\n\r\n```\r\n$ ./emcc.py -s IN_TEST_HARNESS=1 /tmp/emscripten_test/hello_file.cpp -o test.html -s ASMFS=1 -s WASM=0 -s USE_PTHREADS=1 -s FETCH_DEBUG=1 -s PROXY_TO_PTHREAD=1 -s WASM=0   -s ERROR_ON_UNDEFINED_SYMBOLS=1\r\nerror: unresolved symbol: pthread_mutex_lock\r\nerror: unresolved symbol: pthread_mutex_unlock\r\nAborting compilation due to previous errors | undefined\r\nTraceback (most recent call last):\r\n  File \"./emcc.py\", line 3074, in <module>\r\n    sys.exit(run())\r\n  File \"./emcc.py\", line 1790, in run\r\n    final = shared.Building.emscripten(final, append_ext=False, extra_args=extra_args)\r\n  File \"/usr/local/google/home/sbc/dev/wasm/emscripten/tools/shared.py\", line 2259, in emscripten\r\n    emscripten._main(cmdline)\r\n  File \"/usr/local/google/home/sbc/dev/wasm/emscripten/emscripten.py\", line 2239, in _main\r\n    return temp_files.run_and_clean(lambda: main(\r\n  File \"/usr/local/google/home/sbc/dev/wasm/emscripten/tools/tempfiles.py\", line 98, in run_and_clean\r\n    return func()\r\n  File \"/usr/local/google/home/sbc/dev/wasm/emscripten/emscripten.py\", line 2244, in <lambda>\r\n    DEBUG=DEBUG,\r\n  File \"/usr/local/google/home/sbc/dev/wasm/emscripten/emscripten.py\", line 2170, in main\r\n    temp_files=temp_files, DEBUG=DEBUG)\r\n  File \"/usr/local/google/home/sbc/dev/wasm/emscripten/emscripten.py\", line 91, in emscript\r\n    glue, forwarded_data = compiler_glue(metadata, libraries, compiler_engine, temp_files, DEBUG)\r\n  File \"/usr/local/google/home/sbc/dev/wasm/emscripten/emscripten.py\", line 223, in compiler_glue\r\n    glue, forwarded_data = compile_settings(compiler_engine, libraries, temp_files)\r\n  File \"/usr/local/google/home/sbc/dev/wasm/emscripten/emscripten.py\", line 535, in compile_settings\r\n    cwd=path_from_root('src'), error_limit=300)\r\n  File \"/usr/local/google/home/sbc/dev/wasm/emscripten/tools/jsrun.py\", line 137, in run_js\r\n    raise Exception('Expected the command ' + str(command) + ' to finish with return code ' + str(assert_returncode) + ', but it returned with code ' + str(proc.returncode) + ' instead! Output: ' + str(ret)[:error_limit])\r\nException: Expected the command ['/usr/local/google/home/sbc/bin/node-v8.9.3-linux-x64/bin/node', '/usr/local/google/home/sbc/dev/wasm/emscripten/src/compiler.js', '/tmp/tmpdx7Lpl.txt', '/usr/local/google/home/sbc/dev/wasm/emscripten/src/library_fetch.js', '/usr/local/google/home/sbc/dev/wasm/emscripten/src/library_pthread.js'] to finish with return code 0, but it returned with code 1 instead! Output: // Copyright 2010 The Emscripten Authors.  All rights reserved.\r\n// Emscripten is available under two separate licenses, the MIT license and the\r\n// University of Illinois/NCSA Open Source License.  Both these licenses can be\r\n// found in the LICENSE file.\r\n\r\n// The Module object: Our interface to the ou\r\n```\r\n\r\nAfter:\r\n\r\n```\r\n$ ./emcc.py -s IN_TEST_HARNESS=1 /tmp/emscripten_test/hello_file.cpp -o test.html -s ASMFS=1 -s WASM=0 -s USE_PTHREADS=1 -s FETCH_DEBUG=1 -s PROXY_TO_PTHREAD=1 -s WASM=0   -s ERROR_ON_UNDEFINED_SYMBOLS=1\r\nerror: unresolved symbol: pthread_mutex_lock\r\nerror: unresolved symbol: pthread_mutex_unlock\r\nAborting compilation due to previous errors | undefined\r\nERROR:root:'/usr/local/google/home/sbc/bin/node-v8.9.3-linux-x64/bin/node /usr/local/google/home/sbc/dev/wasm/emscripten/src/compiler.js /tmp/tmpwWpaGd.txt /usr/local/google/home/sbc/dev/wasm/emscripten/src/library_fetch.js /usr/local/google/home/sbc/dev/wasm/emscripten/src/library_pthread.js' failed (1)\r\n```\r\n\r\n* feedback + fixes","distinct":false,"url":"https://api.github.com/repos/aheejin/emscripten/commits/1f773bf3534e3c36cff041b4144c12cd459b79c0"},{"sha":"aa71e741a88d63635f5868f62e210ff8407f670b","author":{"name":"Heejin Ahn","email":"3fdccf5e498a980112d395241b036ed81adc7f86@gmail.com"},"message":"Merge branch 'incoming' into test","distinct":true,"url":"https://api.github.com/repos/aheejin/emscripten/commits/aa71e741a88d63635f5868f62e210ff8407f670b"}]}
{"actor":{"display_login":"aheejin"}}