Skip to content

gstreamer: add video module#19633

Merged
opencv-pushbot merged 1 commit intoopencv:masterfrom
mshabunin:add-gstreamer-video
Feb 26, 2021
Merged

gstreamer: add video module#19633
opencv-pushbot merged 1 commit intoopencv:masterfrom
mshabunin:add-gstreamer-video

Conversation

@mshabunin
Copy link
Copy Markdown
Contributor

@mshabunin mshabunin commented Feb 26, 2021 •
edited
Loading

related #19609

GstVideoInfo is located in the Video library which was not used directly in our cmake scripts, it is implicitly added to the build as a dependency of the pbutils library. But on some platforms (versions?) video library is not included in the dependency list of pbutils and should be added explicitly. Otherwise linking error would occur:

modules/videoio/CMakeFiles/opencv_videoio_gstreamer.dir/src/cap_gstreamer.cpp.o: In function `cv::GStreamerCapture::retrieveFrame(int, cv::_OutputArray const&)':
cap_gstreamer.cpp:(.text._ZN2cv16GStreamerCapture13retrieveFrameEiRKNS_12_OutputArrayE+0xc6): undefined reference to `gst_video_info_from_caps'

Debian 9, cross-compilation for ARM:

$ egrep 'Requires|Version' /usr/lib/arm-linux-gnueabihf/pkgconfig/gstreamer-pbutils-1.0.pc
Requires: gstreamer-1.0
Version: 1.10.4

Ubuntu 18:

$ egrep 'Requires|Version' /usr/lib/x86_64-linux-gnu/pkgconfig/gstreamer-pbutils-1.0.pc
Requires: gstreamer-1.0 gstreamer-audio-1.0 gstreamer-base-1.0 gstreamer-video-1.0
Requires.private: gstreamer-tag-1.0
Version: 1.14.5

Note: also added for Windows/manual-search case, but did not test it

alalek
alalek approved these changes Feb 26, 2021
View reviewed changes
Copy link
Copy Markdown
Member

@alalek alalek left a comment

Choose a reason for hiding this comment

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

Thank you 👍

@opencv-pushbot opencv-pushbot merged commit c7f0381 into opencv:master Feb 26, 2021
@mshabunin mshabunin added the backport is needed Label for maintainers. Authors of PR can ignore this label Feb 26, 2021
@mshabunin mshabunin deleted the add-gstreamer-video branch February 26, 2021 12:22
@alalek alalek mentioned this pull request Apr 9, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alalek alalek alalek approved these changes

Assignees

@alalek alalek

Labels

backport is needed Label for maintainers. Authors of PR can ignore this bug category: build/install category: videoio

Projects

None yet

Milestone

4.5.2

Development

Successfully merging this pull request may close these issues.

3 participants

@mshabunin @alalek @opencv-pushbot