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 779619 - new libmediaart 1.9.1 symbols look wrong
new libmediaart 1.9.1 symbols look wrong
Status: RESOLVED OBSOLETE
Product: libmediaart
Classification: Other
Component: Cache
1.9.x
Other Linux
: Normal normal
: ---
Assigned To: libmediaart maintainer(s)
libmediaart maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2017-03-05 16:00 UTC by Jeremy Bicha
Modified: 2021-05-25 11:25 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Jeremy Bicha 2017-03-05 16:00:41 UTC
These new symbols look wrong to me (the extra _VOID__STRING* stuff)

  media_art_file_to_jpeg@Base 1.9.0
  media_art_get_file@Base 1.9.0
  media_art_get_path@Base 1.9.0
+ media_art_marshal_VOID__STRING_STRING@Base 1.9.1
+ media_art_marshal_VOID__STRING_STRING_STRING_BOOLEAN_BOOLEAN@Base 1.9.1
  media_art_plugin_init@Base 1.9.0
  media_art_plugin_shutdown@Base 1.9.0
  media_art_process_buffer@Base 1.9.0
Comment 1 Michael Biebl 2017-03-05 16:18:47 UTC
Those are generated by marshal.list. I would suggest using a underscore like
--prefix=_media_art_marshal so those symbols are hidden
Comment 2 Michael Biebl 2017-03-05 16:19:27 UTC
Oh, I see my change was reverted in 52eb6498c97ce0005186946bc9162d87597abc2c
Comment 3 Michael Biebl 2017-03-05 16:21:51 UTC
(In reply to Michael Biebl from comment #2)
> Oh, I see my change was reverted in 52eb6498c97ce0005186946bc9162d87597abc2c

Well, partially at least

@Martin which build failure did you run into? We shipped this patch [1] in Debian for a while without problems.

[1] https://anonscm.debian.org/cgit/collab-maint/libmediaart.git/tree/debian/patches/0001-build-hide-private-symbols.patch
Comment 4 Michael Biebl 2017-03-05 16:35:49 UTC
One issue I notice (which was probably an oversight on my part) is:

./libmediaart-sections.txt:44: warning: No declaration found for media_art_marshal_VOID__STRING_STRING.
./libmediaart-sections.txt:45: warning: No declaration found for media_art_marshal_VOID__STRING_STRING_STRING_BOOLEAN_BOOLEAN.
See libmediaart-undeclared.txt for the list of undeclared symbols.

I would remove that section from docs/reference/libmediaart/libmediaart-sections.txt:

<SECTION>
<FILE>marshal</FILE>
media_art_marshal_VOID__STRING_STRING
media_art_marshal_VOID__STRING_STRING_STRING_BOOLEAN_BOOLEAN
</SECTION>
Comment 5 Jens Georg 2017-04-17 11:22:12 UTC
Can't we just use the FFI marshaller here?
Comment 6 Michael Biebl 2017-06-30 10:19:52 UTC
Martin, can you chime in here please?
Comment 7 André Klapper 2021-05-25 11:25:43 UTC
GNOME is going to shut down bugzilla.gnome.org in favor of gitlab.gnome.org.
As part of that, we are mass-closing older open tickets in bugzilla.gnome.org
which have not seen updates for a longer time (resources are unfortunately
quite limited so not every ticket can get handled).

If you can still reproduce the situation described in this ticket in a recent
and supported software version, then please follow
  https://wiki.gnome.org/GettingInTouch/BugReportingGuidelines
and create a new enhancement request ticket at
  https://gitlab.gnome.org/GNOME/libmediaart/-/issues/

Thank you for your understanding and your help.