• Dirk Pranke's avatar
    Revert "[bootstraps] Sync at gclient, suppress output." · 44855b94
    Dirk Pranke authored
    This reverts commit 4d92fe43.
    
    Reason for revert: This moved the logging from invoking `led` or `vpython` to whenever someone would run something that would end up invoking `update_depot_tools`. It's good that we are calling this when we run update_depot_tools, but we probably should've suppressed the logging there in at least the success case, because now things are even more confusing. See crbug.com/748651.
    
    Original change's description:
    > [bootstraps] Sync at gclient, suppress output.
    > 
    > Have the "update_depot_tools" script also do a CIPD tool sync. This will
    > ensure that users and systems have access to tooling at sync-time rather
    > than just-in-time loading them at execution time.
    > 
    > Update the tool boostraps to suppress any sort of syncing logic, if it
    > does happen. This will ensure that users who execute the tools don't se
    > unexpected output.
    > 
    > BUG=None
    > TEST=local
    >   - Tested on Mac and Windows.
    > 
    > R=​dpranke@chromium.org, iannucci@chromium.org
    > 
    > Change-Id: I8efce8c73cc4e82ffdf5067ba9b917119a81e843
    > Reviewed-on: https://chromium-review.googlesource.com/581494
    > Commit-Queue: Daniel Jacques <dnj@chromium.org>
    > Reviewed-by: Robbie Iannucci <iannucci@chromium.org>
    
    TBR=iannucci@chromium.org,dpranke@chromium.org,dnj@chromium.org
    
    # Not skipping CQ checks because original CL landed > 1 day ago.
    
    Bug: None
    Change-Id: I2485c9dd2e48a8dbdeebfff5da9d4c708e0edcb7
    Reviewed-on: https://chromium-review.googlesource.com/585867Reviewed-by: 's avatarDirk Pranke <dpranke@chromium.org>
    Commit-Queue: Dirk Pranke <dpranke@chromium.org>
    44855b94
led.bat 244 Bytes