Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

gh-118343: (void *) casting to avoid gcc-14 build failure issue#118430

Closed
corona10 wants to merge 1 commit into
python:mainpython/cpython:mainfrom
corona10:gh-118343-workaroundcorona10/cpython:gh-118343-workaroundCopy head branch name to clipboard
Closed

gh-118343: (void *) casting to avoid gcc-14 build failure issue#118430
corona10 wants to merge 1 commit into
python:mainpython/cpython:mainfrom
corona10:gh-118343-workaroundcorona10/cpython:gh-118343-workaroundCopy head branch name to clipboard

Conversation

@corona10

@corona10 corona10 commented Apr 30, 2024

Copy link
Copy Markdown
Member

@corona10 corona10 force-pushed the gh-118343-workaround branch from 78cc72a to 7f3b43a Compare April 30, 2024 14:22
@corona10

corona10 commented Apr 30, 2024

Copy link
Copy Markdown
Member Author

@markshannon @gvanrossum I know that this is not a good solution, but from Fedora 40, it uses gcc-14, and the incompatible-pointer-types issue causes a build failure due to the changed compiler policy.
(Now it checks incompatible-pointer-types striclty)

See a similar issue: https://forums.developer.nvidia.com/t/nvidia-modules-build-failure-with-upcoming-gcc-14-and-recent-kernels-due-to-misfiring-conftest-sh-test-heads-up/279072

cc @vstinner

out.emit(f"if ({var.name} == NULL) goto out_of_space;\n")
else:
out.emit(f"{var.name} = sym_new_not_null(ctx);\n")
out.emit(f"{var.name} = (void *)sym_new_not_null(ctx);\n")

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

It will be great if we can infer the type of var.

@corona10 corona10 closed this Apr 30, 2024
@corona10

corona10 commented Apr 30, 2024

Copy link
Copy Markdown
Member Author

Well now it looks like be fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Morty Proxy This is a proxified and sanitized view of the page, visit original site.