Consolidating compiler warning options (#682)

src/CMakeLists.txt

@@ -12,23 +12,17 @@ set(MAJOR_VERSION "1" CACHE STRING "Major version")
 set(MINOR_VERSION "0" CACHE STRING "Minor version")
 set(PATCH_VERSION "5" CACHE STRING "Patch version")
 
-option(BUILD_ADAPTERS   "Build OSConfig Adapters" ON)
-option(BUILD_MODULES    "Build OSConfig Modules" ON)
-option(BUILD_PLATFORM   "Build OSConfig Platform" ON)
-option(BUILD_TESTS      "Build test collateral" ON)
+option(BUILD_ADAPTERS "Build OSConfig Adapters" ON)
+option(BUILD_MODULES "Build OSConfig Modules" ON)
+option(BUILD_PLATFORM "Build OSConfig Platform" ON)
+option(BUILD_TESTS "Build test collateral" ON)
 option(BUILD_MODULETEST "Build the moduletest tool" ON)
-option(BUILD_SAMPLES    "Build samples" OFF)
-option(COVERAGE         "Enable code coverage" OFF)
-
-add_compile_options("$<$<COMPILE_LANGUAGE:CXX>:-Wall;-Wextra;-Wunused;-Werror;-Wformat;-Wformat-security;-Wreorder;-Wno-nonnull;-Wno-unused-result;-Wunused-macros>")
+option(BUILD_SAMPLES "Build samples" OFF)
+option(COVERAGE "Enable code coverage" OFF)
 
 if (CMAKE_C_COMPILER_ID STREQUAL "GNU")
     add_compile_options("-Wno-psabi;-fPIC")
-
-    if ((NOT CMAKE_C_COMPILER_VERSION VERSION_LESS "5.3.0") AND (NOT CMAKE_CXX_COMPILER_VERSION VERSION_LESS "5.3.0"))
-        add_compile_options("-unused-const-variable=2")
-    endif()
-
+
     # Special Ubuntu 14/GNU 4.8 C compiler build configuration for the Universal NRP adapter
     if (CMAKE_C_COMPILER_VERSION VERSION_GREATER "4.7" AND CMAKE_C_COMPILER_VERSION VERSION_LESS "4.9")
         set(BUILD_MODULES OFF)
@@ -77,15 +71,15 @@ else()
     set(CMAKE_PROJECT_VERSION_TWEAK "${_timestamp}")
 endif()
 
-set(OsConfigProjectName         "osconfig")
-set(OsConfigProjectLongName     "Azure OSConfig")
-set(OsConfigProjectDescription  "Azure OSConfig is a modular security configuration stack for Linux Edge devices. OSConfig supports multi-authority device management over Azure and Azure Portal/CLI, GitOps, as well as local management.")
-set(OsConfigProjectVendor       "Microsoft")
-set(OsConfigProjectEmail        "osconfigsupport@microsoft.com")
-set(OsConfigProjectUrl          "https://github.com/Azure/azure-osconfig/")
-set(OsConfigHashAlgorithm       "SHA256")
-set(OsConfigVersionString       ${CMAKE_PROJECT_VERSION_MAJOR}.${CMAKE_PROJECT_VERSION_MINOR}.${CMAKE_PROJECT_VERSION_PATCH}.${CMAKE_PROJECT_VERSION_TWEAK})
-set(OsConfigRootBinaryDir       ${CMAKE_BINARY_DIR})
+set(OsConfigProjectName "osconfig")
+set(OsConfigProjectLongName "Azure OSConfig")
+set(OsConfigProjectDescription "Azure OSConfig is a modular security configuration stack for Linux Edge devices that supports multi-authority device management over Azure, GitOps, as well as local management.")
+set(OsConfigProjectVendor "Microsoft")
+set(OsConfigProjectEmail "osconfigcore@microsoft.com")
+set(OsConfigProjectUrl "https://github.com/Azure/azure-osconfig/")
+set(OsConfigHashAlgorithm "SHA256")
+set(OsConfigVersionString ${CMAKE_PROJECT_VERSION_MAJOR}.${CMAKE_PROJECT_VERSION_MINOR}.${CMAKE_PROJECT_VERSION_PATCH}.${CMAKE_PROJECT_VERSION_TWEAK})
+set(OsConfigRootBinaryDir ${CMAKE_BINARY_DIR})
 
 message(STATUS "${OsConfigProjectName} v${OsConfigVersionString}")
 configure_file("${CMAKE_CURRENT_SOURCE_DIR}/cmake/templates/version.h.in" "${CMAKE_CURRENT_SOURCE_DIR}/../src/common/commonutils/version.h" @ONLY)
@@ -122,52 +116,51 @@ if (BUILD_MODULES)
     add_subdirectory(modules)
 endif()
 
-set(CMAKE_MODULE_PATH                  ${CMAKE_MODULE_PATH} "${CMAKE_MODULE_PATH};${CMAKE_CURRENT_SOURCE_DIR}/cmake")
+set(CMAKE_MODULE_PATH ${CMAKE_MODULE_PATH} "${CMAKE_MODULE_PATH};${CMAKE_CURRENT_SOURCE_DIR}/cmake")
 
-set(CPACK_PACKAGE_VENDOR               ${OsConfigProjectVendor})
-set(CPACK_DEBIAN_PACKAGE_MAINTAINER    ${OsConfigProjectEmail})
+set(CPACK_PACKAGE_VENDOR ${OsConfigProjectVendor})
+set(CPACK_DEBIAN_PACKAGE_MAINTAINER ${OsConfigProjectEmail})
 
 # Common packaging
-set(CPACK_PACKAGE_NAME                 ${OsConfigProjectName})
-set(CPACK_PACKAGE_HOMEPAGE_URL         ${OsConfigProjectUrl})
-set(CPACK_PACKAGE_DESCRIPTION_SUMMARY  ${OsConfigProjectLongName})
-set(CPACK_PACKAGE_DESCRIPTION          ${OsConfigProjectDescription})
-set(CPACK_PACKAGE_VERSION              ${OsConfigVersionString})
-set(CPACK_PACKAGE_VENDOR               ${OsConfigProjectVendor})
-set(CPACK_PACKAGE_CONTACT              ${OsConfigProjectEmail})
-set(CPACK_PACKAGE_CHECKSUM             ${OsConfigHashAlgorithm})
+set(CPACK_PACKAGE_NAME ${OsConfigProjectName})
+set(CPACK_PACKAGE_HOMEPAGE_URL ${OsConfigProjectUrl})
+set(CPACK_PACKAGE_DESCRIPTION_SUMMARY ${OsConfigProjectLongName})
+set(CPACK_PACKAGE_DESCRIPTION ${OsConfigProjectDescription})
+set(CPACK_PACKAGE_VERSION ${OsConfigVersionString})
+set(CPACK_PACKAGE_VENDOR ${OsConfigProjectVendor})
+set(CPACK_PACKAGE_CONTACT ${OsConfigProjectEmail})
+set(CPACK_PACKAGE_CHECKSUM ${OsConfigHashAlgorithm})
 
 set(CPACK_EXCLUDES
     "${CMAKE_SOURCE_DIR}/adapters/pnp/azure-iot-sdk-c/"
     "${CMAKE_SOURCE_DIR}/reference/"
-    "${CMAKE_SOURCE_DIR}/build/"
-)
+    "${CMAKE_SOURCE_DIR}/build/")
 
 # Source packaging
-set(CPACK_SOURCE_GENERATOR      "TGZ")
-set(CPACK_SOURCE_IGNORE_FILES   ${CPACK_EXCLUDES})
-set(CPACK_VERBATIM_VARIABLES    YES)
+set(CPACK_SOURCE_GENERATOR "TGZ")
+set(CPACK_SOURCE_IGNORE_FILES ${CPACK_EXCLUDES})
+set(CPACK_VERBATIM_VARIABLES YES)
 
 # RPM packaging - see /devops/rpm for postinst/postun/preun scripts
-set(CPACK_RPM_BUILDREQUIRES                 "systemd, cmake, build-essential, tar, gcc, gtest, gtest-devel, gmock, gmock-devel, curl-devel, openssl, rapidjson")
-set(CPACK_RPM_CHANGELOG_FILE                "${CMAKE_SOURCE_DIR}/../devops/rpm/changelog")
-set(CPACK_RPM_PACKAGE_AUTOREQ               1)
-set(CPACK_RPM_PACKAGE_LICENSE               "MIT")
-set(CPACK_RPM_PACKAGE_NAME                  ${OsConfigProjectName})
-set(CPACK_RPM_PACKAGE_DESCRIPTION           ${OsConfigProjectDescription})
-set(CPACK_RPM_POST_INSTALL_SCRIPT_FILE      "${CMAKE_SOURCE_DIR}/../devops/rpm/postinst")
-set(CPACK_RPM_POST_UNINSTALL_SCRIPT_FILE    "${CMAKE_SOURCE_DIR}/../devops/rpm/postun")
-set(CPACK_RPM_PRE_UNINSTALL_SCRIPT_FILE     "${CMAKE_SOURCE_DIR}/../devops/rpm/preun")
+set(CPACK_RPM_BUILDREQUIRES "systemd, cmake, build-essential, tar, gcc, gtest, gtest-devel, gmock, gmock-devel, curl-devel, openssl, rapidjson")
+set(CPACK_RPM_CHANGELOG_FILE "${CMAKE_SOURCE_DIR}/../devops/rpm/changelog")
+set(CPACK_RPM_PACKAGE_AUTOREQ 1)
+set(CPACK_RPM_PACKAGE_LICENSE "MIT")
+set(CPACK_RPM_PACKAGE_NAME ${OsConfigProjectName})
+set(CPACK_RPM_PACKAGE_DESCRIPTION ${OsConfigProjectDescription})
+set(CPACK_RPM_POST_INSTALL_SCRIPT_FILE "${CMAKE_SOURCE_DIR}/../devops/rpm/postinst")
+set(CPACK_RPM_POST_UNINSTALL_SCRIPT_FILE "${CMAKE_SOURCE_DIR}/../devops/rpm/postun")
+set(CPACK_RPM_PRE_UNINSTALL_SCRIPT_FILE "${CMAKE_SOURCE_DIR}/../devops/rpm/preun")
 
 # Debian packaging - see /devops/debian for postinst/postrm/prerm scripts
 # https://www.debian.org/doc/manuals/maint-guide/
 set(CPACK_DEBIAN_EXCLUDE_FROM_AUTO_FILELIST ${CPACK_EXCLUDES})
