This is an archive of the discontinued LLVM Phabricator instance.

[Bazel] Typo fix
ClosedPublic

Authored by xgupta on Jul 30 2021, 2:37 AM.

Diff Detail

Event Timeline

xgupta requested review of this revision.Jul 30 2021, 2:37 AM
xgupta created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptJul 30 2021, 2:37 AM

For some reason, Bazel build is failing on buildbot. Actually in utils/bazel/WORKSPACE seems SKYLIB_VERSION = "1.0.3" is not working, I need to copy the bazel-skylib WORKSPACE setup from https://github.com/bazelbuild/bazel-skylib/releases/tag/1.0.3 and makes URL values hardcoded.

GMNGeoffrey accepted this revision.Jul 30 2021, 8:17 AM

Thanks for the fix :-) The error is because you're synced a bit behind and the digest for the bazel-skylib archive was previously messed up, but fixed in https://reviews.llvm.org/rG91842bf6aa93bf7b71c9b55687620df08d9e4170

This is a doc fix, so quite sure it doesn't break the build :-)

This revision is now accepted and ready to land.Jul 30 2021, 8:17 AM

Do you have commit access, or do you need me to land this for you?

Yes, please land this for me. Author property should be "Shivam Gupta <xgupta@aol.com>"

-shivam

This revision was landed with ongoing or failed builds.Jul 30 2021, 8:38 AM
This revision was automatically updated to reflect the committed changes.