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

Gst-base: fix version for gst_flow_combiner_ref/unref #5

Closed
wants to merge 1 commit into from
Closed

Gst-base: fix version for gst_flow_combiner_ref/unref #5

wants to merge 1 commit into from

Conversation

fengalin
Copy link
Contributor

gst_flow_combiner_ref/unref were introduced in gstreamer-1.12.1 but they inherit the since version declared for GstFlowCombiner which is 1.4.

See also: https://bugzilla.gnome.org/show_bug.cgi?id=788778

@sdroege
Copy link
Owner

sdroege commented Oct 10, 2017

Please squash both commits together. This also requires changes in gstreamer-rs though. Instead of requiring 1.12.1 for usage of gst_base::FlowCombiner, I'd like to use g_boxed_copy and g_boxed_free instead though. See gst::ParseContext for an example of that.

Do you want to provide a PR for that too?

@fengalin
Copy link
Contributor Author

Yes, I will. I was just wondering how to handle this as Gir wouldn't use the version while generating gst_base::FlowCombiner. I'll take a look at gst::ParseContext.

gst_flow_combiner_ref/unref were introduced in gstreamer-1.12.1 but they inherit the `since version` declared for `GstFlowCombiner` which is 1.4.
@sdroege
Copy link
Owner

sdroege commented Oct 10, 2017

It unfortunately means that the whole type has to be done manually. Thanks! :)

Copy link
Owner

@sdroege sdroege left a comment

Choose a reason for hiding this comment

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

Thanks, looks good to me

@sdroege sdroege closed this in 4b228f5 Oct 11, 2017
@fengalin fengalin deleted the combiner_ref_version branch October 13, 2017 16:52
JayDouglass pushed a commit to angeleye/gir-files-rs that referenced this pull request Jan 6, 2022
gst_flow_combiner_ref/unref were introduced in gstreamer-1.12.1 but they
inherit the `since version` declared for `GstFlowCombiner` which is 1.4.

Fixes sdroege/gstreamer-sys#5
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