TheCloud/cpp-subprocess
1
0
Fork 0
mirror of https://github.com/arun11299/cpp-subprocess.git synced 2025-08-04 20:36:20 -04:00
Code Issues Packages Projects Releases Wiki Activity

Namespace library with github name.

Browse Source 
While this is header-only, and users are expected
to download the header and commit it to their project, it's better to
have unique namespace for the header by default.

This has two reasons:

  1. I wish to enable installing it with conan.io, and want to reduce
     the possibility for namespace clashes. The term "subprocess" is
     pretty overloaded.
  2. Currently the README and the CMakeLists differ in their include
     path, it's better to be consistent, even though users can put it
     wherever they please in their project.

Since most users are using it as header only anyway, I don't think it'd
make much diference.
This commit is contained in:
Elazar Leibovich 2022-01-12 14:53:38 +02:00
parent fa58db9d6e
commit a129a08ca5
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
CMakeLists.txt
View File

@ -9,4 +9,4 @@ if(BUILD_TESTING)
add_subdirectory(test)
endif()
install(FILES subprocess.hpp DESTINATION include/subprocess/)
install(FILES subprocess.hpp DESTINATION include/cpp-subprocess/)

2
README.md
View File

@ -22,7 +22,7 @@ and they will be fixed as they are reported.
Subprocess library has just a single source `subprocess.hpp` present at the top directory of this repository. All you need to do is add
```cpp
#inlcude "subprocess.hpp"
#inlcude "cpp-subprocess/subprocess.hpp"
using namespace subprocess;
// OR