You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
rules_fuzzing is compatible with recent versions of rules_python.
Actual Behavior
rules_fuzzing_init calls pip_parse, but the signature of that function changed in rules_python 0.13 (specifically in bazelbuild/rules_python#807). Instead of a single requirements attribute, there's now three per-OS attributes (requirements_linux, requirements_windows, requirements_darwin).
This means a project that uses rules_fuzzing needs to replace a call to rules_fuzzing_init with a reimplementation that looks like,
The fix is simple enough (change the rules_fuzzing_init to look like this), but will make rules_fuzzing incompatible with versions of rules_python older than 0.13. That version is a little over a year old, it came out on September 25, 2022.
I'll send a PR.
Specifications
Version: Still broken on the master branch.
Platform: Broken on any platform.
The text was updated successfully, but these errors were encountered:
I ended up a little confused about what's going on here, because rules_fuzzing apparently already uses rules_python 0.20, in which pip_parse should expect per-OS requirements.
It turns out that until bazelbuild/rules_python#1514, pip_parse had a wrapper that accepted requirements as an argument, and passed them to pip_repository as requirements_lock. So the incompatibility will only be with the next version of rules_python. I'll update the issue title accordingly.
tpudlik
changed the title
rules_fuzzing incompatible with rules_python >= 0.13
rules_fuzzing incompatible with rules_python > 0.26
Nov 16, 2023
tpudlik
added a commit
to tpudlik/rules_fuzzing
that referenced
this issue
Nov 16, 2023
Expected Behavior
rules_fuzzing
is compatible with recent versions ofrules_python
.Actual Behavior
rules_fuzzing_init
callspip_parse
, but the signature of that function changed in rules_python 0.13 (specifically in bazelbuild/rules_python#807). Instead of a singlerequirements
attribute, there's now three per-OS attributes (requirements_linux
,requirements_windows
,requirements_darwin
).This means a project that uses
rules_fuzzing
needs to replace a call torules_fuzzing_init
with a reimplementation that looks like,The fix is simple enough (change the
rules_fuzzing_init
to look like this), but will makerules_fuzzing
incompatible with versions ofrules_python
older than 0.13. That version is a little over a year old, it came out on September 25, 2022.I'll send a PR.
Specifications
master
branch.The text was updated successfully, but these errors were encountered: