Skip to content

Ignore missing __file__#476

Merged
eriknw merged 2 commits into
pytoolz:masterfrom
jayvdb:no-__file__
Jan 14, 2020
Merged

Ignore missing __file__#476
eriknw merged 2 commits into
pytoolz:masterfrom
jayvdb:no-__file__

Conversation

@jayvdb

@jayvdb jayvdb commented Nov 4, 2019

Copy link
Copy Markdown
Contributor

No description provided.

Many Python freezer tools have loaders that either do not set
__file__ or they set __file__ with a guess, or it triggers a
slower load process.
Skip asserts if __file__ is missing.
@eriknw

eriknw commented Jan 14, 2020

Copy link
Copy Markdown
Member

Which module doesn't have __file__? Regardless, this looks fine to me. Thanks @jayvdb!

@eriknw eriknw merged commit afef2c1 into pytoolz:master Jan 14, 2020
@jayvdb

jayvdb commented Jan 14, 2020

Copy link
Copy Markdown
Contributor Author

@eriknw , none have __file__ if toolz is used in a zipapp, or PyOxidizer app, etc.

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.

2 participants