Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Support
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in / Register
Toggle navigation
J
json
Project overview
Project overview
Details
Activity
Releases
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Issues
0
Issues
0
List
Boards
Labels
Milestones
Merge Requests
0
Merge Requests
0
CI / CD
CI / CD
Pipelines
Jobs
Schedules
Analytics
Analytics
CI / CD
Repository
Value Stream
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Create a new issue
Jobs
Commits
Issue Boards
Open sidebar
Libraries
json
Commits
a6d112ff
Unverified
Commit
a6d112ff
authored
5 years ago
by
Niels Lohmann
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
add regression test for #2281
parent
2301bd11
Changes
1
Hide whitespace changes
Inline
Side-by-side
Showing
1 changed file
with
2 additions
and
2 deletions
+2
-2
test/cmake_target_include_directories/project/CMakeLists.txt
test/cmake_target_include_directories/project/CMakeLists.txt
+2
-2
No files found.
test/cmake_target_include_directories/project/CMakeLists.txt
View file @
a6d112ff
...
...
@@ -11,11 +11,11 @@ target_include_directories(with_private_system_target PRIVATE SYSTEM ${nlohmann_
set_target_properties
(
with_private_system_target PROPERTIES CXX_STANDARD 11
)
# regression from https://github.com/nlohmann/json/discussions/2281
add_library
(
Foo S
HARED
Foo.cpp Bar.cpp
)
add_library
(
Foo S
TATIC
Foo.cpp Bar.cpp
)
target_include_directories
(
Foo PUBLIC
${
CMAKE_CURRENT_SOURCE_DIR
}
${
nlohmann_json_source
}
/include
)
set_target_properties
(
Foo PROPERTIES CXX_STANDARD 11
)
add_library
(
Bar S
HARED
Bar.cpp
)
add_library
(
Bar S
TATIC
Bar.cpp
)
target_link_libraries
(
Bar PRIVATE Foo
)
target_include_directories
(
Bar PUBLIC
${
CMAKE_CURRENT_SOURCE_DIR
}
${
nlohmann_json_source
}
/include
)
set_target_properties
(
Bar PROPERTIES CXX_STANDARD 11
)
This diff is collapsed.
Click to expand it.
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment