build system: more accurate error information for git_describe

Closes https://github.com/espressif/esp-idf/issues/9071

See merge request espressif/esp-idf!23891
This commit is contained in:
Zim Kalinowski 2023-05-23 18:29:41 +02:00
parent d41ddfcf8a
commit 369d8a0f8a
2 changed files with 9 additions and 4 deletions

View File

@ -79,8 +79,7 @@ function(__project_get_revision var)
if(PROJECT_VER_GIT)
set(PROJECT_VER ${PROJECT_VER_GIT})
else()
message(STATUS "Project is not inside a git repository, or git repository has no commits;"
" will not use 'git describe' to determine PROJECT_VER.")
message(STATUS "Could not use 'git describe' to determine PROJECT_VER.")
set(PROJECT_VER 1)
endif()
endif()

View File

@ -53,10 +53,13 @@ function(get_git_head_revision _refspecvar _hashvar _repo_dir)
res
OUTPUT_VARIABLE
GIT_DIR
ERROR_QUIET
ERROR_VARIABLE
error
OUTPUT_STRIP_TRAILING_WHITESPACE)
if(NOT res EQUAL 0)
string(STRIP "${error}" error)
message(STATUS "git rev-parse returned '${error}'")
return()
endif()
@ -123,9 +126,12 @@ function(git_describe _var _repo_dir)
res
OUTPUT_VARIABLE
out
ERROR_QUIET
ERROR_VARIABLE
error
OUTPUT_STRIP_TRAILING_WHITESPACE)
if(NOT res EQUAL 0)
string(STRIP "${error}" error)
message(STATUS "git describe returned '${error}'")
set(out "${out}-${res}-NOTFOUND")
endif()