a nitpick comment:

Diff comments:

> diff --git a/security-tools/unembargo b/security-tools/unembargo
> index a78c826..c178015 100755
> --- a/security-tools/unembargo
> +++ b/security-tools/unembargo
> @@ -92,7 +99,7 @@ if len(args) < 1:
>  # If --esm, --esm-apps or --esm-infra option is provided, the script will 
> move

I think we can also add '--esm-infra-legacy' to the comments

>  # from the respective ESM staging PPA to the respective ESM PROD PPA. So the
>  # user should not indicate such values.
> -if opt.esm or opt.esm_apps or opt.esm_infra:
> +if opt.esm or opt.esm_apps or opt.esm_infra or opt.esm_infra_legacy:
>      if opt.destination != UBUNTU_ARCHIVE or opt.ppa is not None:
>          parser.error("If an esm option is provided, do not specify values 
> for "
>                       "package origin and destination as this "


-- 
https://code.launchpad.net/~hlibk/ubuntu-qa-tools/+git/ubuntu-qa-tools/+merge/477362
Your team Ubuntu Bug Control is requested to review the proposed merge of 
~hlibk/ubuntu-qa-tools:add-legacy into ubuntu-qa-tools:master.


_______________________________________________
Mailing list: https://launchpad.net/~ubuntu-bugcontrol
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~ubuntu-bugcontrol
More help   : https://help.launchpad.net/ListHelp

Reply via email to