{"push_id":2831951690,"size":1,"distinct_size":1,"ref":"refs/heads/master","head":"5c077985f386c44cdf408853090dcc61c544eea0","before":"3b38b07fc66936279b29230042bb2b0ab9ec6826","commits":[{"sha":"5c077985f386c44cdf408853090dcc61c544eea0","author":{"name":"Stella Stamenova","email":"a7da1dee431f087223f80e75eb40c7bfb8cc59ec@microsoft.com"},"message":"[lit, shtest-timeout] Always use an internal shell for the shtest-timeout to diagnose buildbot failures\n\nSummary:\nRight now this test is failing on the builtbots on Windows but we have a very similar setup where the test passes. The test is meant to test that specifying a timeout works correctly by running an infnite loop and having it timeout - on the buildbot, the infinite loop doesn't actually execute. This change runs all of the tests in the set using an internal shell rather than an external shell. I expect this will make the test pass which means that either the way the external shell is invoked or the external shell setup on the buildbots is not correct. Regardless of whether the test passes with this change, we'll need to undo this change and have a real fix.\n\n@gkistanova was able to get logs from the buildbot to rule out a number of theories as to why this test is failing, but they didn't have enough information to confirm exactly what the issue is. The purpose of this change is to narrow it down, but if someone has a local repro and can aid in debugging, that would make it much speedier (and less prone to making the bots fail).\n\nReviewers: gkistanova, asmith, zturner, modocache, rnk, delcypher\n\nReviewed By: rnk\n\nSubscribers: delcypher, llvm-commits, gkistanova\n\nDifferential Revision: https://reviews.llvm.org/D51326\n\ngit-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@340840 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/llvm-project/llvm/commits/5c077985f386c44cdf408853090dcc61c544eea0"}]}
{"actor":{"display_login":"chapuni"}}