Skip to content

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

2 participants