savoirfairelinux / cqfd

cqfd helps running commands inside the Docker container configured for your project, keeping the user and working directory the same inside the container
GNU General Public License v3.0
65 stars 31 forks source link

Incoherent options doesn't trigger any warnings #91

Closed eroussy closed 1 year ago

eroussy commented 1 year ago

The command cqfd -b flavor1 -b flavor2 launch only flavor2 and doesn't trigger a warning

joufella commented 1 year ago

The behavior is consistent with most unix tools which take an option/argument couple. For example ls -l --block-size=512 --block-size=1024 will not raise any warning and the last specified wins.

We could make the option cumulative (as in docker-run -e foo=x -e bar=y), in this case this would be a good thing to mention in the doc.

joufella commented 1 year ago

As a consequence, I'd close this issue as a non-issue, unless we're breaking some spec here.

eroussy commented 1 year ago

I'm ok with that.