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).
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!