Skip to content

C
Chromium_Depot_tools
Switch branch/tag
  1. 13 May, 2022 8 commits
  3. 12 May, 2022 2 commits
  5. 11 May, 2022 1 commit
  7. 10 May, 2022 2 commits
    • Bruce Dawson's avatar
      Actually add a trailing slash to dir_with_slash · 31bfd519
      Bruce Dawson authored 
      AffectedFiles carefully added a slash to dir_with_slash, but then called
normpath - which trims trailing path separators. This meant that
ui/views/PRESUBMIT.py would analyze files in ui\views_content_client,
leading to spooky action at a distance. This was noticed when this
command triggered a presubmit error:

    git cl presubmit "--files=ui/views_content_client/*.cc;ui/views/readme.md"

when running presubmit on either file individually found nothing.

This change was tested by getting CheckChangeLintsClean to print the
files that it was asked to analyze, making the behavior quite obvious.

This bug appears to have existed for about thirteen years, but only
triggers in rare cases, and even then the incorrect behavior was almost
impossible to notice.

One of the tests was inadvertently testing the broken AffectedFiles
behavior and another seemed to be requiring it to handle '.'
correctly which I'm not sure we want to support.

Bug: 1309977
Change-Id: Ibdc39981d69664b03448acb228d4ab05b49436f6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3632034
Commit-Queue: Bruce Dawson <brucedawson@chromium.org>
Reviewed-by: 's avatarRobbie Iannucci <iannucci@chromium.org>
      31bfd519
    • Sylvain Defresne's avatar
      [mac] Restore an upper limit on the number of jobs on macOS · cb2cef9a
      Sylvain Defresne authored 
      Even when increasing the maximum number of file descriptor
as recommended in the documentation, using a large number
of concurrent jobs cause build error on certain macOS devices
(mostly iMacPro with 18 cores which ends up with a limit of
1440 jobs, and end up reaching the heightened file descriptor
limit).

Put a limit of 800 which has been found to work on all of
the devices available and still allow to have a multiplier
of 80 on highest end M1 devices (currently 10 cores).

Bug: 1317620, 936864
Change-Id: I32560c5ae9462e94f61a773d625ef3758bf05ee5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3634807Reviewed-by: 's avatarTakuto Ikuta <tikuta@chromium.org>
Auto-Submit: Sylvain Defresne <sdefresne@chromium.org>
Commit-Queue: Jochen Eisinger <jochen@chromium.org>
Reviewed-by: 's avatarJochen Eisinger <jochen@chromium.org>
      cb2cef9a
  9. 09 May, 2022 1 commit
  11. 06 May, 2022 1 commit
  13. 05 May, 2022 2 commits
  15. 04 May, 2022 3 commits
  17. 03 May, 2022 4 commits
  19. 02 May, 2022 2 commits
  21. 29 Apr, 2022 3 commits
  23. 28 Apr, 2022 3 commits
  25. 27 Apr, 2022 8 commits