[Fuchsia] remove the use of fuchsia-sdk:build/config/config.gni [chromium/mini_chromium : main]

0 views
Skip to first unread message

Zijie He (Gerrit)

unread,
May 30, 2024, 7:29:34 PMMay 30
to Mark Mentovai, Ben Pastene, Justin Cohen, crashp...@chromium.org
Attention needed from Mark Mentovai

Zijie He added 1 comment

Patchset-level comments
File-level comment, Patchset 2 (Latest):
Zijie He . resolved

Hey, Mark, would you please take a look at this change? Thank you.

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: chromium/mini_chromium
Gerrit-Branch: main
Gerrit-Change-Id: I26cf3e35e67e4ed478091781aa62e0298347ba76
Gerrit-Change-Number: 5586319
Gerrit-PatchSet: 2
Gerrit-Owner: Zijie He <zij...@google.com>
Gerrit-Reviewer: Mark Mentovai <ma...@chromium.org>
Gerrit-CC: Ben Pastene <bpas...@chromium.org>
Gerrit-CC: Justin Cohen <justi...@chromium.org>
Gerrit-Attention: Mark Mentovai <ma...@chromium.org>
Gerrit-Comment-Date: Thu, 30 May 2024 23:29:31 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
unsatisfied_requirement
open
diffy

Mark Mentovai (Gerrit)

unread,
May 31, 2024, 10:21:21 AMMay 31
to Zijie He, Ben Pastene, Justin Cohen, crashp...@chromium.org
Attention needed from Zijie He

Mark Mentovai voted Code-Review+1

Code-Review+1
Open in Gerrit

Related details

Attention is currently required from:
  • Zijie He
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: chromium/mini_chromium
Gerrit-Branch: main
Gerrit-Change-Id: I26cf3e35e67e4ed478091781aa62e0298347ba76
Gerrit-Change-Number: 5586319
Gerrit-PatchSet: 2
Gerrit-Owner: Zijie He <zij...@google.com>
Gerrit-Reviewer: Mark Mentovai <ma...@chromium.org>
Gerrit-CC: Ben Pastene <bpas...@chromium.org>
Gerrit-CC: Justin Cohen <justi...@chromium.org>
Gerrit-Attention: Zijie He <zij...@google.com>
Gerrit-Comment-Date: Fri, 31 May 2024 14:21:16 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Mark Mentovai (Gerrit)

unread,
May 31, 2024, 10:21:25 AMMay 31
to Zijie He, Ben Pastene, Justin Cohen, crashp...@chromium.org

Mark Mentovai submitted the change

Change information

Commit message:
[Fuchsia] remove the use of fuchsia-sdk:build/config/config.gni

The file has been removed already. The reference is now breaking
crashpad, see https://crrev.com/c/5585353.

```
zij...@zijiehe.c.googlers.com - Thu May 30-15:29:19:~/chromium/chromium/src$ ll third_party/fuchsia-sdk/sdk/build/
total 16
drwxr-x--- 2 zijiehe primarygroup 4096 May 28 13:27 ./
drwxr-x--- 18 zijiehe primarygroup 4096 May 28 13:27 ../
-rw-r----- 1 zijiehe primarygroup 193 May 28 13:27 component.gni
-rw-r----- 1 zijiehe primarygroup 191 May 28 13:27 package.gni
```
Bug: fuchsia:42085580, fuchsia:327691011
Change-Id: I26cf3e35e67e4ed478091781aa62e0298347ba76
Reviewed-by: Mark Mentovai <ma...@chromium.org>
Files:
  • M build/sysroot.gni
Change size: XS
Delta: 1 file changed, 0 insertions(+), 6 deletions(-)
Branch: refs/heads/main
Submit Requirements:
  • requirement satisfiedCode-Review: +1 by Mark Mentovai
Open in Gerrit
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: merged
Gerrit-Project: chromium/mini_chromium
Gerrit-Branch: main
Gerrit-Change-Id: I26cf3e35e67e4ed478091781aa62e0298347ba76
Gerrit-Change-Number: 5586319
Gerrit-PatchSet: 3
open
diffy
satisfied_requirement
Reply all
Reply to author
Forward
0 new messages