Mergifyio / daiquiri

Python library to easily setup basic logging functionality
Apache License 2.0
333 stars 25 forks source link

Fix wrong constructor call #20

Closed fridex closed 7 years ago

fridex commented 7 years ago

Fixes:

Traceback (most recent call last):
  File "./main.py", line 11, in <module>
    import daiquiri
  File "/usr/lib/python3.6/site-packages/daiquiri/__init__.py", line 19, in <module>
    from daiquiri import output
  File "/usr/lib/python3.6/site-packages/daiquiri/output.py", line 232, in <module>
    preconfigured['journal'] = Journal()
  File "/usr/lib/python3.6/site-packages/daiquiri/output.py", line 180, in __init__
    super(Journal, self).__init__(handlers.JournalHandler(program_name),
  File "/usr/lib/python3.6/site-packages/daiquiri/handlers.py", line 65, in __init__
    super(SyslogHandler, self).__init__()
TypeError: super(type, obj): obj must be an instance or subtype of type
fridex commented 7 years ago

@jd could you please release a new version with this fix? We are experiencing failures because of this, it would help us to remove workarounds that replace packaged version with the current master. Thanks a lot!

jd commented 7 years ago

@fridex 1.2.1 and later have the fix.

1.2.1 was relesed on 19th July 2017. :)

fridex commented 7 years ago

1.2.1 was relesed on 19th July 2017. :)

Ah, missed that. Thanks! :)