[DEPS] Rely on buildtools recursedep to fetch clang-format [crashpad/crashpad : main]

0 views
Skip to first unread message

Josip Sokcevic (Gerrit)

unread,
May 30, 2024, 6:34:34 PMMay 30
to Mark Mentovai, crashp...@chromium.org
Attention needed from Mark Mentovai

Josip Sokcevic added 1 comment

Patchset-level comments
File-level comment, Patchset 1 (Latest):
Josip Sokcevic . resolved

Tested locally by modifying one of the c files, removed ./crashpad/buildtools/linux64/format/clang-format, re-ran gclient sync and executed git cl format.

Open in Gerrit

Related details

Attention is currently required from:
  • Mark Mentovai
Submit Requirements:
  • requirement is not satisfiedCode-Review
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: crashpad/crashpad
Gerrit-Branch: main
Gerrit-Change-Id: I2c29ad9df579b605ff6e1d967da6502ffcd12faf
Gerrit-Change-Number: 5586422
Gerrit-PatchSet: 1
Gerrit-Owner: Josip Sokcevic <sokc...@chromium.org>
Gerrit-Reviewer: Mark Mentovai <ma...@chromium.org>
Gerrit-Attention: Mark Mentovai <ma...@chromium.org>
Gerrit-Comment-Date: Thu, 30 May 2024 22:34:31 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
unsatisfied_requirement
open
diffy

Mark Mentovai (Gerrit)

unread,
May 31, 2024, 10:21:58 AMMay 31
to Josip Sokcevic, crashp...@chromium.org
Attention needed from Josip Sokcevic

Mark Mentovai voted and added 1 comment

Votes added by Mark Mentovai

Code-Review+1

1 comment

Patchset-level comments
Mark Mentovai . resolved

Great! Thanks.

Open in Gerrit

Related details

Attention is currently required from:
  • Josip Sokcevic
Submit Requirements:
  • requirement satisfiedCode-Review
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: crashpad/crashpad
Gerrit-Branch: main
Gerrit-Change-Id: I2c29ad9df579b605ff6e1d967da6502ffcd12faf
Gerrit-Change-Number: 5586422
Gerrit-PatchSet: 1
Gerrit-Owner: Josip Sokcevic <sokc...@chromium.org>
Gerrit-Reviewer: Mark Mentovai <ma...@chromium.org>
Gerrit-Attention: Josip Sokcevic <sokc...@chromium.org>
Gerrit-Comment-Date: Fri, 31 May 2024 14:21:52 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Josip Sokcevic (Gerrit)

unread,
May 31, 2024, 10:28:24 AMMay 31
to Mark Mentovai, crashp...@chromium.org

Josip Sokcevic voted Commit-Queue+2

Commit-Queue+2
Open in Gerrit

Related details

Attention set is empty
Submit Requirements:
  • requirement satisfiedCode-Review
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: crashpad/crashpad
Gerrit-Branch: main
Gerrit-Change-Id: I2c29ad9df579b605ff6e1d967da6502ffcd12faf
Gerrit-Change-Number: 5586422
Gerrit-PatchSet: 1
Gerrit-Owner: Josip Sokcevic <sokc...@chromium.org>
Gerrit-Reviewer: Josip Sokcevic <sokc...@chromium.org>
Gerrit-Reviewer: Mark Mentovai <ma...@chromium.org>
Gerrit-Comment-Date: Fri, 31 May 2024 14:28:21 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Josip Sokcevic (Gerrit)

unread,
Jun 4, 2024, 12:20:00 PMJun 4
to Code Review Nudger, Crashpad LUCI CQ, Mark Mentovai, crashp...@chromium.org

Josip Sokcevic added 1 comment

Patchset-level comments
Josip Sokcevic . resolved

This is failing due to updated version of buildtools. Filed http://b/344879760 to get that fixed.

Open in Gerrit

Related details

Attention set is empty
Submit Requirements:
  • requirement satisfiedCode-Review
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: crashpad/crashpad
Gerrit-Branch: main
Gerrit-Change-Id: I2c29ad9df579b605ff6e1d967da6502ffcd12faf
Gerrit-Change-Number: 5586422
Gerrit-PatchSet: 1
Gerrit-Owner: Josip Sokcevic <sokc...@chromium.org>
Gerrit-Reviewer: Josip Sokcevic <sokc...@chromium.org>
Gerrit-Reviewer: Mark Mentovai <ma...@chromium.org>
Gerrit-CC: Code Review Nudger <android-build...@prod.google.com>
Gerrit-Comment-Date: Tue, 04 Jun 2024 16:19:57 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
satisfied_requirement
open
diffy
Reply all
Reply to author
Forward
0 new messages