Skip to content
This repository was archived by the owner on Feb 8, 2021. It is now read-only.

Conversation

@WeiZhang555
Copy link
Contributor

Print git commit hash id when run runv -v, this will be helpful for
debug.

Signed-off-by: Zhang Wei zhangwei555@huawei.com

Examples:

$ ./runv -v
runv version 0.7.0, commit: v0.7.0-145-g23cbab6-dirty

Means version: 0.7.0, latest tag is v0.7.0, we are 145 commits forward from that tag, current git commit is 23cbab6, dirty means we have uncommited changes in repo when building the binary.

@WeiZhang555
Copy link
Contributor Author

WeiZhang555 commented Mar 15, 2017

Another fix is, after changing the Makefile, runv is statically built binary now, before this, it's still dynamic binary.

Oh no, libvirt-dev doesn't have static library, so revert to dynamicallly built binary

@WeiZhang555 WeiZhang555 force-pushed the add-commit-to-version branch 3 times, most recently from 08b4d85 to 0111778 Compare March 22, 2017 02:28
Makefile.am Outdated
endif

COMMIT=`git describe --dirty --always --tags 2> /dev/null || true`
VERSION=`cat ./VERSION`
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this ./VERSION exist?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, I added a "VERSION" file in this commit.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@gao-feng Rebased.

Copy link
Contributor

@gao-feng gao-feng May 18, 2017

Choose a reason for hiding this comment

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

@WeiZhang555 WeiZhang555 force-pushed the add-commit-to-version branch from 0111778 to c8c1b83 Compare May 17, 2017 15:24
Print git commit hash id when run `runv -v`, this will be helpful for
debug.

Signed-off-by: Zhang Wei <zhangwei555@huawei.com>
@WeiZhang555 WeiZhang555 force-pushed the add-commit-to-version branch from c8c1b83 to 8627454 Compare May 18, 2017 03:45
@WeiZhang555
Copy link
Contributor Author

@gao-feng You're right, update now.

@gao-feng
Copy link
Contributor

LGTM, thanks!

@gao-feng gao-feng merged commit cbdb45e into hyperhq:master May 18, 2017
jimoosciuc pushed a commit to jimoosciuc/runv that referenced this pull request May 26, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants