After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 743406 - Make sure the GESTextOverlayClip is registered on init
Make sure the GESTextOverlayClip is registered on init
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-editing-services
git master
Other Linux
: Normal normal
: 1.5.2
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2015-01-23 15:38 UTC by Kyrylo V. Polezhaiev
Modified: 2015-06-24 14:42 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Automatic update of common submodule (574 bytes, patch)
2015-01-24 09:55 UTC, Thibault Saunier
none Details | Review
ges: Make sure the GESTextOverlayClip is register on init (669 bytes, patch)
2015-01-24 09:56 UTC, Thibault Saunier
committed Details | Review

Description Kyrylo V. Polezhaiev 2015-01-23 15:38:33 UTC
GES documentation (http://gstreamer.freedesktop.org/data/doc/gstreamer/head/gstreamer-editing-services/html/GESTextOverlayClip.html) says:
>> GESTextOverlayClip implements GESExtractable and GESMetaContainer.
I want to use GESTextOverlayClip as asset in .xges file. May be thats wrong idea, but I got error in _parse_asset function at ges/ges-xml-formatter.c:
...
  extractable_type = g_type_from_name (extractable_type_name);
  if (extractable_type == G_TYPE_NONE)
    g_set_error (error, G_MARKUP_ERROR,
        G_MARKUP_ERROR_INVALID_CONTENT,
        "element '%s' invalid extractable_type %s'",
        element_name, extractable_type_name);
  else if (!g_type_is_a (extractable_type, GES_TYPE_EXTRACTABLE))
    g_set_error (error, G_MARKUP_ERROR,
        G_MARKUP_ERROR_INVALID_CONTENT,
        "element '%s', %s not an extractable_type'",
        element_name, extractable_type_name); // I see this message
  else {
...
May be some kind of interface implementation error?
Comment 1 Thibault Saunier 2015-01-23 15:41:09 UTC
It does implement the interface, can you give us more details about your code / xges file?
Comment 2 Kyrylo V. Polezhaiev 2015-01-24 00:08:20 UTC
Hello, you are so extremly fast :-)

Try to add something like <asset id='textoverlay' extractable-type-name='GESTextOverlayClip' properties='properties;' metadatas='metadatas;' /> into your resources section in any .xges file.
Comment 3 Thibault Saunier 2015-01-24 09:55:48 UTC
Created attachment 295315 [details] [review]
Automatic update of common submodule

From f2c6b95 to bc76a8b
Comment 4 Thibault Saunier 2015-01-24 09:56:34 UTC
Created attachment 295316 [details] [review]
ges: Make sure the GESTextOverlayClip is register on init

So it can be used when de serializing projects containing it.
Comment 5 Thibault Saunier 2015-01-24 09:58:14 UTC
Attachment 295316 [details] pushed as 7a9d39f - ges: Make sure the GESTextOverlayClip is register on init

Target Milestone: 1.5.1
Comment 6 Kyrylo V. Polezhaiev 2015-01-25 10:52:31 UTC
Nice :-) Thank you.
So, for workaround, can I temporary just register GESTextOverlayClip manually before parsing if I use older stable version of GES?
Comment 7 Kyrylo V. Polezhaiev 2015-01-25 13:05:28 UTC
I want to make temporary workaround for my old GES, tried to add get_type for this class after ges_init:

    gst_init(&argc, &argv);
    ges_init();
    GES_TYPE_TEXT_OVERLAY;

But I am still facing same error. What am I doing wrong?
Comment 8 Kyrylo V. Polezhaiev 2015-01-25 13:11:02 UTC
Oh, I am so sorry, was confused by similar class names GESTextOverlayClip and GESTextOverlay :-)
Please ignore previous message.