Skip to content
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

Exception when bazel run :target where target is a genrule #372

Closed
damienmg opened this issue Aug 12, 2015 · 6 comments
Closed

Exception when bazel run :target where target is a genrule #372

damienmg opened this issue Aug 12, 2015 · 6 comments
Assignees
Labels
category: misc > misc P3 We're not considering working on this, but happy to review a PR. (No assignee) type: bug
Milestone

Comments

@damienmg
Copy link
Contributor

$ bazel run :ubuntu_14.04-x86_64
Unhandled exception thrown during build; message: Attribute 'executable' is potentially configurable - not allowed here
INFO: Elapsed time: 0.791s
java.lang.IllegalStateException: Attribute 'executable' is potentially configurable - not allowed here
    at com.google.common.base.Preconditions.checkState(Preconditions.java:197)
    at com.google.devtools.build.lib.packages.NonconfigurableAttributeMapper.get(NonconfigurableAttributeMapper.java:45)
    at com.google.devtools.build.lib.bazel.rules.genrule.BazelGenRuleRule$1.getDefault(BazelGenRuleRule.java:196)
    at com.google.devtools.build.lib.packages.AbstractAttributeMapper.get(AbstractAttributeMapper.java:61)
    at com.google.devtools.build.lib.packages.NonconfigurableAttributeMapper.get(NonconfigurableAttributeMapper.java:47)
    at com.google.devtools.build.lib.runtime.commands.RunCommand.isExecutableNonTestRule(RunCommand.java:510)
    at com.google.devtools.build.lib.runtime.commands.RunCommand.isExecutable(RunCommand.java:496)
    at com.google.devtools.build.lib.runtime.commands.RunCommand.validateTarget(RunCommand.java:447)
    at com.google.devtools.build.lib.runtime.commands.RunCommand.validateTargets(RunCommand.java:403)
    at com.google.devtools.build.lib.runtime.commands.RunCommand.access$000(RunCommand.java:83)
    at com.google.devtools.build.lib.runtime.commands.RunCommand$1.validateTargets(RunCommand.java:117)
    at com.google.devtools.build.lib.buildtool.BuildTool$1.notifyTargets(BuildTool.java:410)
    at com.google.devtools.build.lib.pkgcache.LoadingPhaseRunner.execute(LoadingPhaseRunner.java:391)
    at com.google.devtools.build.lib.buildtool.BuildTool.runLoadingPhase(BuildTool.java:420)
    at com.google.devtools.build.lib.buildtool.BuildTool.buildTargets(BuildTool.java:165)
    at com.google.devtools.build.lib.buildtool.BuildTool.processRequest(BuildTool.java:342)
    at com.google.devtools.build.lib.runtime.commands.RunCommand.processRequest(RunCommand.java:113)
    at com.google.devtools.build.lib.runtime.commands.RunCommand.exec(RunCommand.java:152)
    at com.google.devtools.build.lib.runtime.BlazeCommandDispatcher.exec(BlazeCommandDispatcher.java:427)
    at com.google.devtools.build.lib.runtime.BlazeRuntime$3.exec(BlazeRuntime.java:1341)
    at com.google.devtools.build.lib.server.RPCService.executeRequest(RPCService.java:65)
    at com.google.devtools.build.lib.server.RPCServer.executeRequest(RPCServer.java:420)
    at com.google.devtools.build.lib.server.RPCServer.serve(RPCServer.java:227)
    at com.google.devtools.build.lib.runtime.BlazeRuntime.serverMain(BlazeRuntime.java:1299)
    at com.google.devtools.build.lib.runtime.BlazeRuntime.main(BlazeRuntime.java:1095)
    at com.google.devtools.build.lib.bazel.BazelMain.main(BazelMain.java:49)
@damienmg damienmg added type: bug P3 We're not considering working on this, but happy to review a PR. (No assignee) labels Aug 12, 2015
@stephenchu
Copy link

I hit the same thing, with an extremely simple test case to reproduce.

