Movatterモバイル変換


[0]ホーム

URL:


homepage

Issue36106

This issue trackerhas been migrated toGitHub, and is currentlyread-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title:resolve sinpi() name clash with libm
Type:compile errorStage:resolved
Components:Extension ModulesVersions:Python 3.8, Python 3.7, Python 2.7
process
Status:closedResolution:fixed
Dependencies:Superseder:
Assigned To:Nosy List: cheryl.sabella, dimpase, eamanu, koobs, mark.dickinson, miss-islington, ned.deily, serhiy.storchaka
Priority:normalKeywords:patch

Created on2019-02-25 13:47 bydimpase, last changed2022-04-11 14:59 byadmin. This issue is nowclosed.

Pull Requests
URLStatusLinkedEdit
PR 12027mergeddimpase,2019-02-25 13:47
PR 12046mergedmiss-islington,2019-02-26 06:36
PR 12050mergeddimpase,2019-02-26 09:57
Messages (11)
msg336519 -(view)Author: Dmitrii Pasechnik (dimpase)*Date: 2019-02-25 13:47
The standard math library (libm) may follow IEEE-754 recommendation toinclude an implementation of sinPi(), i.e. sinPi(x):=sin(pi*x).And this triggers a name clash, found by FreeBSD developerSteve Kargl, who worken on putting sinpi into libm used on FreeBSD(it has to be named "sinpi", not "sinPi", cf. e.g.https://en.cppreference.com/w/c/experimental/fpext4)
msg336525 -(view)Author: Emmanuel Arias (eamanu)*Date: 2019-02-25 14:53
Hi!, It seem a good improve to resolve a potential problem. But I am not convinced if that problem could appear because sinpi() is definded locally.
msg336528 -(view)Author: Dmitrii Pasechnik (dimpase)*Date: 2019-02-25 15:00
This has a potential header conflict. For the same reason functions likelgamma, etc. are prefixed with `m_` inModules/mathmodule.c
msg336529 -(view)Author: Dmitrii Pasechnik (dimpase)*Date: 2019-02-25 15:05
e.g. if I rename `m_lgamma` to `lgamma` I get```mathmodule.c:389:1: error: static declaration of ‘lgamma’ follows non-static declaration lgamma(double x)```
msg336618 -(view)Author: Serhiy Storchaka (serhiy.storchaka)*(Python committer)Date: 2019-02-26 06:36
New changesetf57cd8288dbe6aba99c057f37ad6d58f8db75350 by Serhiy Storchaka (Dima Pasechnik) in branch 'master':bpo-36106: Resolve sinpi name clash with libm (IEEE-754 violation). (GH-12027)https://github.com/python/cpython/commit/f57cd8288dbe6aba99c057f37ad6d58f8db75350
msg336622 -(view)Author: miss-islington (miss-islington)Date: 2019-02-26 07:10
New changeset4e6646fef5d2cc53422e4eca0b18201ed5a5c4b6 by Miss Islington (bot) in branch '3.7':bpo-36106: Resolve sinpi name clash with libm (IEEE-754 violation). (GH-12027)https://github.com/python/cpython/commit/4e6646fef5d2cc53422e4eca0b18201ed5a5c4b6
msg336633 -(view)Author: Kubilay Kocak (koobs)(Python triager)Date: 2019-02-26 08:59
Downstream issue:https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=232792
msg336641 -(view)Author: Serhiy Storchaka (serhiy.storchaka)*(Python committer)Date: 2019-02-26 10:41
New changesetb545ba0a508a5980ab147ed2641a42be3b31a2db by Serhiy Storchaka (Dima Pasechnik) in branch '2.7':[2.7]bpo-36106: resolve sinpi name clash with libm (IEEE-754 violation). (GH-12027) (GH-12050)https://github.com/python/cpython/commit/b545ba0a508a5980ab147ed2641a42be3b31a2db
msg336692 -(view)Author: Mark Dickinson (mark.dickinson)*(Python committer)Date: 2019-02-26 16:34
This one was my fault. Thanks for the fix!
msg340099 -(view)Author: Dmitrii Pasechnik (dimpase)*Date: 2019-04-12 19:15
Can this also be backported to 3.5. and 3.6, as requested by FreeBSD maintainers?
msg340110 -(view)Author: Cheryl Sabella (cheryl.sabella)*(Python committer)Date: 2019-04-12 20:33
I'll nosy Ned as the release manager about the request to 3.6.
History
DateUserActionArgs
2022-04-11 14:59:11adminsetgithub: 80287
2019-04-12 20:33:52cheryl.sabellasetnosy: +cheryl.sabella,ned.deily
messages: +msg340110
2019-04-12 19:15:40dimpasesetmessages: +msg340099
2019-02-26 16:34:18mark.dickinsonsetmessages: +msg336692
2019-02-26 10:42:10serhiy.storchakasetstatus: open -> closed
resolution: fixed
stage: patch review -> resolved
2019-02-26 10:41:18serhiy.storchakasetmessages: +msg336641
2019-02-26 09:57:40dimpasesetpull_requests: +pull_request12076
2019-02-26 08:59:25koobssetnosy: +koobs

messages: +msg336633
versions: + Python 2.7, Python 3.8
2019-02-26 07:10:58miss-islingtonsetnosy: +miss-islington
messages: +msg336622
2019-02-26 06:36:25miss-islingtonsetkeywords: +patch
pull_requests: +pull_request12071
2019-02-26 06:36:17serhiy.storchakasetnosy: +serhiy.storchaka
messages: +msg336618
2019-02-25 15:05:12dimpasesetmessages: +msg336529
2019-02-25 15:00:24dimpasesetmessages: +msg336528
2019-02-25 14:53:31eamanusetmessages: +msg336525
2019-02-25 14:48:47eamanusetnosy: +eamanu
2019-02-25 13:53:56SilentGhostsetnosy: +mark.dickinson

stage: patch review
2019-02-25 13:47:03dimpasecreate
Supported byThe Python Software Foundation,
Powered byRoundup
Copyright © 1990-2022,Python Software Foundation
Legal Statements

[8]ページ先頭

©2009-2026 Movatter.jp