Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

C issues: #34

Merged
merged 2 commits into from
Jul 17, 2024
Merged

C issues: #34

merged 2 commits into from
Jul 17, 2024

Conversation

billbrod
Copy link
Member

In conda-forge/staged-recipes#25506, the OSX build of pyrtools was failing because of a compilation error in the C code (which uses gcc in our deploy action here, but clang16 in the conda build).

I installed clang=16 from conda-forge in a new conda environment, then ran CC=clang pip install . in a clean version of the repo (which forced the compilation to use clang, see setuptools docs), and was able to reproduce the error (if uninstalling and reinstalling, in order to try possible solutions, must also delete the build/ directory that gets created by pip install; adding -vvv will cause the output to include the compilation command, also useful for debugging).

The issue was the x_stop and y_stop did not have a type declared: parameter 'y_stop' was not declared, defaults to 'int'; ISO C99 and later do not support implicit int [-Wimplicit-int]

I also removed unused i,j, which were raising a warning. There are many other warnings, largely about function definitions without prototypes, and I don't understand C well enough to fix them. Writing out these notes in case we need to come back and fix them at some point.

@billbrod
Copy link
Member Author

billbrod commented Jul 17, 2024

After this is merged, will update the version on pypi and then trigger the conda build again.

@billbrod billbrod merged commit b6ba387 into main Jul 17, 2024
37 checks passed
@billbrod billbrod deleted the c_issues branch July 17, 2024 18:48
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.

1 participant