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

Namespace library with github name. (#72)

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.

Co-authored-by: Elazar Leibovich <elazar.leibovich@nextsilicon.com>
This commit is contained in:
Elazar Leibovich 2022-03-18 05:50:06 +02:00 committed by GitHub
parent fa58db9d6e
commit 0218c51b8c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
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