commit d8cacfd45466b0695b1af00dbcedaa6907e085f1
author Matt Spinler <spinler@us.ibm.com> Mon Apr 26 09:56:21 2021 -0500
committer Matt Spinler <spinler@us.ibm.com> Wed Apr 28 13:35:24 2021 -0500
tree 790fdacdaa9b9f39d0661f626f90c2ae19bc0a42
parent 8fa48a46b4db114a671e2dafe5edeccc1a294cd9

Change #ifdef to #if after move to meson

Unlike with autotools, meson options negative-errno-on-fail and
update-functional-on-fail are written to the generated config.h file
regardless of if they are enabled are not, such as

    #define NEGATIVE_ERRNO_ON_FAIL 1
and
    #define NEGATIVE_ERRNO_ON_FAIL 0

That means the code that checks the values needs to be an #if and not an

The bitbake recipe has now switched over to building with meson, so
right now openbmc master may not be building with the correct options
depending on how these are set in the various bbappends.

Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I9db63c3b66c1e02175528b224cc9f0d997d31272

mainloop.cpp

diff --git a/mainloop.cpp b/mainloop.cpp
index ecceee5..40c429a 100644
--- a/mainloop.cpp
+++ b/mainloop.cpp
@@ -527,7 +527,7 @@
         }
         catch (const std::system_error& e)
         {
-#ifdef UPDATE_FUNCTIONAL_ON_FAIL
+#if UPDATE_FUNCTIONAL_ON_FAIL
             // If UPDATE_FUNCTIONAL_ON_FAIL is defined and an exception was
             // thrown, set the functional property to false.
             // We cannot set this with the 'continue' in the lower block
@@ -553,7 +553,7 @@
                 }
                 continue;
             }
-#ifdef UPDATE_FUNCTIONAL_ON_FAIL
+#if UPDATE_FUNCTIONAL_ON_FAIL
             // Do not exit with failure if UPDATE_FUNCTIONAL_ON_FAIL is set
             continue;
 #endif