Comment 10 for bug 1061921

Revision history for this message
Martin Pitt (pitti) wrote :

Fixed the syntax error in Python 2.7 in r2517, sorry about that.

"-c --crash-db="gala" --config-dir=/path" is indeed a really unfortunate combination of options. It means that it first configures the config directory to be '--crash-db="gala"', and then overwrites it to "/path...". If you had used a different order, or "--crash-db gala" (without the =), or not specified --config-dir a second time, you would have gotten an appropriate error message.

I'm not sure how to improve that. argparse/optparse parse don't throw a warning if you specify an option twice, and once it's done you don't see the implausible '--crash-db="gala"' config path value any more.

So I think I fixed the two side issues, and can't do much about the option syntax (If you have a proposal how to improve it, I'm glad to hear).

Thanks!