Skip to content

Commit 1a0c6df

Browse files
committed
Address review comments.
1 parent 9a4099c commit 1a0c6df

File tree

6 files changed

+7
-7
lines changed

6 files changed

+7
-7
lines changed

CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@ if(ystdlib_ENABLE_TESTS)
8686
endif()
8787

8888
# All libraries are required to use the same minimum version of dependencies to avoid issues.
89-
set(MIN_BOOST_VERSOIN "1.81.0")
89+
set(MIN_BOOST_VERSION "1.81.0")
9090

9191
add_subdirectory(src/ystdlib)
9292

cmake/ystdlib-helpers.cmake

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ macro(check_required_arguments_exist REQUIRED_ARG_NAMES)
88
set(_NAMES "${REQUIRED_ARG_NAMES}")
99
foreach(_NAME IN LISTS _NAMES)
1010
if(NOT DEFINED ARG_${_NAME} OR ARG_${_NAME} STREQUAL "")
11-
message(FATAL_ERROR "Empty value for argument: '${_NAME}'")
11+
message(FATAL_ERROR "Missing or empty value for argument: '${_NAME}'")
1212
endif()
1313
endforeach()
1414
endmacro()

examples/CMakeLists.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,13 +19,13 @@ if(ystdlib-examples_IS_TOP_LEVEL)
1919
# Include dependency settings if the project isn't being included as a subproject.
2020
# NOTE: We mark the file optional because if the user happens to have the dependencies
2121
# installed, this file is not necessary.
22-
include("${CMAKE_SOURCE_DIR}/../build/deps/cmake-settings/all.cmake" OPTIONAL)
22+
include("${CMAKE_CURRENT_LIST_DIR}/../build/deps/cmake-settings/all.cmake" OPTIONAL)
2323
endif()
2424

2525
find_package(ystdlib REQUIRED)
2626
message(STATUS "Found ystdlib ${ystdlib_VERSION}.")
2727

28-
add_executable(linking-tests "${CMAKE_SOURCE_DIR}/src/linking-tests.cpp")
28+
add_executable(linking-tests "${CMAKE_CURRENT_LIST_DIR}/src/linking-tests.cpp")
2929

3030
target_compile_features(linking-tests PRIVATE cxx_std_20)
3131

examples/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ This directory contains example programs that demonstrate how to use the ystdlib
55
The example program `linking-tests` references all of ystdlib's library targets to ensure they can
66
be installed and linked correctly.
77

8-
# Requirements
8+
## Requirements
99

1010
[Build](../README.md#building) and [install](../README.md#installing) ystdlib. The commands below
1111
assume you've built and installed ystdlib to `./build/examples/ystdlib`. If you installed it to a

examples/src/linking-tests.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@ auto main() -> int {
112112
}
113113

114114
if (false == test_io_interface()) {
115-
std::cerr << "Error: error_handling test failed. Could not use FailureReader.\n";
115+
std::cerr << "Error: io_interface test failed. Could not use FailureReader.\n";
116116
return 3;
117117
}
118118

src/ystdlib/error_handling/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ if(TARGET ystdlib::error_handling)
33
endif()
44

55
set(BOOST_FIND_PACKAGE_ARGS
6-
"${MIN_BOOST_VERSOIN}"
6+
"${MIN_BOOST_VERSION}"
77
REQUIRED
88
COMPONENTS
99
headers

0 commit comments

Comments
 (0)