-
Notifications
You must be signed in to change notification settings - Fork 96
ERROR: Failed to load Skylark extension '@bazel_skylib//:lib.bzl'. #233
Comments
Duplicate of #202 We should have noted the breaking change. I think this was introduced when we added a dependency on rules_webtesting. bazelbuild/rules_webtesting@7a29649 |
Thanks! One thing I've noticed through this though, is that depending on |
right, you should never have floating dependencies - always make sure your
dependencies are locked so that breakages occur only when you make a change.
…On Tue, Jul 24, 2018 at 12:46 AM Ofer Bartal ***@***.***> wrote:
Thanks!
This commit fixes the issue: ***@***.***
<google/startup-os@f911766>
One thing I've noticed through this though, is that depending on master
makes bazel builds non-reproducible, as in this example where it was
working until I did bazel clean --expunge which made it stop working
(probably due to getting the latest master).
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<#233 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAC5I50sYp75v1yUnqUytv4-3LRh8S9pks5uJtDygaJpZM4Vbp1D>
.
|
We have one developer running into this problem, but we don't know why yet. We're are pinned to a specific commit in our WORKSPACE:
|
We think we figured it out. Our rules_webtesting dependency points to their master because that's what their README has as the examples. |
I'm getting the following error:
It disappears when I remove the following from
WORKSPACE
:load("@build_bazel_rules_typescript//:defs.bzl", "ts_setup_workspace")
ts_setup_workspace()
That's why I'm opening this issue here.
Another odd thing, is that it worked, until I did
bazel clean --expunge
and then it started giving the error.Steps to reproduce (done on Linux):
git clone https://github.com/google/startup-os
bazel build //common:all
(though any build seems to give the error)Thanks!
The text was updated successfully, but these errors were encountered: