Skip to content

Restore compatibility with portalocker < 1.4.0 - #50

Merged
Ray Luo (rayluo) merged 1 commit into
AzureAD:devfrom
bluca:portalocker_1.4.0
Apr 25, 2020
Merged

Restore compatibility with portalocker < 1.4.0#50
Ray Luo (rayluo) merged 1 commit into
AzureAD:devfrom
bluca:portalocker_1.4.0

Conversation

@bluca

Copy link
Copy Markdown
Contributor

Support for passing parameters through to the open syscall was added in
version 1.4.0. Check before using this feature to keep backward compatibility.

wolph/portalocker@721b3309c2

The dependency would cut off the current Debian Stable and Ubuntu before 19.10

@bluca

Copy link
Copy Markdown
Contributor Author

Do you intend to keep support for Python 2 even though it's EOL? If so let me know and I'll rework this

Comment thread msal_extensions/cache_lock.py Outdated
Comment thread msal_extensions/cache_lock.py
@rayluo

Copy link
Copy Markdown
Contributor

Luca Boccassi (@bluca) If at all possible, please prioritize this fix. We found and fixed a separated bug in 0.2.0+ series, and we would like to have a new bugfix release 0.2.2 first thing next week. Ideally your improvement here can also go in in that release.

Support for passing parameters through to the open syscall was added in
version 1.4.0. Check before using this feature to keep backward compatibility.

wolph/portalocker@721b3309c2

@rayluo Ray Luo (rayluo) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Luca! We will cut a new release soon!

Comment thread msal_extensions/cache_lock.py
@rayluo
Ray Luo (rayluo) merged commit 976e755 into AzureAD:dev Apr 25, 2020
@bluca
Luca Boccassi (bluca) deleted the portalocker_1.4.0 branch April 25, 2020 17:15
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