Resolve symlinks when determining if module is builtin #419
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
To determine whether a module is builtin, Dill compares
mod.__file__
against
sys.base_prefix
. The former will have symlinks resolved, whilethe latter will not. This causes builtin module detection to fail in
Python installations where
sys.base_prefix
involves symlinks, likeHomebrew-managed Python on macOS.
This commit resolves the issue by resolving symlinks in
sys.base_prefix
before comparing module paths against it.Fix #417.