Skip to content

Conversation

@itamarst
Copy link
Owner

Fixes #386.

@itamarst
Copy link
Owner Author

@crwood could you try this?

@itamarst itamarst merged commit 760f885 into master Mar 20, 2019
@itamarst itamarst deleted the 386.pyinstaller branch March 20, 2019 13:36
@crwood
Copy link

crwood commented Mar 21, 2019

@itamarst My apologies for missing this earlier..

If it matters, I have since tested a non-trivial (python2) application running under a frozen/PyInstaller interpreter on Linux, macOS, and Windows and can confirm that, in all three cases, with these changes in place, eliot logging is now working as expected.

Thank you very much for this -- and for eliot more generally! I'm greatly looking forward to using it in future projects. :)

@itamarst
Copy link
Owner Author

No problem, Jean-Paul conveyed that it worked. Thanks for the further checks!

tomprince pushed a commit to tp-la/eliot that referenced this pull request Jul 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants