Thursday, June 01, 2017

Re: Update to py-argcomplete-1.8.2

On Tue, 23 May 2017 12:48:42 +0100, Stuart Henderson
<stu@spacehopper.org> wrote:

> On 2017/03/07 20:15, Daniel Jakots wrote:
> > On Tue, 21 Feb 2017 16:13:50 +0000, Stuart Henderson
> > <stu@spacehopper.org> wrote:
> >
> > Sorry for the delay, I slacked a bit.
> >
> > > Tests are a bit messed up though, some missing deps - py-coverage,
> > > py-pexpect, bash, tcsh we have - there's also devel/flake8 of
> > > which we only have a py2 version (and this was imported as just
> > > "flake8" rather than "py-flake8" so adding a py3 flavour is going
> > > to be more fiddly than usual). It also uses "wheel" which isn't
> > > in ports as far as I can see.
> >
> > I talked with flake8 maintainer about splitting it and he said that
> > it was useless. Users only need the 'binary' and the fact that
> > it is made from py2 or py3 changes nothing. Sadly he never had the
> > time to write it on ports@.
> >
> > His point sounds legit so I checked for argcomplete and it seems
> > it's not even used. I just added a patch to zap it from setup.py so
> > we can drop the need to split flake8.
>
> I see where he's coming from, but if you're just working on things
> written using the current version of python, it's annoying to have to
> install a legacy version just to run flake8 (and vice-versa, I
> suppose..)

> > Updated diff, ok danj@ if someone wants to commit
>
> Looks good to me. Though I would also support switching flake8 to py3
> and removing the patch..

You mean adding a py3 flavor, not just moving it to py3 with
MODPY_VERSION = ${MODPY_DEFAULT_VERSION_3}
no?

I can remove the dust over the my diffs to add a py3 flavor to flake8
if it's deemed to be the best way.

Cheers,
Daniel

No comments:

Post a Comment