[Lldb-commits] [PATCH] D55607: Make crashlog.py work when a .dSYM is present, but a binary is missing
Stella Stamenova via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Tue Dec 18 10:53:43 PST 2018
stella.stamenova added a comment.
In D55607#1334666 <https://reviews.llvm.org/D55607#1334666>, @aprantl wrote:
> Can you confirm that reverting this path actually fixes the issue? I'm asking because the only test that is executing this script has a REQUIRES: system-darwin line in it.
Removing the test does fix the issue. I suspect the problem is the ordering of the REQUIRES and RUN lines, but I have not tested it. Regardless, the crashlog changes were the only two changes to LLDB in the first build that got stuck (there was also an llvm change). Please fix/revert this in the next couple of hours as we haven't had a good test run for LLDB on Windows since your change.
CHANGES SINCE LAST ACTION
More information about the lldb-commits