-
Notifications
You must be signed in to change notification settings - Fork 114
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
Rename example dir balance_seeds to balance #231
Conversation
Asking @mkirilin: does CMake build all example exactly as the autotools do it? |
@cburstedde
|
Thanks for checking. Likely some of our CMake updates did not update the part about the examples and is now incompatible to the core. Do you see any way to repair this? |
Just FYI. here is what I do to build both p4est and p4est examples with cmake: # 1. build p4est
cd $(P4EST_SRC)
mkdir -p _build/cmake
cd _build/cmake
cmake ../..
make; make install
# you should have a folder "local" where both p4est and libsc were just installed
# 2. now build p4est examples using p4est install in "local"
mkdir examples
cd examples
cmake -DCMAKE_PREFIX_PATH=`pwd`/../local ../../../example
make For me building the p4est example works fine; it just a bit "convoluted" because examples are in separated cmake project. It should be documented somewhere. |
Thanks for the summary!
Personally I'd rather simplify and have the examples built together with the
library. The separate project would no longer be necessary. How hard would
it be to do it this way (which is also the autotools way)?
… currently, the way cmake has been setup in `example` folder, examples are treated as a separated cmake project; so when you build p4est, examples are not built. One has to build p4est examples separated taking care of providing build information about where p4est libs where installed.
here is what I do to build both p4est and p4est examples with cmake:
```shell
# 1. build p4est
cd $(P4EST_SRC)
mkdir -p _build/cmake
cd _build/cmake
cmake ../..
make; make install
# you should have a folder "local" where both p4est and libsc were just installed
# 2. now build p4est examples using p4est install in "local"
mkdir examples
cd examples
cmake -DCMAKE_PREFIX_PATH=`pwd`/../local ../../../example
make
```
For me building the p4est example works fine; it just a bit "convoluted" because examples are in separated cmake project. It should be documented somewhere.
|
Should'nt be hard. I'll look it. |
I'll merge this now to continue with the proper directory structure. |
Rename example balance_seeds to balance
The example directory balance_seeds contains a specific balance test. To add further balance tests in the future, it will be more fitting to rename the directory plainly to balance.