Keith Smiley added the comment:
Someone nonchalantly updated these in
https://github.com/python/cpython/commit/2fc857a5721a5b42bcb696c9cae1bbcc82a91b17
so this bug is now fixed
--
stage: patch review -> resolved
status: open -> closed
___
Keith Smiley added the comment:
Would someone be able to review this change?
--
___
Python tracker
<https://bugs.python.org/issue43220>
___
___
Python-bug
Keith Smiley added the comment:
Thanks for checking, I was able to debug further and it turns out the actual
issue is if you use `uname` from `coreutils`, you get different results:
```
% /opt/homebrew/opt/coreutils/libexec/gnubin/uname -p
arm64
% /usr/bin/uname -p
arm
```
I have this in my
Keith Smiley added the comment:
Yep for sure, this is the first time I've hit a difference with uname
specifically
--
title: ./configure fails on Apple Silicon -> ./configure fails on Apple Silicon
with coreutils uname
___
Python tracker
<
Keith Smiley added the comment:
I think given that this file seems to be updated occasionally anyways we should
still land this. I agree with the sentiment that if this was a super specific
fix just for this edge case maybe it wouldn't be worth
Keith Smiley added the comment:
Thanks for taking a look. My limited understanding is also that these should be
able to be updated separately from autoconf, and I feel slightly more confident
knowing that in the past folks treated this update as trivial. It seems like
the changes should
Change by Keith Smiley :
--
keywords: +patch
pull_requests: +24179
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/25450
___
Python tracker
<https://bugs.python.org/issu
New submission from Keith Smiley :
It seems that Apple Silicon support has been added in
https://github.com/python/cpython/pull/22855, but when I try to build locally I
see this error:
```
% ./configure
checking for git... found
checking build system type... Invalid configuration `arm64
Keith Smiley added the comment:
Here's an example outside of argparse showing this is caused by the `is`
comparison with interned string:
```
import sys
short_string = sys.argv[1]
short_default = '1'
long_string = sys.argv[2]
long_default = 'not-interned'
print(f"short comparisons
New submission from Keith Smiley :
With this code:
```
import argparse
parser = argparse.ArgumentParser()
group = parser.add_mutually_exclusive_group(required=True)
group.add_argument("--foo", default="1")
group.add_argument("--bar")
args = parser.parse_ar
Change by Keith Smiley :
--
keywords: +patch
pull_requests: +23311
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/24526
___
Python tracker
<https://bugs.python.org/issu
11 matches
Mail list logo