Skip to content

Fix pickles() flagging __builtins__ as unpickleable due to the all() function#538

Merged
mmckerns merged 1 commit intouqfoundation:masterfrom
leogama:patch-1
Aug 10, 2022
Merged

Fix pickles() flagging __builtins__ as unpickleable due to the all() function#538
mmckerns merged 1 commit intouqfoundation:masterfrom
leogama:patch-1

Conversation

@leogama
Copy link
Contributor

@leogama leogama commented Aug 9, 2022

Currently:

>>> import dill
>>> dill.pickles(__builtins__)  # __builtins__.all() raises a TypeError for a missing argument
False
>>> dill.copy(__builtins__)
<module 'builtins' (built-in)>
>>> _ is __builtins__
True

Expected:

>>> import dill
>>> dill.pickles(__builtins__, exact=True)
True

…ll()` function

Currently:
```python
>>> import dill
>>> dill.pickles(__builtins__)  # __builtins__.all() raises a TypeError for a missing argument
False
>>> dill.copy(__builtins__)
<module 'builtins' (built-in)>
>>> _ is __builtins__
True
```

Expected:
```python
>>> import dill
>>> dill.pickles(__builtins__, exact=True)
True
```
@mmckerns mmckerns added this to the dill-0.3.6 milestone Aug 10, 2022
@mmckerns mmckerns merged commit 7d17338 into uqfoundation:master Aug 10, 2022
@leogama leogama deleted the patch-1 branch August 13, 2022 21:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants