Skip to content

fix: add -Wno-incompatible-pointer-types for build#107

Merged
esheldon merged 13 commits intoesheldon:masterfrom
beckermr:patch-1
Feb 6, 2026
Merged

fix: add -Wno-incompatible-pointer-types for build#107
esheldon merged 13 commits intoesheldon:masterfrom
beckermr:patch-1

Conversation

@beckermr
Copy link
Contributor

@beckermr beckermr commented Feb 5, 2026

Trying to help fix #105.

@sidneymau
Copy link

Thanks! Confirming that

pip install git+https://github.com/beckermr/esutil.git@77860b441ecd63c36a0165a853364d30dd7d1b23

succeeds where installing from the main branch fails

@beckermr beckermr marked this pull request as ready for review February 6, 2026 10:02
@beckermr beckermr marked this pull request as draft February 6, 2026 10:03
Refactor extra_compile_args to conditionally include flags based on platform and compilation test.
Refactor build process to always check compiler flags.
Add verbose flag to pip install command for better output.
Removed verbose flag from pip install command.
Add print statement for debugging extension build.
Add verbose flag to pip install command in workflow.
Use copy to avoid modifying original compile flags.
Removed debug print statement from check_flags function.
@beckermr beckermr marked this pull request as ready for review February 6, 2026 10:33
@beckermr
Copy link
Contributor Author

beckermr commented Feb 6, 2026

@esheldon this one is ready to go!

@beckermr
Copy link
Contributor Author

beckermr commented Feb 6, 2026

@esheldon I do not have push access here. You'll need to merge.

@esheldon esheldon merged commit e10ce03 into esheldon:master Feb 6, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

incompatible pointer types

3 participants