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
let platform.uname try harder #47161
Comments
Sometimes os.uname (which is used to bootstrap platform.uname) can |
I can work on this task. |
i'm looking at the source and there doesn't appear to be a function |
much handling code already seems to exist under the line |
Thanks for volunteering, James! I think you misunderstood the task. Do |
Alright, that makes things much clearer. So essentially what you want me to do is add code that tries to replace |
Ah, ok, the code under except AttributeError: gives me some good ideas. |
Right on! You don't need to write any new code, just make sure the code |
Here is the patch (apply to platform.py) |
Thanks for doing this. Would you please try on Windows the patch, I'm |
Your patch works perfectly on windows. Thanks for your help. |
Marc, could you look at this please? |
There are two patches. Which one do you want me to look at ? Note that platform.py should stay Python 1.5.2 compatible, ie. no new The second patch also appears to mix tabs/spaces. |
Ok. I ran it through reindent.py and removed the True and False. |
On 2008-06-13 16:20, Benjamin Peterson wrote:
Thanks, but the all() is still there :-) You can change that to: ... or not filter(None, [system, node, release, version, machine]): should work on all Python versions. |
Wow, I really admire you for keeping it compatible with Python 1.5. As |
On 2008-06-13 16:59, Benjamin Peterson wrote:
Yeah, well... it's a hobby :-) We can probably bump that to Python 2.1 or even 2.3 after 2.6 The main reason for keeping 1.5.2 compatibility was mxCGIPython The patches look OK now. Thanks,Marc-Andre Lemburg Professional Python Services directly from the Source (#1, Jun 13 2008)
>>> Python/Zope Consulting and Support ... http://www.egenix.com/
>>> mxODBC.Zope.Database.Adapter ... http://zope.egenix.com/
>>> mxODBC, mxDateTime, mxTextTools ... http://python.egenix.com/
________________________________________________________________________
2008-07-07: EuroPython 2008, Vilnius, Lithuania 23 days to go :::: Try mxODBC.Zope.DA for Windows,Linux,Solaris,MacOSX for free ! ::::
|
Ok. I applied the patch in r64233. |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: