cloud-init-dev team mailing list archive

[Chad Smith <chad.smith@xxxxxxxxxxxxx>]

s/pylint-tip/tip-pylint/ in commit message.

Diff comments:

> diff --git a/setup.py b/setup.py
> index 4616599..a61c24a 100755
> --- a/setup.py
> +++ b/setup.py
> @@ -76,6 +77,9 @@ INITSYS_ROOTS = {
>      'systemd': pkg_config_read('systemd', 'systemdsystemunitdir'),
>      'systemd.generators': pkg_config_read('systemd',
>                                            'systemdsystemgeneratordir'),
> +    'systemd.fsck-dropin': (
> +        os.path.sep.join([pkg_config_read('systemd', 'systemdsystemunitdir'),

Why not just use os.path.join(pkg_config_read('systemd', 'systemdsystemunitdir'),
                          'systemd-fsck@.service.d')?

I just haven't seen os.path.sep used very often.

> +                          'systemd-fsck@.service.d'])),
>      'upstart': '/etc/init/',
>  }
>  INITSYS_TYPES = sorted([f.partition(".")[0] for f in INITSYS_ROOTS.keys()])


-- 
https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/324401
Your team cloud-init commiters is requested to review the proposed merge of ~smoser/cloud-init:pylint/1444-pylint-tip-ignore-e1101-from-contextlib into cloud-init:master.