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

Use BEZ_ prefix for exported C ABI names #167

Merged
merged 10 commits into from Jan 10, 2020
Merged

Use BEZ_ prefix for exported C ABI names #167

merged 10 commits into from Jan 10, 2020

Conversation

dhermes
Copy link
Owner

@dhermes dhermes commented Jan 10, 2020

Fixes #166.

Done via:

```
git grep -l "name='" -- '*.f90' | xargs sed -i '' s/"name='"/"name='BEZ_"/g
git grep -l "name='" -- '*.f90' | xargs sed -i    s/"name='"/"name='BEZ_"/g
```

This was an automated change, will be followed up by hand-written
changes to all references to these ABI procedures.
(Still need to run `clang-format` on these, it is not yet installed
on my laptop and I am currently offline.)
- Updated some Sphinx references with `BEZ_` prefix
- Fixed an oversight in `example_locate_point_curve`
- Fixed an old reference to `example_curve_specialize.c` (now is
  a reference to `example_specialize_curve.c`)
- Change `-I src/fortran/include/` line to use `...` to indicate
  that path is not in the same directory as the `example*.c`
  files
I introduced this in dc58f0a,
but commited it with `[ci skip]`. (I should never use `ci skip`,
I've learned that lesson before.)
@dhermes
Copy link
Owner Author

dhermes commented Jan 10, 2020

Not sure why new builds didn't get kicked off for Travis or CircleCI

@dhermes
Copy link
Owner Author

dhermes commented Jan 10, 2020

I'm going to merge this in hopes of triggering the correct builds on the master push. (I tried re-delivering the webhooks and CircleCI "skipped" for a 2nd time, citing the absence of .circleci/config.yml in this branch, which is incorrect.)

@dhermes dhermes merged commit fe16754 into master Jan 10, 2020
@dhermes dhermes deleted the issue-166 branch January 10, 2020 17:45
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.

Use BEZ_ prefix for exported C ABI names
1 participant