[Lldb-commits] [lldb] af5504e - Increase timeout to find a dSYM in macos DownloadObjectAndSymbolFile

Jason Molenda via lldb-commits lldb-commits at lists.llvm.org
Thu Oct 15 01:00:25 PDT 2020


Author: Jason Molenda
Date: 2020-10-15T00:57:23-07:00
New Revision: af5504edd6815d085de8aa66efcd7150d6315a51

URL: https://github.com/llvm/llvm-project/commit/af5504edd6815d085de8aa66efcd7150d6315a51
DIFF: https://github.com/llvm/llvm-project/commit/af5504edd6815d085de8aa66efcd7150d6315a51.diff

LOG: Increase timeout to find a dSYM in macos DownloadObjectAndSymbolFile

With a large dSYM over a slow home connection, the two minute timeout
would sometimes be exceeded, and we haven't seen instances of a
long timeout causing people any problems, so we're bumping it up.

640 seconds ought to be enough for anyone.

<rdar://problem/67759526>

Added: 
    

Modified: 
    lldb/source/Symbol/LocateSymbolFileMacOSX.cpp

Removed: 
    


################################################################################
diff  --git a/lldb/source/Symbol/LocateSymbolFileMacOSX.cpp b/lldb/source/Symbol/LocateSymbolFileMacOSX.cpp
index fe30ceeabcb2..344bac8e0632 100644
--- a/lldb/source/Symbol/LocateSymbolFileMacOSX.cpp
+++ b/lldb/source/Symbol/LocateSymbolFileMacOSX.cpp
@@ -621,7 +621,7 @@ bool Symbols::DownloadObjectAndSymbolFile(ModuleSpec &module_spec,
             &signo,          // Signal int *
             &command_output, // Command output
             std::chrono::seconds(
-               120), // Large timeout to allow for long dsym download times
+               640), // Large timeout to allow for long dsym download times
             false);  // Don't run in a shell (we don't need shell expansion)
         if (error.Success() && exit_status == 0 && !command_output.empty()) {
           CFCData data(CFDataCreateWithBytesNoCopy(


        


More information about the lldb-commits mailing list