Page MenuHomeDevCentral

Allow certbot wrapper to work when no argument is provided after the command

Authored by dereckson on Wed, Dec 30, 02:19.



Fixes T1640.

Test Plan

Tested on docker-001:

$ certbot renew
Saving debug log to /var/log/letsencrypt/letsencrypt.log

Diff Detail

rOPS Nasqueron Operations
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

dereckson created this revision.
Another way to solve this according

Quoting this doesn't work, since in the default case, "$debug" would expand to one empty argument while $debug would expand into zero arguments. In this case, you can use an array with zero or one elements as outlined above, or you can use an unquoted expansion with an alternate value:

[[ $1 == "--trace-commands" ]] && debug="yes"
bash ${debug:+"-x"} script

This is better than an unquoted value because the alternative value can be properly quoted, e.g. wget ${output:+ -o "$output"}.

This revision is now accepted and ready to land.Wed, Dec 30, 02:22
This revision was landed with ongoing or failed builds.Wed, Dec 30, 02:23
This revision was automatically updated to reflect the committed changes.