ctdb/utils/etcd/ctdb_etcd_lock python => python3
L.P.H. van Belle
belle at bazuin.nl
Fri Apr 23 12:32:12 UTC 2021
Hai Martin,
> How exactly does the build fail?
Well, i use cowbuild/pbuilder in Debian 10 and in the end lintian checks the packages and errors with.
E: ctdb: python-script-but-no-python-dep usr/lib/x86_64-linux-gnu/ctdb/ctdb_etcd_lock #!python3
Even after i added in debian/control
${python3:Depends} for CTDB package
>
> > I dont know if its really needed but all python references are used
> > with /usr/bin/env python3.
> >
> > Just to besure im not breaking something in ctdb here.
> > Can someone confirm this?
> >
> > My fix.
> >
> > --- a/ctdb/utils/etcd/ctdb_etcd_lock
> > +++ b/ctdb/utils/etcd/ctdb_etcd_lock
> > @@ -1,4 +1,4 @@
> > -#!/usr/bin/python
> > +#!/usr/bin/env python3
>
> I'd like to be able to give a better answer but I'm not entirely sure
> that the script works with python3 without any other changes.
> It looks
> like it should but I don't have a trivial way of testing it
> (or time to setup a test environment).
>
> Given that /usr/bin/python isn't supported anymore, we might as well
> change it as above see what bug reports get generated.
>
> If you're keen to improve the script then you could try to clean it up
> and get rid of warnings from pylint and flake8. getting rid of the
> obvious whitespace issues might make any python3 issues more obvious.
> Perhaps yapf3 would be useful?
I really wished i had these skills todo that, i would then..
:-(
>
> peace & happiness,
> martin
>
Thanks for the reply so far.
Greetz,
Louis
More information about the samba-technical
mailing list