This is merely a convention, but you are advised to follow it, especially if you are working in a large project.
Let’s say you have a foo library.
You have the following files:
This is what your layout should look like:
fooproject
|__ libfoo
| CMakeLists.txt
|__ foo
| |__ foo.hpp
|__ src
| |__ foo.cpp
| |__ foo_private.hpp
| |__ foo_private.cpp
|__ test
|__ CMakeLists.txt
|__ foo_test.cpp
Note: you can download an archive containing the foo project here: fooproject.zip
Here’s what the CMakeLists.txt should look like
include_directories(".")
qi_create_lib(foo
SRC foo/foo.hpp
foo/foo.cpp
src/foo_private.hpp
src/foo_private.cpp
SHARED
)
qi_use_lib(foo)
qi_install_header(KEEP_RELATIVE_PATHS foo/foo.hpp)
qi_stage_lib(foo)
add_subdirectory(test)
Please note that the location of the CMake list file matters.
You will note that:
#include <foo/...>
This way, we are sure that the code we use can be re-distributed when the headers are installed, and that the path to find the headers while in the source tree does not differ from the paths to find the installed headers. This works because:
qi_install_header(foo/foo.hpp SUBFOLDER foo)
With the proposed layout, you have something like:
foooproject
|__ libfoo
| |__ foo
| |__ foo.hpp
|__ libbar
| |__ bar
| |__ bar.hpp
|__ foobar
|__ foobar.cpp
You may want to get rid of the libfoo/foo, libbar/bar redundancy and do this instead:
fooproject
|__ foo
| |__ foo.hpp
|__ bar
| |__ bar.hpp
|__ foobar
|__ foobar.cpp
But, let’s assume you make a mistake, and write
qi_use_lib(foobar foo)
instead of
qi_use_lib(foobar foo bar)
In the first layout, you will have an error during compile time, looking like:
bar/bar.hpp : no such file or directory
because the include directory that has been staged for foo is different from the include directory that has been staged for bar. Using the second layout, you will have an error during link time, looking like:
undefined reference to `bar_func'
because the include directory that was staged was always the same: fooproject. The additional nesting level helps you catch this king of errors early.
Note
For large libraries, also consider using submodules. The documentation can be found here