-
Notifications
You must be signed in to change notification settings - Fork 195
Cannot parse SDP message in python demo #25
Comments
This was just added in GIT master: https://bugzilla.gnome.org/show_bug.cgi?id=796563 It should use the old way in the demos IMHO, want to send a PR? Old way is res, msg = GstSdp.SDPMessage.new()
GstSdp.sdp_message_parse_buffer(bytes(text.encode()), msg) |
I tried that already, but it's greeting me with a good old:
|
Can you get a backtrace for that, or even better run in valgrind? @MathieuDuponchelle, any ideas? |
Here's backtrace gotten from gdb:
|
That's fixed by https://cgit.freedesktop.org/gstreamer/gst-plugins-bad/commit/gst-libs/gst/webrtc/rtcsessiondescription.c?id=0178354e6e8598ab14091103db832e23ee1bdfcc and will be in 1.14.2. @MathieuDuponchelle can we work around that on the Python side? |
When is that due? |
You have to change the |
Probably next week |
@sdroege , there's no workaround I can think of no, if it was a (mini)object the refcount could be manipulated from python but that's not the case :( |
Ok so then this should be documented in the repository :) |
It is: 9a26415 ;) |
Ok so what has to be done here still is to not use the new API from master (or get it backported for 1.14.2) and then document that 1.14.2 is needed for the Python example. Seems suboptimal to require GIT master :) |
Closing manually, this is addressed |
Hello, I just tried :
But I get :
I am using GStreamer 1.15.1 and I don't know if this issue is fixed or if there is a workaround ? Here a backtrace with gdb :
|
The function
SDPMessage.new_from_text
does not exist in my case?What python version is this using? On my system I have python3-gst-1.0 1.14.1-1 (from debian buster)
The text was updated successfully, but these errors were encountered: