Skip to content

Set git_version when .git folder is missing#116

Merged
ArthurSonzogni merged 1 commit intoArthurSonzogni:masterfrom
myd7349:fix-git-version
Jun 17, 2021
Merged

Set git_version when .git folder is missing#116
ArthurSonzogni merged 1 commit intoArthurSonzogni:masterfrom
myd7349:fix-git-version

Conversation

@myd7349
Copy link
Contributor

@myd7349 myd7349 commented Jun 16, 2021

If we download the FTXUI source archive from the GitHub website, then the .git folder will be missing.
In such case, CMake configuration will fail:

git found
fatal: not a git repository (or any of the parent directories): .git
CMake Error at CMakeLists.txt:21 (project):
VERSION "0.6." format invalid.

If we download the FTXUI source archive from the GitHub website, then the `.git` folder will be missing.
In such case, CMake configuration will fail:
> git found
> fatal: not a git repository (or any of the parent directories): .git
> CMake Error at CMakeLists.txt:21 (project):
>   VERSION "0.6." format invalid.
Copy link
Owner

@ArthurSonzogni ArthurSonzogni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks!

@myd7349
Copy link
Contributor Author

myd7349 commented Jun 16, 2021

Please do not merge yet. Let's see if this patch can fix Linux and macOS build errors encountered in microsoft/vcpkg#18360.
(vcpkg's CI is still hard working.)

@myd7349
Copy link
Contributor Author

myd7349 commented Jun 16, 2021

Hi. @ArthurSonzogni I think this PR is ready now.

@ArthurSonzogni ArthurSonzogni merged commit 3d5e4eb into ArthurSonzogni:master Jun 17, 2021
@myd7349 myd7349 deleted the fix-git-version branch June 17, 2021 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants