cloud-init-dev team mailing list archive

[Scott Moser <smoser@xxxxxxxxxx>]

I have some questions inline below. I think most would fall or in actually trying to build an tomorrow in each distro/release

Diff comments:

> diff --git a/packages/pkg-deps.json b/packages/pkg-deps.json
> new file mode 100644
> index 0000000..2100ec7
> --- /dev/null
> +++ b/packages/pkg-deps.json
> @@ -0,0 +1,84 @@
> +{
> +   "debian" : {
> +      "build-requires" : [
> +         "debhelper",
> +         "dh-python",
> +         "dh-systemd"
> +      ],
> +      "renames" : {
> +         "pep8" : {
> +            "2" : "pep8",

I think that in xenial and further this can be standard naming
https://launchpad.net/ubuntu/+source/pep8

> +            "3" : "python3-pep8"
> +         },
> +         "pyflakes" : {

I'm pretty sure this is not exactly correct everywhere.
In xenial and forward the are pyflakes does have standard naming.

https://launchpad.net/ubuntu/+source/pyflakes

Installing pyflakes directly though I think gets you a mixmatch.
I think it might be best to just make this standard name.

> +            "2" : "pyflakes",
> +            "3" : "pyflakes"
> +         },
> +         "pyyaml" : {
> +            "2" : "python-yaml",
> +            "3" : "python3-yaml"
> +         },
> +         "six" : {
> +            "2" : "python-six",

Isn't this standard name?

> +            "3" : "python3-six"
> +         },
> +         "contextlib2" : {
> +            "2" : "python-contextlib2",
> +            "3" : null
> +         },
> +         "pyserial" : {
> +            "2" : "python-serial",
> +            "3" : "python3-serial"
> +         }
> +      },
> +      "requires" : [
> +         "procps"
> +      ]
> +   },
> +   "redhat" : {
> +      "build-requires" : [
> +         "python-devel",
> +         "python-setuptools"
> +      ],
> +      "renames" : {
> +         "pyyaml" : {
> +            "2" : "PyYAML",

What does the null man here? (Below)
Does it mean standard or none?

> +            "3" : null
> +         },
> +         "pyserial" : {
> +            "2" : "pyserial",
> +            "3" : null
> +         }

This doesn't seem right to me.

> +      },
> +      "requires" : [
> +         "e2fsprogs",
> +         "iproute",
> +         "net-tools",
> +         "procps",
> +         "rsyslog",
> +         "shadow-utils",
> +         "sudo >= 1.7.2p2-3"
> +      ]
> +   },
> +   "suse" : {
> +      "renames" : {
> +         "pyyaml" : {
> +            "2" : "python-yaml",
> +            "3" : null
> +         }
> +      },
> +      "build-requires" : [
> +         "fdupes",
> +         "filesystem",
> +         "python-devel",
> +         "python-setuptools"
> +      ],
> +      "requires" : [
> +         "iproute2",
> +         "e2fsprogs",
> +         "net-tools",
> +         "procps",
> +         "sudo"
> +      ]
> +   }
> +}


-- 
https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/325342
Your team cloud-init commiters is requested to review the proposed merge of ~chad.smith/cloud-init:ci-deps into cloud-init:master.