Skip to content

gh-99086: Fix implicit int warning in configure check for PTHREAD_SCOPE_SYSTEM (#99085) - #99085

Merged
erlend-aasland merged 3 commits into
python:mainfrom
thesamesam:clang-16
Nov 5, 2022
Merged

gh-99086: Fix implicit int warning in configure check for PTHREAD_SCOPE_SYSTEM (#99085)#99085
erlend-aasland merged 3 commits into
python:mainfrom
thesamesam:clang-16

Conversation

@thesamesam

@thesamesamthesamesam commented Nov 4, 2022•
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Contributor

Clang 16 makes -Wimplicit-int fatal by default.

Avoids errors like:

conftest.c:142:7: error: type specifier missing, defaults to 'int'; ISO C99 and later do not support implicit int [-Wimplicit-int]

Signed-off-by: Sam James sam@gentoo.org

@bedevere-bot

bedevere-bot commented Nov 4, 2022

Copy link
Copy Markdown

Most changes to Python require a NEWS entry.

Please add it using the blurb_it web app or the blurb command-line tool.

@ghost

ghost commented Nov 4, 2022•
edited by ghost
Loading

Copy link
Copy Markdown

All commit authors signed the Contributor License Agreement.
CLA signed

thesamesam added a commit to thesamesam/cpython that referenced this pull request Nov 4, 2022
@thesamesam
pythongh-99086: Build: Fix -Wimplicit-int in PTHREAD_SCOPE_SYSTEM che…
6d9431f
…ck (python#99085)
Clang 16 makes -Wimplicit-int fatal by default.
Avoids errors like:
```
conftest.c:142:7: error: type specifier missing, defaults to 'int'; ISO C99 and later do not support implicit int [-Wimplicit-int]
```
Signed-off-by: Sam James <sam@gentoo.org>
@thesamesamthesamesam changed the title configure.ac: Fix -Wimplicit-int in PTHREAD_SCOPE_SYSTEM checkh-99086: Build: Fix -Wimplicit-int in PTHREAD_SCOPE_SYSTEM check (#99085)Nov 4, 2022
@thesamesamthesamesam changed the title h-99086: Build: Fix -Wimplicit-int in PTHREAD_SCOPE_SYSTEM check (#99085)gh-99086: Build: Fix -Wimplicit-int in PTHREAD_SCOPE_SYSTEM check (#99085)Nov 4, 2022
@bedevere-botbedevere-bot mentioned this pull request Nov 4, 2022
Closed
thesamesam added a commit to thesamesam/cpython that referenced this pull request Nov 4, 2022
@thesamesam
pythongh-99086: Build: Fix -Wimplicit-int in PTHREAD_SCOPE_SYSTEM che…
70b81e2
…ck (python#99085)
Clang 16 makes -Wimplicit-int fatal by default.
Avoids errors like:
```
conftest.c:142:7: error: type specifier missing, defaults to 'int'; ISO C99 and later do not support implicit int [-Wimplicit-int]
```
Signed-off-by: Sam James <sam@gentoo.org>
@thesamesam
pythongh-99086: Build: Fix -Wimplicit-int in PTHREAD_SCOPE_SYSTEM che…
b89a0f7
…ck (python#99085)
Clang 16 makes -Wimplicit-int fatal by default.
Avoids errors like:
```
conftest.c:142:7: error: type specifier missing, defaults to 'int'; ISO C99 and later do not support implicit int [-Wimplicit-int]
```
Signed-off-by: Sam James <sam@gentoo.org>
kumaraditya303
kumaraditya303 approved these changes Nov 4, 2022
View reviewed changes

@kumaraditya303kumaraditya303 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@kumaraditya303kumaraditya303 added type-bug An unexpected behavior, bug, or error needs backport to 3.10 only security fixes needs backport to 3.11 only security fixes labels Nov 4, 2022
erlend-aasland
erlend-aasland approved these changes Nov 4, 2022
View reviewed changes

@erlend-aaslanderlend-aasland left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

One small nit regarding the NEWS entry. The AC fix is fine with me, but IMO using AC_LANG_PROGRAM iso. AC_LANG_SOURCE would have been better.

Comment threadMisc/NEWS.d/next/Build/2022-11-04-02-58-10.gh-issue-99086.DV_4Br.rst Outdated
@erlend-aaslanderlend-aasland self-assigned this Nov 4, 2022
@erlend-aaslanderlend-aasland changed the title gh-99086: Build: Fix -Wimplicit-int in PTHREAD_SCOPE_SYSTEM check (#99085)gh-99086: Fix implicit int warning in configure check for PTHREAD_SCOPE_SYSTEM (#99085)Nov 4, 2022
gentoo-bot pushed a commit to gentoo/cpython that referenced this pull request Nov 4, 2022
@thesamesam@mgorny
Fix -Wimplicit-int, -Wimplicit-function-declaration
3641763
These warnings became fatal by default in Clang 16.
Most of this has since been fixed upstream but
python#99085 is pending.
Signed-off-by: Sam James <sam@gentoo.org>
@thesamesam

thesamesam commented Nov 4, 2022

Copy link
Copy Markdown
ContributorAuthor

One small nit regarding the NEWS entry. The AC fix is fine with me, but IMO using AC_LANG_PROGRAM iso. AC_LANG_SOURCE would have been better.

Ah, that's a fair point. I've got an absolute tonne of these to fix (https://lwn.net/SubscriberLink/913505/b61bbc0c4296117f/) so I've been trying to make non-invasive changes. Is it alright if I leave this one as-is?

@erlend-aasland

erlend-aasland commented Nov 5, 2022

Copy link
Copy Markdown
Contributor

Ah, that's a fair point. I've got an absolute tonne of these to fix (https://lwn.net/SubscriberLink/913505/b61bbc0c4296117f/) so I've been trying to make non-invasive changes. Is it alright if I leave this one as-is?

Yeah, leave this as it is. We can get rid of AC_LANG_SOURCE in a separate issue/PR.

@erlend-aasland
erlend-aasland merged commit 12078e7 into python:mainNov 5, 2022
@miss-islington

miss-islington commented Nov 5, 2022

Copy link
Copy Markdown
Contributor

Thanks @thesamesam for the PR, and @erlend-aasland for merging it 🌮🎉.. I'm working now to backport this PR to: 3.10, 3.11.
🐍🍒⛏🤖

@miss-islington

miss-islington commented Nov 5, 2022

Copy link
Copy Markdown
Contributor

Sorry, @thesamesam and @erlend-aasland, I could not cleanly backport this to 3.11 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 12078e78f6e4a21f344e4eaff529e1ff3b97734f 3.11

@miss-islington

miss-islington commented Nov 5, 2022

Copy link
Copy Markdown
Contributor

Sorry @thesamesam and @erlend-aasland, I had trouble checking out the 3.10 backport branch.
Please retry by removing and re-adding the "needs backport to 3.10" label.
Alternatively, you can backport using cherry_picker on the command line.
cherry_picker 12078e78f6e4a21f344e4eaff529e1ff3b97734f 3.10

erlend-aasland pushed a commit to erlend-aasland/cpython that referenced this pull request Nov 5, 2022
@thesamesam@erlend-aasland
[3.11]pythongh-99086: Fix implicit int compiler warning in configure…
8f073db
… check for PTHREAD_SCOPE_SYSTEM (pythonGH-99085)
(cherry picked from commit 12078e7)
Co-authored-by: Sam James <sam@cmpct.info>
erlend-aasland pushed a commit to erlend-aasland/cpython that referenced this pull request Nov 5, 2022
@thesamesam@erlend-aasland
[3.10]pythongh-99086: Fix implicit int compiler warning in configure…
51459a8
… check for PTHREAD_SCOPE_SYSTEM (pythonGH-99085)
(cherry picked from commit 12078e7)
Co-authored-by: Sam James <sam@cmpct.info>
@bedevere-bot

bedevere-bot commented Nov 5, 2022

Copy link
Copy Markdown

GH-99118 is a backport of this pull request to the 3.11 branch.

@bedevere-botbedevere-bot removed the needs backport to 3.11 only security fixes label Nov 5, 2022
@bedevere-bot

bedevere-bot commented Nov 5, 2022

Copy link
Copy Markdown

GH-99119 is a backport of this pull request to the 3.10 branch.

@bedevere-botbedevere-bot removed the needs backport to 3.10 only security fixes label Nov 5, 2022
erlend-aasland added a commit that referenced this pull request Nov 5, 2022
@erlend-aasland@thesamesam
[3.11]gh-99086: Fix implicit int compiler warning in configure check…
573b451
… for PTHREAD_SCOPE_SYSTEM (GH-99085) (#99118)
(cherry picked from commit 12078e7)
Co-authored-by: Sam James <sam@cmpct.info>
@thesamesam

thesamesam commented Nov 5, 2022•
edited
Loading

Copy link
Copy Markdown
ContributorAuthor

Thank you!

@thesamesam
thesamesam deleted the clang-16 branch November 5, 2022 18:31
erlend-aasland added a commit that referenced this pull request Nov 6, 2022
@erlend-aasland@thesamesam
[3.10]gh-99086: Fix implicit int compiler warning in configure check…
0d5b25b
… for PTHREAD_SCOPE_SYSTEM (GH-99085) (#99119)
(cherry picked from commit 12078e7)
Co-authored-by: Sam James <sam@cmpct.info>
Co-authored-by: Sam James <sam@cmpct.info>
This was referenced Nov 12, 2022
Merged
Closed
gentoo-bot pushed a commit to gentoo/cpython that referenced this pull request May 21, 2024
@thesamesam@mgorny
pythongh-99086: Build: Fix -Wimplicit-int in PTHREAD_SCOPE_SYSTEM che…
3e72cda
…ck (python#99085)
Clang 16 makes -Wimplicit-int fatal by default.
Avoids errors like:
```
conftest.c:142:7: error: type specifier missing, defaults to 'int'; ISO C99 and later do not support implicit int [-Wimplicit-int]
```
Signed-off-by: Sam James <sam@gentoo.org>
gentoo-bot pushed a commit to gentoo/cpython that referenced this pull request May 21, 2024
@thesamesam@mgorny
pythongh-99086: Build: Fix -Wimplicit-int in PTHREAD_SCOPE_SYSTEM che…
b829383
…ck (python#99085)
Clang 16 makes -Wimplicit-int fatal by default.
Avoids errors like:
```
conftest.c:142:7: error: type specifier missing, defaults to 'int'; ISO C99 and later do not support implicit int [-Wimplicit-int]
```
Signed-off-by: Sam James <sam@gentoo.org>
gentoo-bot pushed a commit to gentoo/cpython that referenced this pull request Sep 19, 2024
@thesamesam@mgorny
pythongh-99086: Build: Fix -Wimplicit-int in PTHREAD_SCOPE_SYSTEM che…
0f754d5
…ck (python#99085)
Clang 16 makes -Wimplicit-int fatal by default.
Avoids errors like:
```
conftest.c:142:7: error: type specifier missing, defaults to 'int'; ISO C99 and later do not support implicit int [-Wimplicit-int]
```
Signed-off-by: Sam James <sam@gentoo.org>
gentoo-bot pushed a commit to gentoo/cpython that referenced this pull request Sep 19, 2024
@thesamesam@mgorny
pythongh-99086: Build: Fix -Wimplicit-int in PTHREAD_SCOPE_SYSTEM che…
c0aaae3
…ck (python#99085)
Clang 16 makes -Wimplicit-int fatal by default.
Avoids errors like:
```
conftest.c:142:7: error: type specifier missing, defaults to 'int'; ISO C99 and later do not support implicit int [-Wimplicit-int]
```
Signed-off-by: Sam James <sam@gentoo.org>
gentoo-bot pushed a commit to gentoo/cpython that referenced this pull request Jul 30, 2025
@thesamesam@mgorny
pythongh-99086: Build: Fix -Wimplicit-int in PTHREAD_SCOPE_SYSTEM che…
0128edd
…ck (python#99085)
Clang 16 makes -Wimplicit-int fatal by default.
Avoids errors like:
```
conftest.c:142:7: error: type specifier missing, defaults to 'int'; ISO C99 and later do not support implicit int [-Wimplicit-int]
```
Signed-off-by: Sam James <sam@gentoo.org>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@erlend-aaslanderlend-aaslanderlend-aasland approved these changes
@kumaraditya303kumaraditya303kumaraditya303 approved these changes

Assignees

@erlend-aaslanderlend-aasland

Labels

type-bugAn unexpected behavior, bug, or error

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@thesamesam@bedevere-bot@erlend-aasland@miss-islington@kumaraditya303