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
Gazelle does not seem to pick up the docker_config.go file added by the patch; manually adding it to the generated BUILD.bazel file solves the problem.
The text was updated successfully, but these errors were encountered:
Patches are applied after Gazelle is run, so this is expected. We could have "after Gazelle patches", but I'm not sure whether that's worth the complexity. How much larger is your patch due to this?
If I'm understanding you correctly, are you asking how much we have to add to our patch to make it work? Like could we just add a section to the patch to add the missing line to the generated BUILD.bazel? If that's possible (I'm not that that familiar with patch mechanics), I suppose we could. Its just a bit weird to support some patch operations and require special hacking for others.
My team, and hopefully other teams, are not overly reliant on patching, so totally understand if this isn't a boundry case you plan to support.
What version of rules_go are you using?
0.50.0
What version of gazelle are you using?
0.38.0
What version of Bazel are you using?
7.3.1
Does this issue reproduce with the latest releases of all the above?
Yes
What operating system and processor architecture are you using?
macOS
Any other potentially useful information about your toolchain?
What did you do?
Attepmted to apply this patch with the following directive after our
use_repo(go_deps,...
statement inMODULE.bazel
:What did you expect to see?
A successful build.
What did you see instead?
Gazelle does not seem to pick up the
docker_config.go
file added by the patch; manually adding it to the generatedBUILD.bazel
file solves the problem.The text was updated successfully, but these errors were encountered: