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 768595 - lv2: Does not build with lilv 0.20
lv2: Does not build with lilv 0.20
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal normal
: 1.9.2
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2016-07-08 21:08 UTC by Nicolas Dufresne (ndufresne)
Modified: 2016-07-14 17:41 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Nicolas Dufresne (ndufresne) 2016-07-08 21:08:12 UTC
The lv2 plugin does not build against lilv 0.20 after:

commit adf2092b3da83d837001711614570f6b9996f538
Author: Stefan Sauer <ensonic@users.sf.net>
Date:   Sat Jul 2 10:24:51 2016 +0200

    lv2: add support for saving presets
Comment 1 Stefan Sauer (gstreamer, gtkdoc dev) 2016-07-09 20:07:46 UTC
I'll downgrade mine, but the next time, please post the errors you get.
Comment 2 Stefan Sauer (gstreamer, gtkdoc dev) 2016-07-09 20:10:55 UTC
I've been building against 0.22.1
Comment 3 Nicolas Dufresne (ndufresne) 2016-07-10 15:50:25 UTC
It is just that you use newly introduced API. You and Thibault just need to agree in bumping the requirement or dropping this patch.
Comment 4 Stefan Sauer (gstreamer, gtkdoc dev) 2016-07-10 19:06:33 UTC
commit 13d963fbf0223910cff55dae1f2bd0b653b19394
Author: Stefan Sauer <ensonic@users.sf.net>
Date:   Sun Jul 10 20:42:42 2016 +0200

    lv2: keep working with older versions
    
    Provide a fallback impl. for the only new function we were using from 0.22.
    Fixes #768595
Comment 5 Nicolas Dufresne (ndufresne) 2016-07-14 03:24:37 UTC
Not yet apparently:

  CC       libgstlv2_la-gstlv2utils.lo
gstlv2utils.c: In function 'gst_lv2_delete_preset':
gstlv2utils.c:364:38: error: implicit declaration of function 'lilv_state_get_uri' [-Werror=implicit-function-declaration]
   lilv_world_unload_resource (world, lilv_state_get_uri (state));
                                      ^~~~~~~~~~~~~~~~~~
gstlv2utils.c:364:3: error: nested extern declaration of 'lilv_state_get_uri' [-Werror=nested-externs]
   lilv_world_unload_resource (world, lilv_state_get_uri (state));
   ^~~~~~~~~~~~~~~~~~~~~~~~~~
gstlv2utils.c:364:38: error: passing argument 2 of 'lilv_world_unload_resource' makes pointer from integer without a cast [-Werror=int-conversion]
   lilv_world_unload_resource (world, lilv_state_get_uri (state));
                                      ^~~~~~~~~~~~~~~~~~
In file included from gstlv2.h:26:0,
                 from gstlv2utils.c:28:
/usr/include/lilv-0/lilv/lilv.h:580:1: note: expected 'const LilvNode * {aka const struct LilvNodeImpl *}' but argument is of type 'int'
 lilv_world_unload_resource(LilvWorld*      world,
 ^~~~~~~~~~~~~~~~~~~~~~~~~~
gstlv2utils.c:365:3: error: implicit declaration of function 'lilv_state_delete' [-Werror=implicit-function-declaration]
   lilv_state_delete (world, state);
   ^~~~~~~~~~~~~~~~~
gstlv2utils.c:365:3: error: nested extern declaration of 'lilv_state_delete' [-Werror=nested-externs]
cc1: all warnings being treated as errors
Makefile:919: recipe for target 'libgstlv2_la-gstlv2utils.lo' failed
make: *** [libgstlv2_la-gstlv2utils.lo] Error 1
Comment 6 Stefan Sauer (gstreamer, gtkdoc dev) 2016-07-14 17:41:31 UTC
commit ea443a3fcc4e5cdd76b986b594f0d653eb72f6a5
Author: Stefan Sauer <ensonic@users.sf.net>
Date:   Thu Jul 14 19:38:26 2016 +0200

    lv2: make it build with older lilv
    
    Implementing the removal of presets is unfortunately not feasibla with
    versions < 0.22.
    Fixes #768595