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

Impossible to utilize play_routes rules in remote builds #38

Closed
vmax opened this issue Jul 14, 2020 · 1 comment · Fixed by #39
Closed

Impossible to utilize play_routes rules in remote builds #38

vmax opened this issue Jul 14, 2020 · 1 comment · Fixed by #39
Assignees

Comments

@vmax
Copy link
Contributor

vmax commented Jul 14, 2020

Due to bazelbuild/bazel#6393 and the fact that implementation uses ctx.actions.declare_directory, contents of which are not transferred to remote machine, it's impossible to build play_routes targets remotely. Attempting to do so results in:

INFO: From Bundling compiled play routes into srcjar:
find: ‘bazel-out/k8-fastbuild/bin/server/play/routes/server_routes_file’: No such file or directory
ERROR: /home/circleci/grabl/server/BUILD:39:14: scala //server:server-routes-lib failed (Exit 1)
ChUI4/gCEOnPAhjsrPUCIKMuYJcFaAESFAgnELnJgSIY3Y4sIAwoklAwgOAD
java.lang.RuntimeException: Must have input files from either source jars or local files.
	at io.bazel.rulesscala.scalac.ScalacProcessor.processRequest(ScalacProcessor.java:68)
	at io.bazel.rulesscala.worker.GenericWorker.run(GenericWorker.java:114)
	at io.bazel.rulesscala.scalac.ScalaCInvoker.main(ScalaCInvoker.java:41)

(that's with //server:server-routes-lib (scala_library) depending on //server:server-routes-lib (play_routes))

@vmax vmax changed the title Impossible to utilize rule for remote builds Impossible to utilize play_routes rules in remote builds Jul 14, 2020
@SrodriguezO
Copy link
Contributor

Thanks for bringing this to our attention and for including a PR!

I'm assigning this to @jjudd since he's more familiar with the original logic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants