Skip to content

Run git from the source directory (#589)#590

Merged
dmah42 merged 1 commit intogoogle:masterfrom
php1ic:fix-GetGitVersion
May 8, 2018
Merged

Run git from the source directory (#589)#590
dmah42 merged 1 commit intogoogle:masterfrom
php1ic:fix-GetGitVersion

Conversation

@php1ic
Copy link
Copy Markdown
Contributor

@php1ic php1ic commented May 6, 2018

Git was being executed in the current directory, so could not get the
latest tag if cmake was run from a build directory. Force git to be
run from with the source directory.

Git was being executed in the current directory, so could not get the
latest tag if cmake was run from a build directory. Force git to be
run from with the source directory.
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 87.115% when pulling d7c954a on php1ic:fix-GetGitVersion into 8986839 on google:master.

@AppVeyorBot
Copy link
Copy Markdown

Build benchmark 1216 completed (commit 7ed1818809 by @php1ic)

@dmah42
Copy link
Copy Markdown
Member

dmah42 commented May 8, 2018

that's odd... i've been running cmake from a build directory for ever:

$ cd _build
$ cmake ../

and it was working fine.

I don't think this breaks anything, but i'm curious what other differences you might have on your platform.

@dmah42 dmah42 merged commit 16af645 into google:master May 8, 2018
@dmah42
Copy link
Copy Markdown
Member

dmah42 commented May 8, 2018

oh i see.. your build directory is parallel to the source directory :)

@php1ic php1ic deleted the fix-GetGitVersion branch May 11, 2018 13:36
JBakamovic pushed a commit to JBakamovic/benchmark that referenced this pull request Dec 6, 2018
Git was being executed in the current directory, so could not get the
latest tag if cmake was run from a build directory. Force git to be
run from with the source directory.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants