Skip to content

Commit 67d27da

Browse files
dmitshurgopherbot
authored andcommitted
main.star: fix up top-level edge case for files related to vendored code
The good news is that CL 594615 works on paths like src/cmd/go.mod; see CL 594635. The bad news, I realize now, is that I fumbled the top-level edge case. Fix it. For golang/go#42661. Change-Id: Ie993d5789b3185e822a254ee8953c25ed9055ebe Reviewed-on: https://go-review.googlesource.com/c/build/+/594616 LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org> Reviewed-by: Dmitri Shuralyov <dmitshur@google.com> Reviewed-by: Michael Knyszek <mknyszek@google.com>
1 parent e61ecf6 commit 67d27da

File tree

2 files changed

+12
-12
lines changed

2 files changed

+12
-12
lines changed

generated/commit-queue.cfg

+10-10
Original file line numberDiff line numberDiff line change
@@ -29340,12 +29340,12 @@ config_groups {
2934029340
location_filters {
2934129341
gerrit_host_regexp: "go-review.googlesource.com"
2934229342
gerrit_project_regexp: "^go$"
29343-
path_regexp: "src/.+/go[.](mod|sum)"
29343+
path_regexp: "src(|/.+)/go[.](mod|sum)"
2934429344
}
2934529345
location_filters {
2934629346
gerrit_host_regexp: "go-review.googlesource.com"
2934729347
gerrit_project_regexp: "^go$"
29348-
path_regexp: "src/.+/vendor/.+"
29348+
path_regexp: "src(|/.+)/vendor/.+"
2934929349
}
2935029350
location_filters {
2935129351
gerrit_host_regexp: "go-review.googlesource.com"
@@ -29387,12 +29387,12 @@ config_groups {
2938729387
location_filters {
2938829388
gerrit_host_regexp: "go-review.googlesource.com"
2938929389
gerrit_project_regexp: "^go$"
29390-
path_regexp: "src/.+/go[.](mod|sum)"
29390+
path_regexp: "src(|/.+)/go[.](mod|sum)"
2939129391
}
2939229392
location_filters {
2939329393
gerrit_host_regexp: "go-review.googlesource.com"
2939429394
gerrit_project_regexp: "^go$"
29395-
path_regexp: "src/.+/vendor/.+"
29395+
path_regexp: "src(|/.+)/vendor/.+"
2939629396
}
2939729397
location_filters {
2939829398
gerrit_host_regexp: "go-review.googlesource.com"
@@ -29411,12 +29411,12 @@ config_groups {
2941129411
location_filters {
2941229412
gerrit_host_regexp: "go-review.googlesource.com"
2941329413
gerrit_project_regexp: "^go$"
29414-
path_regexp: "src/.+/go[.](mod|sum)"
29414+
path_regexp: "src(|/.+)/go[.](mod|sum)"
2941529415
}
2941629416
location_filters {
2941729417
gerrit_host_regexp: "go-review.googlesource.com"
2941829418
gerrit_project_regexp: "^go$"
29419-
path_regexp: "src/.+/vendor/.+"
29419+
path_regexp: "src(|/.+)/vendor/.+"
2942029420
}
2942129421
location_filters {
2942229422
gerrit_host_regexp: "go-review.googlesource.com"
@@ -29528,12 +29528,12 @@ config_groups {
2952829528
location_filters {
2952929529
gerrit_host_regexp: "go-review.googlesource.com"
2953029530
gerrit_project_regexp: "^go$"
29531-
path_regexp: "src/.+/go[.](mod|sum)"
29531+
path_regexp: "src(|/.+)/go[.](mod|sum)"
2953229532
}
2953329533
location_filters {
2953429534
gerrit_host_regexp: "go-review.googlesource.com"
2953529535
gerrit_project_regexp: "^go$"
29536-
path_regexp: "src/.+/vendor/.+"
29536+
path_regexp: "src(|/.+)/vendor/.+"
2953729537
}
2953829538
location_filters {
2953929539
gerrit_host_regexp: "go-review.googlesource.com"
@@ -29703,12 +29703,12 @@ config_groups {
2970329703
location_filters {
2970429704
gerrit_host_regexp: "go-review.googlesource.com"
2970529705
gerrit_project_regexp: "^go$"
29706-
path_regexp: "src/.+/go[.](mod|sum)"
29706+
path_regexp: "src(|/.+)/go[.](mod|sum)"
2970729707
}
2970829708
location_filters {
2970929709
gerrit_host_regexp: "go-review.googlesource.com"
2971029710
gerrit_project_regexp: "^go$"
29711-
path_regexp: "src/.+/vendor/.+"
29711+
path_regexp: "src(|/.+)/vendor/.+"
2971229712
}
2971329713
location_filters {
2971429714
gerrit_host_regexp: "go-review.googlesource.com"

main.star

+2-2
Original file line numberDiff line numberDiff line change
@@ -804,8 +804,8 @@ RUN_MODS = dict(
804804
"build": [],
805805
"go": [
806806
# Enable longtest builders on go against tip if files related to vendored code are modified.
807-
"src/.+/go[.](mod|sum)",
808-
"src/.+/vendor/.+",
807+
"src(|/.+)/go[.](mod|sum)",
808+
"src(|/.+)/vendor/.+",
809809
"src/.+_bundle.go",
810810
# Enable longtest builders on go against tip if files in the crypto/tls tree are modified,
811811
# so that the BoGo test suite is run.

0 commit comments

Comments
 (0)