-set(CPACK_DEBIAN_FILE_NAME                  ${OsConfigProjectName}_${OsConfigVersionString}_${LSB_RELEASE_CODENAME_SHORT}_${CMAKE_SYSTEM_PROCESSOR}.deb)
-set(CPACK_DEBIAN_PACKAGE_CONTROL_EXTRA      "${CMAKE_SOURCE_DIR}/../devops/debian/postinst;${CMAKE_SOURCE_DIR}/../devops/debian/prerm;${CMAKE_SOURCE_DIR}/../devops/debian/postrm;")
-set(CPACK_DEBIAN_PACKAGE_GENERATE_SHLIBS    ON)
-set(CPACK_DEBIAN_PACKAGE_MAINTAINER         ${OsConfigProjectEmail})
-set(CPACK_DEBIAN_PACKAGE_NAME               ${OsConfigProjectName})
-set(CPACK_DEBIAN_PACKAGE_PREDEPENDS         "")
-set(CPACK_DEBIAN_PACKAGE_SUGGESTS           "aziot-identity-service (>= 1.2.0)")
+set(CPACK_DEBIAN_FILE_NAME ${OsConfigProjectName}_${OsConfigVersionString}_${LSB_RELEASE_CODENAME_SHORT}_${CMAKE_SYSTEM_PROCESSOR}.deb)
+set(CPACK_DEBIAN_PACKAGE_CONTROL_EXTRA "${CMAKE_SOURCE_DIR}/../devops/debian/postinst;${CMAKE_SOURCE_DIR}/../devops/debian/prerm;${CMAKE_SOURCE_DIR}/../devops/debian/postrm;")
+set(CPACK_DEBIAN_PACKAGE_GENERATE_SHLIBS ON)
+set(CPACK_DEBIAN_PACKAGE_MAINTAINER ${OsConfigProjectEmail})
+set(CPACK_DEBIAN_PACKAGE_NAME ${OsConfigProjectName})
+set(CPACK_DEBIAN_PACKAGE_PREDEPENDS "")
+set(CPACK_DEBIAN_PACKAGE_SUGGESTS "aziot-identity-service (>= 1.2.0)")
 
 include(CPack)

src/adapters/mc/CMakeLists.txt

@@ -1,5 +1,7 @@
 # Copyright (c) Microsoft Corporation. All rights reserved.
 # Licensed under the MIT License.
 
