-
Notifications
You must be signed in to change notification settings - Fork 111
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
ibazel does not pass arguments to Bazel #13
Comments
@endobson Sorry for the really slow response. The presumption here is that you would like all blaze test args to be sent along to later commands but that we would need to filter the args list to only take things that are shaped like a target for query. That seems possible. Especially given the refactor I did recently. Is this a thing you would be interested in taking on? |
Relatedly: ibazel cannot pass any arguments to bazel. I spent some time trying to understand why I'm including this here because it's related. The issue could maybe be renamed to support passing arguments to bazel. |
It should be renamed as long as we somehow make sure streamed mode can be
turned off
…On Wed, 2 Aug 2017 at 23:24 Kamal Marhubi ***@***.***> wrote:
Relatedly: ibazel cannot pass *any* arguments to bazel. I spent some time
trying to understand why ibazel run //path/to:some_binary -- --an-arg
wasn't working.
I'm including this here because it's related. The issue could maybe be
renamed to support passing arguments to bazel.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#13 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABUIF9sVyRVQO9jKmgt9L5zyDa250duYks5sUNrkgaJpZM4NsarP>
.
|
Now when you pass in arguments to ibazel it should pass them through to bazel or to your running job if you pass it after a `--`. Fixes #13. Also this no longer sets --test_output=streamed by default.
This does not seem work as expected. ibazel test //path/to:target -- --test_output=all This would not make the test to produce all output as the following does: bazel test //path/to:target --test_output all |
When you put the extra
and tell me if that works? Also, in the future it would be nice if you could open a new issue instead of commenting on a closed one. |
I want to run my tests in parallel and ibazel adds
--test_output=streamed
to the args when runningbazel test
, which is unexpected and undocumented. I would expect it to just run exactly the same had the command line beenbazel test
.The text was updated successfully, but these errors were encountered: