launchpad-reviewers team mailing list archive

[Łukasz Zemczak <lukasz.zemczak@xxxxxxxxxxxxx>]

Thanks Dave! Actually, hm, I might consider finding a new name, since I now see that that the build_id name is used inside launchpad-buildd a lot! To avoid confusion, for now I think I'll prefix it somehow.

Diff comments:

> 
> === modified file 'lpbuildd/target/build_livefs.py'
> --- lpbuildd/target/build_livefs.py	2019-12-16 19:23:47 +0000
> +++ lpbuildd/target/build_livefs.py	2020-01-15 11:57:20 +0000
> @@ -76,6 +76,9 @@
>              "--channel", metavar="CHANNEL",
>              help="pull snaps from channel CHANNEL for ubuntu-core image")
>          parser.add_argument(
> +            "--build_id", metavar="BUILD_ID",

Ah indeed! Yes, looks and feels much better this way. Let me change it, thanks!

> +            help="optional build ID to be passed to the image build")
> +        parser.add_argument(
>              "--http-proxy", action="store", help="use this HTTP proxy for apt")
>          parser.add_argument(
>              "--debug", default=False, action="store_true",


-- 
https://code.launchpad.net/~sil2100/launchpad-buildd/pass_build_id/+merge/377635
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~sil2100/launchpad-buildd/pass_build_id into lp:launchpad-buildd.