Am I doing something wrong? Or, does this have a workaround?

Steps to Reproduce

$ touch WORKSPACE

$ mkdir one

$ echo "FILE1 CONTENT" > one/file1.txt

$ echo '
genrule(
  name = "buggy",
  srcs = [],
  outs = ["output.txt"],
  cmd = "cat file1.txt > $@",
)
' > one/BUILD

Output

$ bazel run //one:buggy -- cat_output.txt
.......
WARNING: Sandboxed execution is not supported on your system and thus hermeticity of actions cannot be guaranteed. See http://bazel.io/docs/bazel-user-manual.html#sandboxing for more information. You can turn off this warning via --ignore_unsupported_sandboxing.
Unhandled exception thrown during build; message: Attribute 'executable' is potentially configurable - not allowed here
INFO: Elapsed time: 0.472s
java.lang.IllegalStateException: Attribute 'executable' is potentially configurable - not allowed here
        at com.google.common.base.Preconditions.checkState(Preconditions.java:197)
        at com.google.devtools.build.lib.packages.NonconfigurableAttributeMapper.get(NonconfigurableAttributeMapper.java:45)
        at com.google.devtools.build.lib.bazel.rules.genrule.BazelGenRuleRule$1.getDefault(BazelGenRuleRule.java:204)
        at com.google.devtools.build.lib.packages.AbstractAttributeMapper.get(AbstractAttributeMapper.java:67)
        at com.google.devtools.build.lib.packages.NonconfigurableAttributeMapper.get(NonconfigurableAttributeMapper.java:47)
        at com.google.devtools.build.lib.runtime.commands.RunCommand.isExecutableNonTestRule(RunCommand.java:512)
        at com.google.devtools.build.lib.runtime.commands.RunCommand.isExecutable(RunCommand.java:498)
        at com.google.devtools.build.lib.runtime.commands.RunCommand.validateTarget(RunCommand.java:449)
        at com.google.devtools.build.lib.runtime.commands.RunCommand.validateTargets(RunCommand.java:405)
        at com.google.devtools.build.lib.runtime.commands.RunCommand.access$000(RunCommand.java:85)
        at com.google.devtools.build.lib.runtime.commands.RunCommand$1.validateTargets(RunCommand.java:119)
        at com.google.devtools.build.lib.buildtool.BuildTool$1.notifyTargets(BuildTool.java:391)
        at com.google.devtools.build.lib.pkgcache.LegacyLoadingPhaseRunner.execute(LegacyLoadingPhaseRunner.java:201)
        at com.google.devtools.build.lib.buildtool.BuildTool.runLoadingPhase(BuildTool.java:401)
        at com.google.devtools.build.lib.buildtool.BuildTool.buildTargets(BuildTool.java:165)
        at com.google.devtools.build.lib.buildtool.BuildTool.processRequest(BuildTool.java:334)
        at com.google.devtools.build.lib.runtime.commands.RunCommand.processRequest(RunCommand.java:115)
        at com.google.devtools.build.lib.runtime.commands.RunCommand.exec(RunCommand.java:155)
        at com.google.devtools.build.lib.runtime.BlazeCommandDispatcher.exec(BlazeCommandDispatcher.java:370)
        at com.google.devtools.build.lib.runtime.BlazeRuntime$3.exec(BlazeRuntime.java:1009)
        at com.google.devtools.build.lib.server.RPCService.executeRequest(RPCService.java:65)
        at com.google.devtools.build.lib.server.RPCServer.executeRequest(RPCServer.java:434)
        at com.google.devtools.build.lib.server.RPCServer.serve(RPCServer.java:229)
        at com.google.devtools.build.lib.runtime.BlazeRuntime.serverMain(BlazeRuntime.java:968)
        at com.google.devtools.build.lib.runtime.BlazeRuntime.main(BlazeRuntime.java:765)
        at com.google.devtools.build.lib.bazel.BazelMain.main(BazelMain.java:55)

