-
Notifications
You must be signed in to change notification settings - Fork 107
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
name 'module_extension' is not defined when used in Tensorflow-1.15.5 version #176
Comments
Hi @linlang1837! 👋🏻 That's a super old version of bazel! (And that's the cause of the issue.) I don't think it makes sense for us to support something that far back. Any chance you can upgrade? If not, the versions before we added bzlmod support should work for you, but you won't get the latest bug fixes. Chris |
Thanks, I will try the versions before adding bzlmod support. By the way, how can I can find the version before adding bzlmod support. |
Just do a quick search of the commits! Oh, actually, well, this isn't really future-proof and I wouldn't necessarily recommend it generally, but you could instead just skip calling all those workspace_setup functions. It'll break in some future commit, but that way you'd get the latest fixes for now but skip those errors. Really, it makes sense to update from that ancient, pre-release version of Bazel! Bazel 5 is the oldest one that they're even considering to be in maintenance mode--and the latest those lines work with--I'd strongly recommend bazelisk and the rolling releases. |
thanks, I will try. |
Bazel version: 0.26.1
ERROR: /root/.cache/bazel/_bazel_root/80709a9f7d7f227d2bcf9e421ac096e0/external/hedron_compile_commands/workspace_setup.bzl:28:37: name 'module_extension' is not defined
ERROR: error loading package '': Extension 'workspace_setup.bzl' has errors
ERROR: error loading package '': Extension 'workspace_setup.bzl' has errors
The text was updated successfully, but these errors were encountered: