fix(userspace/libsinsp): missing ifndef for minimal build in addrinfo

Co-authored-by: Leonardo Grasso <me@leonardograsso.com> Signed-off-by: Leonardo Di Donato <leodidonato@gmail.com>
draios · Sep 10, 2020 · 46b90ad · 46b90ad
1 parent e3713a3
commit 46b90ad
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/userspace/libsinsp/CMakeLists.txt b/userspace/libsinsp/CMakeLists.txt
@@ -22,7 +22,6 @@ include_directories(../async)
 include_directories(./include)
 include_directories("${JSONCPP_INCLUDE}")
 include_directories("${LUAJIT_INCLUDE}")
-include_directories("${CARES_INCLUDE}")
 
 if(NOT WIN32 AND NOT APPLE)
 	include_directories("${B64_INCLUDE}")
@@ -32,6 +31,7 @@ if(NOT WIN32 AND NOT APPLE)
 		include_directories("${PROTOBUF_INCLUDE}")
 		include_directories("${OPENSSL_INCLUDE_DIR}")
 		include_directories("${CURL_INCLUDE_DIR}")
+		include_directories("${CARES_INCLUDE}")
 	endif() # NOT MINIMAL_BUILD
 	include_directories("${JQ_INCLUDE}")
 	include_directories("${CMAKE_CURRENT_BINARY_DIR}")

diff --git a/userspace/libsinsp/addrinfo.cpp b/userspace/libsinsp/addrinfo.cpp
@@ -15,7 +15,7 @@ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 See the License for the specific language governing permissions and
 limitations under the License.
 */
-
+#ifndef MINIMAL_BUILD
 #include "addrinfo.h"
 
 #include <ares.h>
@@ -38,3 +38,4 @@ void ares_cb(void *arg, int status, int timeouts, struct hostent *host)
         res->done = true;
     }
 }
+#endif // MINIMAL_BUILD