"bacon test -- -p some_package" doesn't work about bacon HOT 6 CLOSED

Hi, I also had this issue when specifying --bin option for test. I have an alternate solution that does not require bacon.toml file. The solution is to use an extra -- when specifying options to cargo instead of bacon.

In this case @Hubro would have change only one thing about the command, This would also keep the fix provided by #124 intact.

bacon test -- -- -p my_sub_package

from bacon.

It works for me.

Can you please give your configuration and/or build a complete reproductible example ?

from bacon.

It works for me.

Can you please give your configuration and/or build a complete reproductible example ?

Sure, here's a minimal example:

https://github.com/Hubro/bacon-issue-134-demo

To reproduce:

1. cargo test

   Fails, because there's a syntax error in crate dos

2. cargo test -p uno

   Succeeds, because only crate uno is built

3. bacon test -- -p uno

   Fails with syntax error from crate dos, just like the command cargo test. It's as if the option -p uno wasn't forwarded at all.

$ bacon --version
bacon 2.8.1

from bacon.

Ok, it's a consequence of #124

Here's the current "test" job definition:

[jobs.test]
command = [
    "cargo", "test", "--color", "always",
    "--", "--color", "always", # see https://github.com/Canop/bacon/issues/124
]
need_stdout = true

The addition made for #124 was unfortunate here.

As a workaround, you can change this job definition to

[jobs.test]
command = [
    "cargo", "test", "--color", "always",
]
need_stdout = true

To do that, run bacon --init then edit the bacon.toml file.

(don't close the issue)

from bacon.

Right now I'm tempted to solve the problem with just some comment in the bacon.toml file, and maybe the website

from bacon.

Hi, I also had this issue when specifying --bin option for test. I have an alternate solution that does not require bacon.toml file. The solution is to use an extra -- when specifying options to cargo instead of bacon.

In this case @Hubro would have change only one thing about the command, This would also keep the fix provided by #124 intact.

bacon test -- -- -p my_sub_package

I think this work around may not work anymore. tested on bacon 2.13.0. had to bacon --init and remove the "--", "--color", "always", # see https://github.com/Canop/bacon/issues/124 line

from bacon.