From ba6058ba588313fba91d1f82ab00cc7aac835cd4 Mon Sep 17 00:00:00 2001
From: Renz Christian Bagaporo <renz@espressif.com>
Date: Thu, 29 Nov 2018 14:51:21 +0800
Subject: [PATCH] cmake: fix error on building idf as lib when env IDF_PATH is
 not set

---
 tools/cmake/idf_functions.cmake | 15 +++++++++++----
 tools/cmake/project.cmake       | 12 +-----------
 2 files changed, 12 insertions(+), 15 deletions(-)

diff --git a/tools/cmake/idf_functions.cmake b/tools/cmake/idf_functions.cmake
index 65ba66351e..5cf0f18ddd 100644
--- a/tools/cmake/idf_functions.cmake
+++ b/tools/cmake/idf_functions.cmake
@@ -2,13 +2,20 @@
 # Load cmake modules
 #
 
-if(NOT IDF_PATH)
-    set(IDF_PATH $ENV{IDF_PATH})
-endif()
-
 get_property(__idf_environment_set GLOBAL PROPERTY __IDF_ENVIRONMENT_SET)
 
 if(NOT __idf_environment_set)
+
+    # Set IDF_PATH, as nothing else will work without this.
+    set(IDF_PATH "$ENV{IDF_PATH}")
+    if(NOT IDF_PATH)
+        # Documentation says you should set IDF_PATH in your environment, but we
+        # can infer it relative to tools/cmake directory if it's not set.
+        get_filename_component(IDF_PATH "${CMAKE_CURRENT_LIST_DIR}/../.." ABSOLUTE)
+    endif()
+    file(TO_CMAKE_PATH "${IDF_PATH}" IDF_PATH)
+    set(ENV{IDF_PATH} ${IDF_PATH})
+
     set(CMAKE_MODULE_PATH
         "${IDF_PATH}/tools/cmake"
         "${IDF_PATH}/tools/cmake/third_party"
diff --git a/tools/cmake/project.cmake b/tools/cmake/project.cmake
index e2aa7c19fa..04459a1e83 100644
--- a/tools/cmake/project.cmake
+++ b/tools/cmake/project.cmake
@@ -2,15 +2,7 @@
 #
 cmake_minimum_required(VERSION 3.5)
 
-# Set IDF_PATH, as nothing else will work without this.
-set(IDF_PATH "$ENV{IDF_PATH}")
-if(NOT IDF_PATH)
-    # Documentation says you should set IDF_PATH in your environment, but we
-    # can infer it relative to tools/cmake directory if it's not set.
-    get_filename_component(IDF_PATH "${CMAKE_CURRENT_LIST_DIR}/../.." ABSOLUTE)
-endif()
-file(TO_CMAKE_PATH "${IDF_PATH}" IDF_PATH)
-set(ENV{IDF_PATH} ${IDF_PATH})
+include(${CMAKE_CURRENT_LIST_DIR}/idf_functions.cmake)
 
 # Set the path of idf.py.
 set(IDFTOOL ${PYTHON} "${IDF_PATH}/tools/idf.py")
@@ -24,8 +16,6 @@ endfunction()
 function(_project)
 endfunction()
 
-include(${IDF_PATH}/tools/cmake/idf_functions.cmake)
-
 macro(project name)
 
     # Bridge existing documented variable names with library namespaced variables in order for old projects to work.