Skip to content

Commit c4ae20c

Browse files
authored
Fix bazel_lint job (#205)
* Fix bazel_lint job go get is no longer a supported way to install binaries. * Apply buildifier fixes
1 parent 41c0753 commit c4ae20c

File tree

3 files changed

+5
-5
lines changed

3 files changed

+5
-5
lines changed

.github/workflows/bazel_lint.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,6 @@ jobs:
3535
- uses: actions/setup-go@v2
3636
with:
3737
go-version: '^1.13.1' # The Go version to download (if necessary) and use.
38-
- run: go get github.com/bazelbuild/buildtools/buildifier
38+
- run: go install github.com/bazelbuild/buildtools/buildifier@latest
3939
- run: buildifier -lint=warn -mode=check $(find . -type f \( -name BUILD.bazel -or -name BUILD -or -name *.bzl \))
4040

fuzzing/private/binary.bzl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ Provider for storing information about a fuzz test binary.
4545
},
4646
)
4747

48-
def _fuzzing_binary_transition_impl(settings, attr):
48+
def _fuzzing_binary_transition_impl(settings, _attr):
4949
opts = instrum_opts.make(
5050
copts = settings["//command_line_option:copt"],
5151
conlyopts = settings["//command_line_option:conlyopt"],

fuzzing/private/common.bzl

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ Rule for creating a script to run the fuzzing test.
6666
default = Label("//fuzzing/tools:launcher"),
6767
doc = "The launcher script to start the fuzzing test.",
6868
executable = True,
69-
cfg = "host",
69+
cfg = "exec",
7070
),
7171
"binary": attr.label(
7272
executable = True,
@@ -112,7 +112,7 @@ specified in the srcs attribute.
112112
default = Label("//fuzzing/tools:make_corpus_dir"),
113113
doc = "The tool script to copy and rename the corpus.",
114114
executable = True,
115-
cfg = "host",
115+
cfg = "exec",
116116
),
117117
"srcs": attr.label_list(
118118
doc = "The corpus files for the fuzzing test.",
@@ -151,7 +151,7 @@ Rule to validate the fuzzing dictionaries and output a merged dictionary.
151151
default = Label("//fuzzing/tools:validate_dict"),
152152
doc = "The tool script to validate and merge the dictionaries.",
153153
executable = True,
154-
cfg = "host",
154+
cfg = "exec",
155155
),
156156
"dicts": attr.label_list(
157157
doc = "The fuzzing dictionaries.",

0 commit comments

Comments
 (0)