Skip to content
This repository was archived by the owner on May 31, 2020. It is now read-only.

Add lib kwarg to process_events()#43

Open
Holzhaus wants to merge 1 commit intosbarakat:masterfrom
Holzhaus:master
Open

Add lib kwarg to process_events()#43
Holzhaus wants to merge 1 commit intosbarakat:masterfrom
Holzhaus:master

Conversation

@Holzhaus
Copy link

Without this fix, the following exception is thrown:

Traceback (most recent call last):
File "/data/jan/Projects/beets/beets/plugins.py", line 140, in wrapper
    return func(*args, **kwargs)
TypeError: process_events() got an unexpected keyword argument 'lib'

The lib kwarg has been introduced by this commit in beetbox/beets:

commit 2ad5b4c665bd270243d7e31f77680e969d7cb315
Author: Dang Mai <contact@dangmai.net>
Date:   Thu Jan 31 16:33:53 2013 -0500

    Add lib parameter for cli_exit

This is a partial fix for issue #38.

Without this fix, the following exception is thrown:

    Traceback (most recent call last):
    File "/data/jan/Projects/beets/beets/plugins.py", line 140, in wrapper
        return func(*args, **kwargs)
    TypeError: process_events() got an unexpected keyword argument 'lib'

The lib kwarg has been introduced by this commit in beetbox/beets:

    commit 2ad5b4c665bd270243d7e31f77680e969d7cb315
    Author: Dang Mai <contact@dangmai.net>
    Date:   Thu Jan 31 16:33:53 2013 -0500

        Add lib parameter for cli_exit

This is a partial fix for issue sbarakat#38.
@ghost
Copy link

ghost commented Dec 27, 2018

I'm running a large beets import with this branch of beets-copyartifacts installed, and it seems to work well for me!

@sbarakat
Copy link
Owner

sbarakat commented Feb 3, 2019

Thank you so much for the pull request, and sorry my very late response! We recently introduced a baby into the world and I've been caught up with everything he's brought along with him 😁

I'll get this reviewed and merged soon.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants