Allow passing arguments after `--` to the job command about bacon HOT 10 CLOSED

Examples here: https://dystroy.org/bacon/cookbook/#variable-arguments

from bacon.

It seems to work.

from bacon.

As noticed by @jyn514 , the current argh doesn't make that easy, so args parsing would have to be changed.

from bacon.

TODO: check if clap brand new version 3 handle it. I believe 2 already handle this.

from bacon.

If you find out how it's done in clap3, I'm interested.

from bacon.

Look perfect https://github.com/clap-rs/clap/blob/master/examples/escaped-positional-derive.md

code: https://github.com/clap-rs/clap/blob/master/examples/escaped-positional-derive.rs

from bacon.

This "escaped positional derive" example looks good enough. I'll check ASAP. I hope the bacon executable won't be twice as big from the clap crate.

from bacon.

I hope the bacon executable won't be twice as big from the clap crate.

If you are talking about this from argh

Derive-based argument parsing optimized for code size

I believe it's about code size not binary size. Mainly what they mean is "we offer derive feature". clap 3 is mainly about having derive feature avoiding code size that was needed when using clap 2.

Anyway rust bin are pretty big and I don't expect clap to change the size much.

from bacon.

@Stargateur just using clap3 instead of argh makes bacon go from 4.8M to 5.2M. It's probably acceptable for bacon but I've dropped clap from smaller applications in the past as it made them double in size

from bacon.

A question regarding this feature: should those additional arguments be also added to other jobs launched with shortcuts ? Answer doesn't seem obvious and it may limit the usability of this change. I'll start with a yes.

from bacon.