$ bazel version
Build label: head (@f1c411b)
Build target: bazel-out/local_linux-fastbuild/bin/src/main/java/com/google/devtools/build/lib/bazel-main_deploy.jar
Build time: Tue Nov 24 23:34:22 2015 (1448408062)
Build timestamp: 1448408062
Build timestamp as int: 1448408062

@damienmg
Copy link
Contributor Author

Yes a genrule is not runnable except if you set the executable attribute to 1. In which case the output of the genrule can be run with bazel run. However, this is wrong to throw a stack trace and we should have an explicit message for the user instead.

What you want to do is simply `bazel build //one:buggy && cat bazel-genfiles/one/output.txt

Note that you should declare "file1.txt" as an input of your genrule in the srcs attribute (your system don't support sandboxing so you won't see the problem on yours but in system with sandboxing on, your command would fail).

@stephenchu
Copy link

Thanks for the explanation. It makes perfect sense.

Do you want me to repurpose this issue to ~"bazel run a genrule should not throw stack trace"?

@kchodorow
Copy link
Contributor

The title seems fine as-is, but thanks.

@damienmg damienmg modified the milestone: 1.0 Jun 14, 2016
@helenalt
Copy link
Contributor

@gregestren are you still the right assignee for this issue?

@gregestren
Copy link
Contributor

This was fixed by 291032a#diff-dc563e4148084671f7d953e1fee35737.

bazel-io pushed a commit that referenced this issue Aug 12, 2021
  - 3c7ec07fe0418446ffdb1a04c671a3810d74ae30 [cas] Add cas package (#300) by nodirg <56001730+nodirg@users.noreply.github.com>
  - 28fa42989a6c2d05cddb6b42494f34ef742c3de9 [cas] Implement file reading (#302) by nodirg <56001730+nodirg@users.noreply.github.com>
  - 69c6642c3a006636256c1d2e591899f07d4c74bf Simplify caching packages (#303) by nodirg <56001730+nodirg@users.noreply.github.com>
  - f7087af662fe5481f2e75abd652af46d9a376534 [cas] Implement presence check (#304) by nodirg <56001730+nodirg@users.noreply.github.com>
  - 847bca232f884b1e7ee059c64317a5140f11477e Simplify caching packages further (#306) by nodirg <56001730+nodirg@users.noreply.github.com>
  - 989513ef4a567a812f21b27f2a5d83f1a2145600 Rename singleflightcache.Cache (#310) by nodirg <56001730+nodirg@users.noreply.github.com>
  - 7eceb37537dec7c50e52d4f1af1a5d671c267144 [cas] Fix the build (#308) by nodirg <56001730+nodirg@users.noreply.github.com>
  - fd877b05ba2ed611f6d34e711da05917b729eff9 [cas] Add retries (#311) by nodirg <56001730+nodirg@users.noreply.github.com>
  - 5bc303584ef03ded33f03fa2976015e38da9c050 [cas] Add support for Symlinks. (#309) by nodirg <56001730+nodirg@users.noreply.github.com>
  - 4cfed65947cba54af1b2259d9e3facf7dd3007b9 [cas] Implement batch upload (#307) by nodirg <56001730+nodirg@users.noreply.github.com>
  - 144126c43e73aeeabdbeb8bc4ef0290fa7a91ba7 [cas] Move file IO semaphore to Client (#313) by nodirg <56001730+nodirg@users.noreply.github.com>
  - e1da041171a715b7bab3178acfe1cd774b9f5019 [cas] Reuse file read buffers (#317) by nodirg <56001730+nodirg@users.noreply.github.com>
  - e9184e44947661852a8887f3183e00c346208aaa [cas] Add UploadOptions (#314) by nodirg <56001730+nodirg@users.noreply.github.com>
  - e2bd6c8e2d6bc183ff059c688a604ba1ac18b840 [cas] Implement ServerCapabilities check (#315) by nodirg <56001730+nodirg@users.noreply.github.com>
  - b00d91e726265f23ad0a88730545831b8d5519ef [cas] Add RPCConfig (#318) by nodirg <56001730+nodirg@users.noreply.github.com>
  - 1c678dec65b62e49840419ab777c7b6ce65cfd76 [cas] Improve error messages (#321) by nodirg <56001730+nodirg@users.noreply.github.com>
  - ad8d2cfffe1f3728469a8dd5a7531b9157280ecd [cas] Limit FindMissingBlobs concurrency (#319) by nodirg <56001730+nodirg@users.noreply.github.com>
  - b1b54ee4d55b5d5bd71ca2df4353058515581697 [cas] Increase FindMissingBlobs concurrency to 256 (#323) by nodirg <56001730+nodirg@users.noreply.github.com>
  - b4a0e12d87c946ced360d723d743be1f57a47995 [cas] Move file IO buffering deeper (#322) by nodirg <56001730+nodirg@users.noreply.github.com>
  - abb14633e09633368f06d8e4fd28bd1553509061 [cas] Add filtering/callback (#316) by nodirg <56001730+nodirg@users.noreply.github.com>
  - 7447b28dd69e22848ee850936a1cdd28e2d0e20b Add Mtime to the file metadata cache. (#326) by ramymedhat <abdelaal@google.com>
  - 7182b476eb6260fae9ba2bd8995b97a7096e340c [cas] Implement streaming (#320) by nodirg <56001730+nodirg@users.noreply.github.com>
  - b0605647bbe2ff7d046a286c3023e7714376fb83 include file path in upload error (#327) by Takuto Ikuta <tikuta@google.com>
  - 3b602dd48f7f63a76cb1087a10355b3c668d583f [cas] Implement per-request timeouts in a stream (#325) by nodirg <56001730+nodirg@users.noreply.github.com>
  - 395c674af7a9cd696dfd1f2b4a950f6899ccb3a0 remove unused variables (#332) by Takuto Ikuta <tikuta@google.com>
  - 45f49a9529f755fb586fe2f8bf3e78b1eae39e81 [cas] Unembed cas.Client.ClientConfig. (#333) by nodirg <56001730+nodirg@users.noreply.github.com>
  - 2a9b29928abe867026e37833fa480ed16238df7a [cas] Rename UploadOptions.Callback to Prelude (#334) by nodirg <56001730+nodirg@users.noreply.github.com>
  - 80ea864b211ee3d87e14f4be0ca8d17e5917062c [cas] Read files once (#335) by nodirg <56001730+nodirg@users.noreply.github.com>
  - 0e577525a2dce2d0e7bddd80927c8901d28bf0fb [cas] PathExclude: use forward-slash-separated paths (#341) by nodirg <56001730+nodirg@users.noreply.github.com>
  - e155d015bcc4c9eb9978572422e3404f26220700 Add useful error message for uploading files. (#339) by bansalvinayak <vinayakbansal@google.com>
  - 3dfb518d390280a2ffef5a85ab1852a2a526a983 [cas] Rename UploadInput to PathSpec (#342) by nodirg <56001730+nodirg@users.noreply.github.com>
  - 752e4efb2631b45f5c27120f83d517cc2b2846d2 google/uuid -> pborman/uuid (#344) by Rubens Farias <rubensf@google.com>
  - d94f8a8ba888d384686a7cd74d8a9d0795ba4b6d Small tweaks to appease internal import checks. (#346) by Rubens Farias <rubensf@google.com>
  - f9d52cdef1c3aa8612d9c3ed7a0b65f96e55d870 Catch another pool check (#347) by Rubens Farias <rubensf@google.com>
  - ead1458eda2b7c756138429121a3e22bd5c9aa5a Preserving symlink or not can be configured from Command.... by Yoshisato Yanagisawa <yoshisato.yanagisawa@gmail.com>
  - f831c118b9c9e1dd3e857fbe43da7c992c91b20f make (*Chunker).Reset returns error (#348) by Takuto Ikuta <tikuta@google.com>
  - 1a7d2a4198fa0eb8515593b5259f495cdacf75ab [cas] Require PathSpec.Path to be absolute (#345) by nodirg <56001730+nodirg@users.noreply.github.com>
  - dd2d3976ed7c6482f361de7d24e24d5a8683c56c Upgrade zstdpool-syncpool for DecoderWrapper.Close bugfix... by Mostyn Bramley-Moore <mostyn@antipode.se>
  - 8544bdc0f3112900675a72decdb7a978cece0be7 [cas] Clean PathSpec.Path (#352) by nodirg <56001730+nodirg@users.noreply.github.com>
  - 882e3342509eb038dc04bf44581e5085c0320d16 [cas] Add UploadResult.Digest() (#340) by nodirg <56001730+nodirg@users.noreply.github.com>
  - 5d4d813411299a285f113cb541f2c5750746cdfa chunker: remove unused field from Chunker (#355) by Takuto Ikuta <tikuta@google.com>
  - a5af2d4316599a3fea87a459290e5f88a022a43c Add return value names in singleflightcache (#356) by nodirg <56001730+nodirg@users.noreply.github.com>
  - e7ea26b93b496d4d30e98390682faf6f69f84cb7 [cas] Fix joinFilePathsFast (#361) by nodirg <56001730+nodirg@users.noreply.github.com>
  - 5a8daf747858747b2bf6bdd59e7b07dedc17a244 [cas] Refactor code (#359) by nodirg <56001730+nodirg@users.noreply.github.com>
  - dd6c290b2ce791f3a3e56583a9632f7b538e8a05 [cas] Simplify UploadResult.Digest() signature (#362) by nodirg <56001730+nodirg@users.noreply.github.com>
  - 05222e7e8939959878a5798e51cf52e911feafaf Fix lint (#357) by nodirg <56001730+nodirg@users.noreply.github.com>
  - b2689fabc306d2cd20356f30b7f99eec445d9212 [cas] Refactor Digest() (#363) by nodirg <56001730+nodirg@users.noreply.github.com>
  - c672e5baca9280d181c7e2e87b3d9afd01650893 [cas] Add PathSpec.Allowlist (#360) by nodirg <56001730+nodirg@users.noreply.github.com>
  - f9e6595d5634ac4d4221ba6dca9f5c2e64114b3f add size check (#365) by Takuto Ikuta <tikuta@google.com>
  - 3d0cf1be08dd52d77204ad1be7e70fdd1e48c206 Revert "Add useful error message for uploading files. (#3... by Rubens Farias <rubensf@google.com>
  - 3ddc89f3e2b39308101060b3eeaa10a919cae13e "Wrap" gRPC error codes. (#367) by Rubens Farias <rubensf@google.com>
  - d965bf95d0af9d88d90e9723aee6b332dc3ef93e cas: fix deadlock (#368) by Takuto Ikuta <tikuta@google.com>
  - 3c4ce9170b6c5a5d64bcc4feecc5bcdf5dd1f101 allow to use streamBufSize larger than 32KiB (#369) by Takuto Ikuta <tikuta@google.com>
  - 21d6adc44e550f7aa5c4e9b3fedab838920a9632 use semaphore for large file upload (#370) by Takuto Ikuta <tikuta@google.com>
  - 1cec173a5bf76c02f435050d6fc9a02e1ccea637 update remote-apis (#371) by Takuto Ikuta <tikuta@google.com>
  - e96eb06339fb616167ee02535ab54d9c3a382232 add more log around upload (#372) by Takuto Ikuta <tikuta@google.com>
  - 3f34e744d83161ddcb602121b45b0b33185a36c5 Make glog import consistent. (#373) by Rubens Farias <rubensf@google.com>
  - 3db822c86088434a4d2d53ec2d3b889b9f8cf331 Remove typos (#374) by Rubens Farias <rubensf@google.com>

PiperOrigin-RevId: 390319167
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: misc > misc P3 We're not considering working on this, but happy to review a PR. (No assignee) type: bug
Projects
None yet
Development

No branches or pull requests

5 participants