[PATCH 4 of 4] mpatch: implement cffi version of mpatch
Augie Fackler
raf at durin42.com
Thu Jun 23 09:23:14 EDT 2016
> On Jun 23, 2016, at 2:51 AM, Maciej Fijalkowski <fijall at gmail.com> wrote:
>
> ok, the next problem is:
>
> import os
> import cffi
>
> "Imports not lexically sorted"
>
> import cffi
> import os
>
> "Stdlib import follows non-stdlib import"
>
> what do I do?
Smells like a bug in the import checker, where it’s not recognizing cffi as part of the sodlib. I’ll add cffi to the whitelist.
AF
>
> On Wed, Jun 22, 2016 at 5:11 PM, Augie Fackler <raf at durin42.com> wrote:
>>
>>> On Jun 22, 2016, at 3:06 AM, Maciej Fijalkowski <fijall at gmail.com> wrote:
>>>
>>> This seems to be the diff necessary (to Augie's wip branch)
>>>
>>> https://paste.pound-python.org/show/CMz1afIMKS0r7369G3Jr/
>>>
>>> This fails test-check-py3 for stupid reasons.
>>>
>>> Augie: how should I submit that patch, given all the rebasing and stuff?
>>
>> Apply the patch, use histedit to fold it into the old version of 4 of 4, rebase to @, and mail again.
>>
>> (That’s my advice, but I’m only half paying attention this week.)
>>
>>>
>>> Best regards,
>>> Maciej Fijalkowski
>>>
>>> On Tue, Jun 21, 2016 at 1:55 PM, Yuya Nishihara <yuya at tcha.org> wrote:
>>>> On Mon, 20 Jun 2016 17:04:25 +0200, Maciej Fijalkowski wrote:
>>>>> er ok, so there is no good way to do imports that way? At least I
>>>>> didn't find any that would not make the linter barf (using from
>>>>> mercurial). Can we do something that special cases ".." here?
>>>>
>>>>>>>>>> +from .. import modulepolicy, policynocffi
>>>>>>>>>> +
>>>>>>>>>> from . import pycompat
>>>>
>>>> from . import (
>>>> policy,
>>>> pycompat,
>>>> )
>>>>
>>>> I don't know if it works for mercurial/__init__.py, but we won't have to
>>>> care because modulepolicy constants has been moved to policy.py.
>>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: Message signed with OpenPGP using GPGMail
URL: <http://www.mercurial-scm.org/pipermail/mercurial-devel/attachments/20160623/8e1ee946/attachment.sig>
More information about the Mercurial-devel
mailing list