◐ Shell
reader mode source ↗
Skip to content

bpo-45548: Remove checks for finite and gamma (GH-29206)#29206

Merged
tiran merged 1 commit into
python:mainfrom
tiran:bpo-45548-gamma
Oct 25, 2021
Merged

bpo-45548: Remove checks for finite and gamma (GH-29206)#29206
tiran merged 1 commit into
python:mainfrom
tiran:bpo-45548-gamma

Conversation

@tiran

@tiran tiran commented Oct 25, 2021 •
edited by miss-islington
Loading

Copy link
Copy Markdown
Member

HAVE_FINITE and HAVE_*GAMMA are not used any more. This fixes a build
issue. Configure doesn't detect finite and gamma on macOS.

Signed-off-by: Christian Heimes christian@python.org

https://bugs.python.org/issue45548

Automerge-Triggered-By: GH:tiran

@tiran
bpo-45548: Remove checks for finite and gamma …
745e1b0 
HAVE_FINITE and HAVE_*GAMMA are not used any more. This fixes a build
issue. Configure doesn't detect finite and gamma on macOS.

Signed-off-by: Christian Heimes <christian@python.org>
@tiran tiran requested a review from mdickinson October 25, 2021 09:25
@tiran

tiran commented Oct 25, 2021

Copy link
Copy Markdown
Member Author

@mdickinson ARM64 macOS does not have finite and gamma functions:

https://buildbot.python.org/all/#/builders/721/builds/63/steps/2/logs/stdio

checking for x87-style double rounding... no
checking for acosh... yes
checking for asinh... yes
checking for atanh... yes
checking for erf... yes
checking for erfc... yes
checking for expm1... yes
checking for finite... no
checking for gamma... no
checking for lgamma... yes
checking for log1p... yes
checking for log2... yes
checking for tgamma... yes

@tiran tiran added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Oct 25, 2021
@bedevere-bot

bedevere-bot commented Oct 25, 2021

Copy link
Copy Markdown

🤖 New build scheduled with the buildbot fleet by @tiran for commit 745e1b0 🤖

If you want to schedule another build, you need to add the ":hammer: test-with-buildbots" label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Oct 25, 2021
@mdickinson

mdickinson commented Oct 25, 2021

Copy link
Copy Markdown
Member

ARM64 macOS does not have finite and gamma functions:

Makes sense; I don't think either of those is standard. C99 specifies lgamma and tgamma but not gamma, and an isfinite macro but not finite.

mdickinson
mdickinson approved these changes Oct 25, 2021
View reviewed changes

@mdickinson mdickinson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hide comment

LGTM

@tiran tiran changed the title bpo-45548: Remove checks for finite and gamma Oct 25, 2021
@tiran tiran merged commit 77e3f22 into python:main Oct 25, 2021
@tiran tiran deleted the bpo-45548-gamma branch October 25, 2021 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@mdickinson mdickinson

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@tiran @bedevere-bot @mdickinson @the-knights-who-say-ni