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 git describe --tags on topic branches #136

Merged
merged 1 commit into from
Nov 1, 2018
Merged

Fix git describe --tags on topic branches #136

merged 1 commit into from
Nov 1, 2018

Conversation

native-api
Copy link
Contributor

If there are no tags on a branch, git describe --tags in find_version.py produces an error.

Was like this in Travis for https://github.com/native-api/opencv-python/tree/readd_ipp_linux , see https://travis-ci.org/native-api/opencv-python/jobs/447134765 for a sample failure.

@native-api
Copy link
Contributor Author

native-api commented Oct 28, 2018

I also considered using git describe --tags --always which never fails and produces a shortened hash in this case.

Rejected that because in makes it impossible to distinguish between no tags and a tag that is named the same as a short hash (a remote possibility but still a defect that we can do without).

@native-api
Copy link
Contributor Author

native-api commented Oct 29, 2018

Travis failures are due to the bug fixed in #135 . I could rebase the branch to the new master after that is merged to rerun the checks...

@skvark
Copy link
Member

skvark commented Oct 29, 2018

Thanks. I merged the other PR so you can rebase.

@native-api
Copy link
Contributor Author

Linux builds are now okay; MacOS fails are unrelated (ffmpeg build timeout).

@skvark
Copy link
Member

skvark commented Nov 1, 2018

Yeah, thanks. I'll merge this, but we have to figure out what's going on with the FFmpeg builds.

@skvark skvark merged commit 77a3278 into opencv:master Nov 1, 2018
@native-api native-api deleted the find_version_topic_branches branch November 1, 2018 17:54
@native-api
Copy link
Contributor Author

we have to figure out what's going on with the FFmpeg builds.

#139

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