Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

replace linux_distribution function #8

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

yearski
Copy link

@yearski yearski commented Dec 31, 2019

linux_distribution() has been removed from the platform library. This pull request uses platform.release() for the user agent string for linux clients.

#7 is also a viable alternative, but requiring a 3rd party package seems overkill just to generate a user agent string but using the release() will not be consistent with historical data, @rdegges decision if that is significant.

@fchorney
Copy link

Without this fix, this package doesn't work on python 3.8. Would it be possible to get one of these PRs merged in and a new release any time soon? @rdegges

Thanks!

@col-panic
Copy link

Just stumbled upon this - interesting that this is still open :)

@yearski
Copy link
Author

yearski commented Nov 18, 2021

Just stumbled upon this - interesting that this is still open :)

This repo is no longer maintened per comment on other pull request #11 (comment).

@SigireddyBalasai
Copy link

i am creating new module with syncronous asyncronous and cli named PyIpify

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.

None yet

4 participants