threads / discuss / 36784

git-multimail: migration: Config is not iterable

Subject: git-multimail: migration: Config is not iterable

## tl;dr

4 messages between May 29, 2014 and May 31, 2014.

replies: 3people: 3as markdown or json

Azat Khuzhin· May 29, 2014, 14:22 UTC · lore

Hi there,

Using the latest version of git-multimail there is an issue with migration:

$ ~azat/git-multimail/git-multimail/migrate-mailhook-config --overwrite
Traceback (most recent call last):
  File "/home/azat/git-multimail/git-multimail/migrate-mailhook-config", line 271, in <module>
    main(sys.argv[1:])
  File "/home/azat/git-multimail/git-multimail/migrate-mailhook-config", line 268, in main
    migrate_config(strict=options.strict, retain=options.retain, overwrite=options.overwrite)
  File "/home/azat/git-multimail/git-multimail/migrate-mailhook-config", line 159, in migrate_config
    if not _check_old_config_exists(old):
  File "/home/azat/git-multimail/git-multimail/migrate-mailhook-config", line 66, in _check_old_config_exists
    if name in old:
TypeError: argument of type 'Config' is not iterable

Tested on 2.6 and 2.7 python versions.

If you revert 09d0d5b92203f019763e43cef1e57f76f117d2b4 ("Get Python files to pass pep8's tests.") there issue goes away. I understand that this is not the right solution and I'm not the guru of python, so just let you know.

Thanks

Elijah Newren· May 29, 2014, 16:06 UTC · re: Azat Khuzhin · lore

Re: git-multimail: migration: Config is not iterable

On Thu, May 29, 2014 at 7:22 AM, Azat Khuzhin <a3at.mail@gmail.com> wrote:

Show 7 quoted lines

> Hi there,
>
> Using the latest version of git-multimail there is an issue with
> migration:
>
> $ ~azat/git-multimail/git-multimail/migrate-mailhook-config --overwrite
> Traceback (most recent call last):

...

>   File "/home/azat/git-multimail/git-multimail/migrate-mailhook-config", line 66, in _check_old_config_exists
>     if name in old:
> TypeError: argument of type 'Config' is not iterable

Thanks for the report. I'm not the strongest on python data model and its special methods but I believe I've got a fix: https://github.com/mhagger/git-multimail/pull/58

Michael Haggerty· May 31, 2014, 14:59 UTC · re: Azat Khuzhin · lore

Re: git-multimail: migration: Config is not iterable

On 05/29/2014 04:22 PM, Azat Khuzhin wrote:

Show 20 quoted lines

> Using the latest version of git-multimail there is an issue with
> migration:
> 
> $ ~azat/git-multimail/git-multimail/migrate-mailhook-config --overwrite
> Traceback (most recent call last):
>   File "/home/azat/git-multimail/git-multimail/migrate-mailhook-config", line 271, in <module>
>     main(sys.argv[1:])
>   File "/home/azat/git-multimail/git-multimail/migrate-mailhook-config", line 268, in main
>     migrate_config(strict=options.strict, retain=options.retain, overwrite=options.overwrite)
>   File "/home/azat/git-multimail/git-multimail/migrate-mailhook-config", line 159, in migrate_config
>     if not _check_old_config_exists(old):
>   File "/home/azat/git-multimail/git-multimail/migrate-mailhook-config", line 66, in _check_old_config_exists
>     if name in old:
> TypeError: argument of type 'Config' is not iterable
> 
> Tested on 2.6 and 2.7 python versions.
> 
> If you revert 09d0d5b92203f019763e43cef1e57f76f117d2b4 ("Get Python files to
> pass pep8's tests.") there issue goes away. I understand that this is not the
> right solution and I'm not the guru of python, so just let you know.

Thanks for the bug report and for narrowing it down to the broken commit. I just pushed a fix to GitHub. Let me know if it works for you now.

Elijah: your fix is also correct, but I didn't see it before I pushed my
own solution.  Sorry about that.

Junio: you don't have to worry about any of this, because the commit
that caused the breakage is not in your tree yet.

Michael

-- 
Michael Haggerty
mhagger@alum.mit.edu

Azat Khuzhin· May 31, 2014, 15:21 UTC · re: Michael Haggerty · lore

Re: git-multimail: migration: Config is not iterable

On Sat, May 31, 2014 at 04:59:45PM +0200, Michael Haggerty wrote:

Show 25 quoted lines

> On 05/29/2014 04:22 PM, Azat Khuzhin wrote:
> > Using the latest version of git-multimail there is an issue with
> > migration:
> > 
> > $ ~azat/git-multimail/git-multimail/migrate-mailhook-config --overwrite
> > Traceback (most recent call last):
> >   File "/home/azat/git-multimail/git-multimail/migrate-mailhook-config", line 271, in <module>
> >     main(sys.argv[1:])
> >   File "/home/azat/git-multimail/git-multimail/migrate-mailhook-config", line 268, in main
> >     migrate_config(strict=options.strict, retain=options.retain, overwrite=options.overwrite)
> >   File "/home/azat/git-multimail/git-multimail/migrate-mailhook-config", line 159, in migrate_config
> >     if not _check_old_config_exists(old):
> >   File "/home/azat/git-multimail/git-multimail/migrate-mailhook-config", line 66, in _check_old_config_exists
> >     if name in old:
> > TypeError: argument of type 'Config' is not iterable
> > 
> > Tested on 2.6 and 2.7 python versions.
> > 
> > If you revert 09d0d5b92203f019763e43cef1e57f76f117d2b4 ("Get Python files to
> > pass pep8's tests.") there issue goes away. I understand that this is not the
> > right solution and I'm not the guru of python, so just let you know.
> 
> Thanks for the bug report and for narrowing it down to the broken
> commit.  I just pushed a fix to GitHub.  Let me know if it works for you
> now.

Yeah, it works, thanks!

-- 
Respectfully
Azat Khuzhin