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 797320 - Skip C-specific constants from gir generation
Skip C-specific constants from gir generation
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
unspecified
Other Linux
: Normal normal
: 1.15.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2018-10-22 13:20 UTC by Jordan Petridis
Modified: 2018-10-23 15:24 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch to skip gst-error-system from gir generation (811 bytes, patch)
2018-10-22 13:20 UTC, Jordan Petridis
committed Details | Review
skip format specifier from gir generation (1.12 KB, patch)
2018-10-22 13:21 UTC, Jordan Petridis
none Details | Review
skip gst strime format from gir generation (935 bytes, patch)
2018-10-22 13:21 UTC, Jordan Petridis
none Details | Review
Skip all the format specifiers requested (3.39 KB, patch)
2018-10-23 10:06 UTC, Jordan Petridis
committed Details | Review

Description Jordan Petridis 2018-10-22 13:20:23 UTC
Created attachment 373998 [details] [review]
patch to skip gst-error-system from gir generation

The following constants are not useful outside of C and currently cause some tests of the rust bindings to fail.

GST_ERROR_SYSTEM
GST_PTR_FORMAT
GST_SEGMENT_FORMAT
GST_STIME_FORMAT
Comment 1 Jordan Petridis 2018-10-22 13:21:05 UTC
Created attachment 373999 [details] [review]
skip format specifier from gir generation
Comment 2 Jordan Petridis 2018-10-22 13:21:36 UTC
Created attachment 374000 [details] [review]
skip gst strime format from gir generation
Comment 3 Sebastian Dröge (slomo) 2018-10-22 13:40:12 UTC
Review of attachment 373999 [details] [review]:

There's also GST_TIME_FORMAT, GST_TIME_ARGS, GST_STIME_FORMAT, GST_STIME_ARGS, GST_FOURCC_FORMAT
Comment 4 Sebastian Dröge (slomo) 2018-10-22 13:40:43 UTC
Review of attachment 373999 [details] [review]:

And GST_FOURCC_ARGS
Comment 5 Sebastian Dröge (slomo) 2018-10-22 13:41:34 UTC
Review of attachment 374000 [details] [review]:

See comment about the second patch. These second and this patch should be merged into one, it's all the same problem :)
Comment 6 Jordan Petridis 2018-10-23 10:06:54 UTC
Created attachment 374011 [details] [review]
Skip all the format specifiers requested
Comment 7 Sebastian Dröge (slomo) 2018-10-23 15:23:59 UTC
commit 9251cd9bf502aa2f218b8cfc65e85439133e4a05 (HEAD -> master, origin/master, origin/HEAD)
Author: Jordan Petridis <jordan@centricular.com>
Date:   Tue Oct 16 19:35:03 2018 +0300

    gst: skip format specifiers from gir generation
    
    GST_TIME_FORMAT, GST_TIME_ARGS, GST_STIME_FORMAT, GST_STIME_ARGS
    GST_PTR_FORMAT, GST_SEGMENT_FORMAT, GST_FOURCC_FORMAT and
    GST_FOURCC_ARGS are format specifiers.
    They can't be used outside of C and should be generated in the gir.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=797320

commit cd6ceb1c41729fcedb0baaae94f60f477d700fe2
Author: Jordan Petridis <jordan@centricular.com>
Date:   Tue Oct 16 19:35:03 2018 +0300

    gst/gsterror.h: skip GST_ERROR_SYSTEM during gir generation
    
    GST_ERROR_SYSTEM can't really be used outside of C and should
    be skipped.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=797320