Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix patch version when running CMake outside the source tree #2

Closed
wants to merge 1 commit into from

Conversation

dbermond
Copy link

Determination of patch version fails when running CMake outside the source tree.

For example, this currently fails to determine the patch version:

$ git clone https://github.com/oneapi-src/level-zero.git
$ cmake -B build -S level-zero
$ make -C build

To fix this, we must ensure that the git command runs in the ${CMAKE_SOURCE_DIR} directory. This can be done by specifying the source tree with the git -C option.

Determination of patch version fails when running CMake outside
the source tree.

For example, this currently fails to determine the patch version:
$ git clone https://github.com/oneapi-src/level-zero.git
$ cmake -B build -S level-zero
$ make -C build

To fix this, we must ensure that the git command runs in the
${CMAKE_SOURCE_DIR} directory. This can be done by specifying
the source tree with the git '-C' option.
@rwmcguir rwmcguir requested a review from a user March 23, 2020 20:39
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Thanks! We have a very similar patch for this queued for pushing upstream. I will update once it is visible in this repository.

@ghost
Copy link

ghost commented Mar 27, 2020

We recently pushed 1403f9f which should resolve this issue. If it does, you can close this pull request.

@dbermond
Copy link
Author

Yes, commit 1403f9f fixed this. Thanks.

@dbermond dbermond closed this Mar 28, 2020
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.

1 participant