[PATCH] Fix build on RHEL6/Centos6, avoid using python str.format()
Douglas Bagnall
douglas.bagnall at catalyst.net.nz
Thu Sep 20 01:06:09 UTC 2018
hi Rowland,
On 19/09/18 21:00, Rowland Penny via samba-technical wrote:
>
> Whilst this will undoubtedly fix the build, it will not help with
> samba-tool, which is now littered with '[]'
>
> See attached file
>
> grep -rnw 'git/samba-master/python/samba/' -e "{}"
>
> git/samba-master/python/samba/tests/dns_base.py:213: self.settings = {}
> git/samba-master/python/samba/tests/source.py:167: tabs = {}
> git/samba-master/python/samba/tests/source.py:178: illegal_newlines = {}
> git/samba-master/python/samba/tests/source.py:188: trailing_whitespace = {}
> git/samba-master/python/samba/tests/source.py:198: files_with_shebang = {}
> git/samba-master/python/samba/tests/source.py:199: files_without_shebang = {}
> git/samba-master/python/samba/tests/kcc/kcc_utils.py:49: self.sites = {}
> git/samba-master/python/samba/tests/kcc/kcc_utils.py:50: self.site_links = {}
> git/samba-master/python/samba/tests/kcc/kcc_utils.py:72: dn = "CN={},CN=Servers,{}".format(name, site)
> git/samba-master/python/samba/tests/kcc/kcc_utils.py:81: dn = "CN={},CN=Sites,{}".format(
Most of those are not format strings (this would get you closer:
git grep -F '{}' :/*.py | grep format) but yes, there are quite a few
nonetheless.
Is nobody using 4.9 on RHEL6?
Douglas
More information about the samba-technical
mailing list