+add_compile_options("-Wall;-Wextra;-Wunused;-Werror;-Wformat;-Wformat-security;-Wno-unused-result")
+
 add_subdirectory(ssh)
 add_subdirectory(asb)

src/adapters/mc/OsConfigResource.c

@@ -101,7 +101,7 @@ static void LogOsConfigVersion(MI_Context* context)
 
     if ((NULL != g_mpiHandle) && (MPI_OK == CallMpiGet(deviceInfoComponent, osConfigVersionObject, &objectValue, &objectValueLength, GetLog())) && (NULL != objectValue))
     {
-        if (NULL != (payloadString = malloc(((objectValueLength > 0) ? objectValueLength : strlen(objectValue)) + 1)))
+        if (NULL != (payloadString = malloc(((objectValueLength > 0) ? objectValueLength : (int)strlen(objectValue)) + 1)))
         {
             memset(payloadString, 0, objectValueLength + 1);
             memcpy(payloadString, objectValue, objectValueLength);
@@ -165,7 +165,6 @@ void MI_CALL OsConfigResource_Load(
     MI_Context* context)
 {
     MI_UNREFERENCED_PARAMETER(selfModule);
-    int status = 0;
 
     *self = NULL;
 
@@ -492,6 +491,9 @@ void MI_CALL OsConfigResource_Invoke_GetTargetResource(
     const OsConfigResource_GetTargetResource* in)
 {
     MI_UNREFERENCED_PARAMETER(self);
+    MI_UNREFERENCED_PARAMETER(nameSpace);
+    MI_UNREFERENCED_PARAMETER(className);
+    MI_UNREFERENCED_PARAMETER(methodName);
     MI_UNREFERENCED_PARAMETER(instanceName);
 
     const char* auditPassed = "Audit passed";
@@ -510,7 +512,7 @@ void MI_CALL OsConfigResource_Invoke_GetTargetResource(
     MI_Value miValueReasonResult = {0};
     MI_Boolean isCompliant = MI_FALSE;
 
-    OsConfigResource_GetTargetResource get_result_object = {0};
+    OsConfigResource_GetTargetResource get_result_object;
 
     if ((NULL == in) || (MI_FALSE == in->InputResource.exists) || (NULL == in->InputResource.value))
     {
@@ -867,9 +869,12 @@ void MI_CALL OsConfigResource_Invoke_TestTargetResource(
     const OsConfigResource_TestTargetResource* in)
 {
     MI_UNREFERENCED_PARAMETER(self);
+    MI_UNREFERENCED_PARAMETER(nameSpace);
+    MI_UNREFERENCED_PARAMETER(className);
+    MI_UNREFERENCED_PARAMETER(methodName);
     MI_UNREFERENCED_PARAMETER(instanceName);
 
-    OsConfigResource_TestTargetResource test_result_object = {0};
+    OsConfigResource_TestTargetResource test_result_object;
 
     MI_Result miResult = MI_RESULT_OK;
     MI_Result miCleanup = MI_RESULT_OK;
@@ -1035,13 +1040,15 @@ void MI_CALL OsConfigResource_Invoke_SetTargetResource(
     const OsConfigResource_SetTargetResource* in)
 {
     MI_UNREFERENCED_PARAMETER(self);
+    MI_UNREFERENCED_PARAMETER(nameSpace);
+    MI_UNREFERENCED_PARAMETER(className);
+    MI_UNREFERENCED_PARAMETER(methodName);
     MI_UNREFERENCED_PARAMETER(instanceName);
 
     MI_Result miResult = MI_RESULT_OK;
     MI_Result miCleanup = MI_RESULT_OK;
-    int mpiResult = MPI_OK;
 
-    OsConfigResource_SetTargetResource set_result_object = {0};
+    OsConfigResource_SetTargetResource set_result_object;
 
     if ((NULL == in) || (MI_FALSE == in->InputResource.exists) || (NULL == in->InputResource.value))
     {

src/adapters/pnp/CMakeLists.txt

@@ -30,7 +30,7 @@ else ()
 endif ()
 
 if (EXISTS ${PROJECT_SOURCE_DIR}/azure-iot-sdk-c/CMakeLists.txt)
-    target_compile_options(${target_name} PRIVATE -Wall -Wextra -Wunused -Werror -Wformat -Wformat-security -Wunused-macros)
+    target_compile_options(${target_name} PRIVATE -Wall -Wextra -Wunused -Werror -Wformat -Wformat-security)
 
     target_include_directories(${target_name} PUBLIC
         ${azure_iot_sdks_SOURCE_DIR}/deps/parson
@@ -56,7 +56,7 @@ if (EXISTS ${PROJECT_SOURCE_DIR}/azure-iot-sdk-c/CMakeLists.txt)
         commonutils
         mpiclient)
 else()
-    target_compile_options(${target_name} PRIVATE -Wall -Wextra -Wunused -Werror -Wformat -Wformat-security -Wno-nonnull -Wno-unused-result -Wunused-macros)
+    target_compile_options(${target_name} PRIVATE -Wall -Wextra -Wunused -Werror -Wformat -Wformat-security -Wno-unused-result)
 
     find_package (azure_c_shared_utility REQUIRED)
     find_package (azure_macro_utils_c REQUIRED)

src/common/CMakeLists.txt

@@ -1,7 +1,7 @@
 # Copyright (c) Microsoft Corporation. All rights reserved.
 # Licensed under the MIT License.
 
-add_compile_options("$<$<COMPILE_LANGUAGE:C>:-Wall;-Wextra;-Wunused;-Werror;-Wformat;-Wformat-security;-Wno-nonnull;-Wno-unused-result>")
+add_compile_options("-Wall;-Wextra;-Wunused;-Werror;-Wformat;-Wformat-security;-Wno-unused-result")
 
 if (BUILD_TESTS)
     add_compile_options(-D TEST_CODE)

src/common/asb/Asb.c

@@ -1986,7 +1986,7 @@ static char* AuditEnsureAllWirelessInterfacesAreDisabled(void* log)
     if (0 == CheckTextNotFoundInCommandOutput("/sbin/iwconfig 2>&1 | /bin/egrep -v 'no wireless extensions|not found'", "Frequency", &reason, log))
     {
         FREE_MEMORY(reason);
-        reason = DuplicateString("No active wireless interfaces are present");
+        reason = FormatAllocateString("%sNo active wireless interfaces are present", g_pass);
     }
     else
     {

src/common/asb/Asb.h

@@ -4,6 +4,51 @@
 #ifndef ASB_H
 #define ASB_H
 
+#define InternalOsConfigAddReason(reason, format, ...) {\
+    char* last = NULL;\
+    char* temp = FormatAllocateString("%s, also ", *reason);\
+    FREE_MEMORY(*reason);\
+    last = FormatAllocateString(format, ##__VA_ARGS__);\
+    last[0] = tolower(last[0]);\
+    *reason = ConcatenateStrings(temp, last);\
+    FREE_MEMORY(temp);\
+    FREE_MEMORY(last);\
+}\
+
+#define OsConfigCaptureReason(reason, format, ...) {\
+    if (NULL != reason) {\
+        if ((NULL != *reason) && (0 != strncmp(*reason, SECURITY_AUDIT_PASS, strlen(SECURITY_AUDIT_PASS)))) {\
+            InternalOsConfigAddReason(reason, format, ##__VA_ARGS__);\
+        } else {\
+            FREE_MEMORY(*reason);\
+            *reason = FormatAllocateString(format, ##__VA_ARGS__);\
+        }\
+    }\
+}\
+
+#define OsConfigCaptureSuccessReason(reason, format, ...) {\
+    char* temp = NULL;\
+    if (NULL != reason) {\
+        if ((NULL != *reason) && (0 == strncmp(*reason, SECURITY_AUDIT_PASS, strlen(SECURITY_AUDIT_PASS)))) {\
+            InternalOsConfigAddReason(reason, format, ##__VA_ARGS__);\
+        } else {\
+            FREE_MEMORY(*reason);\
+            temp = FormatAllocateString(format, ##__VA_ARGS__);\
+            *reason = ConcatenateStrings(SECURITY_AUDIT_PASS, temp);\
+            FREE_MEMORY(temp);\
+        }\
+    }\
+}\
+
+#define OsConfigIsSuccessReason(reason)\
+    (((NULL != reason) && ((NULL == *reason) || (0 == strncmp(*reason, SECURITY_AUDIT_PASS, strlen(SECURITY_AUDIT_PASS))))) ? true : false)\
+
+#define OsConfigResetReason(reason) {\
+    if (NULL != reason) {\
+        FREE_MEMORY(*reason);\
+    }\
+}\
+
 #ifdef __cplusplus
 extern "C"
 {

src/common/commonutils/CommonUtils.h

@@ -42,51 +42,6 @@
     }\
 }\
 
-#define InternalOsConfigAddReason(reason, format, ...) {\
-    char* last = NULL;\
-    char* temp = FormatAllocateString("%s, also ", *reason);\
-    FREE_MEMORY(*reason);\
-    last = FormatAllocateString(format, ##__VA_ARGS__);\
-    last[0] = tolower(last[0]);\
-    *reason = ConcatenateStrings(temp, last);\
-    FREE_MEMORY(temp);\
-    FREE_MEMORY(last);\
-}\
-
-#define OsConfigCaptureReason(reason, format, ...) {\
-    if (NULL != reason) {\
-        if ((NULL != *reason) && (0 != strncmp(*reason, SECURITY_AUDIT_PASS, strlen(SECURITY_AUDIT_PASS)))) {\
-            InternalOsConfigAddReason(reason, format, ##__VA_ARGS__);\
-        } else {\
-            FREE_MEMORY(*reason);\
-            *reason = FormatAllocateString(format, ##__VA_ARGS__);\
-        }\
-    }\
-}\
-
-#define OsConfigCaptureSuccessReason(reason, format, ...) {\
-    char* temp = NULL;\
-    if (NULL != reason) {\
-        if ((NULL != *reason) && (0 == strncmp(*reason, SECURITY_AUDIT_PASS, strlen(SECURITY_AUDIT_PASS)))) {\
-            InternalOsConfigAddReason(reason, format, ##__VA_ARGS__);\
-        } else {\
-            FREE_MEMORY(*reason);\
-            temp = FormatAllocateString(format, ##__VA_ARGS__);\
-            *reason = ConcatenateStrings(SECURITY_AUDIT_PASS, temp);\
-            FREE_MEMORY(temp);\
-        }\
-    }\
-}\
-
-#define OsConfigIsSuccessReason(reason)\
-    (((NULL != reason) && ((NULL == *reason) || (0 == strncmp(*reason, SECURITY_AUDIT_PASS, strlen(SECURITY_AUDIT_PASS))))) ? true : false)\
-
-#define OsConfigResetReason(reason) {\
-    if (NULL != reason) {\
-        FREE_MEMORY(*reason);\
-    }\
-}\
-
 // Linefeed (LF) ASCII character
 #ifndef EOL
 #define EOL 10

src/common/commonutils/FileUtils.c

@@ -865,7 +865,7 @@ int FindTextInFolder(const char* directory, const char* text, void* log)
     {
         while (NULL != (entry = readdir(home)))
         {
-            if (entry->d_name && strcmp(entry->d_name, ".") && strcmp(entry->d_name, ".."))
+            if (strcmp(entry->d_name, ".") && strcmp(entry->d_name, ".."))
             {
                 length = strlen(pathTemplate) + strlen(directory) + strlen(entry->d_name);
                 if (NULL == (path = malloc(length + 1)))

src/common/commonutils/Internal.h

@@ -28,6 +28,8 @@
 #include <Logging.h>
 #include <CommonUtils.h>
 
+#include "../asb/Asb.h"
+
 #if ((__GLIBC__ == 2) && (__GLIBC_MINOR__ < 30))
 #include <sys/syscall.h>
 #define gettid() syscall(SYS_gettid)

src/modules/CMakeLists.txt

@@ -1,7 +1,7 @@
 # Copyright (c) Microsoft Corporation. All rights reserved.
 # Licensed under the MIT License.
 
-add_compile_options("$<$<COMPILE_LANGUAGE:C>:-Wall;-Wextra;-Wunused;-Werror;-Wformat;-Wformat-security;-Wno-nonnull;-Wno-unused-result;-Wunused-const-variable=2;-Wunused-macros>")
+add_compile_options("-Wall;-Wextra;-Wunused;-Werror;-Wformat;-Wformat-security;-Wno-unused-result;-Wunused-const-variable=2")
 
 set(MODULES_INSTALL_DIR "/usr/lib/osconfig" CACHE FILEPATH "Directory used for installing modules")
 set(MODULES_BUILD_BIN_DIR "${CMAKE_CURRENT_BINARY_DIR}/bin" CACHE FILEPATH "Directory used to collect module binaries for testing")