Skip to content

Add a bazel_feature to gate the grep_includes parameter#17

Merged
meteorcloudy merged 3 commits intobazel-contrib:mainfrom
scentini:grep_includes
Jul 31, 2023
Merged

Add a bazel_feature to gate the grep_includes parameter#17
meteorcloudy merged 3 commits intobazel-contrib:mainfrom
scentini:grep_includes

Conversation

@scentini
Copy link
Copy Markdown
Contributor

The grep_includes parameter has been made optional in cc_common APIs as of Bazel 6.3 (e.g. in bazelbuild/bazel#18823). This feature allows for rulesets to support versions < 6.3, where the grep_includes parameter is mandatory, as well as to prepare for future Bazel releases where the grep_includes parameter will not exist.

@meteorcloudy meteorcloudy requested review from Wyverald and fmeum July 31, 2023 10:16
@meteorcloudy meteorcloudy merged commit 4c01421 into bazel-contrib:main Jul 31, 2023
@meteorcloudy
Copy link
Copy Markdown
Collaborator

@fmeum @alexeagle we probably need a new release for rules_rust.

@scentini scentini deleted the grep_includes branch July 31, 2023 13:54
@fmeum
Copy link
Copy Markdown
Member

fmeum commented Jul 31, 2023

I have never done releases in this repo, do I need to do more than push a tag?

@alexeagle
Copy link
Copy Markdown

Just push a tag to this repo when you want a release!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants