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 620310 - vpx plugin check headers in hardcodec location
vpx plugin check headers in hardcodec location
Status: RESOLVED NOTGNOME
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal normal
: NONE
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2010-06-01 21:09 UTC by Stefan Sauer (gstreamer, gtkdoc dev)
Modified: 2010-06-06 17:35 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Stefan Sauer (gstreamer, gtkdoc dev) 2010-06-01 21:09:07 UTC
The configure.ac check does:

AG_GST_CHECK_LIBHEADER(VPX, vpx,
                        vpx_codec_version, ,
                        vpvpx/vpx_codec.h,

while the opensuse packages (https://build.opensuse.org/package/show?package=libvpx&project=multimedia:libs) look like this:

> rpm -ql libvpx-devel
/usr/include/vp8.h
/usr/include/vp8cx.h
/usr/include/vp8dx.h
/usr/include/vp8e.h
/usr/include/vpx_codec.h
/usr/include/vpx_codec_impl_bottom.h
/usr/include/vpx_codec_impl_top.h
/usr/include/vpx_decoder.h
/usr/include/vpx_decoder_compat.h
/usr/include/vpx_encoder.h
/usr/include/vpx_image.h
/usr/include/vpx_integer.h
/usr/lib/libvpx.a

Ideally libvpx-devel would come with a pkg-config file.
Comment 1 Stefan Sauer (gstreamer, gtkdoc dev) 2010-06-01 21:20:01 UTC
Filed an upstream bug for suse as well
https://bugzilla.novell.com/show_bug.cgi?id=610773
Comment 2 Tim-Philipp Müller 2010-06-01 22:36:39 UTC
This looks like either a WONTFIX or NOTGNOME to me tbh.

Yes, upstream should provide a pkg-config .pc file. They hopefully will soon.

In the meantime Novell and others should either package things the way debian/ubuntu/fedora have chosen to do or patch their gst-plugins-bad configure; dumping 12 vp* files into /usr/include just doesn't seem right.
Comment 3 Benjamin Otte (Company) 2010-06-02 06:41:20 UTC
When filing bugs against package bugtrackers, it should be made clear that the /usr/include/vpx/ directory location was a decision by the Fedora, Ubuntu and Debian packagers that we hoped others (in particular upstream once they support make install) would adopt. It's neither the packager's fault nor upstream's when the headers are in the "wrong" directory.
Comment 4 David Schleef 2010-06-02 06:42:00 UTC
This was fixed a week+ ago in libvpx git.  We'd check for an updated version in a .pc file, but...
Comment 5 David Schleef 2010-06-02 06:44:50 UTC
To be clear, "fixed upstream" means that make install installs to ${prefix}/vpx/.
Comment 6 Tim-Philipp Müller 2010-06-02 07:49:18 UTC
> It's neither the packager's fault nor upstream's
> when the headers are in the "wrong" directory.

For me the question is whether we as a project want to encourage consistent packaging of this new library across distros while we have a chance, or if we want to encourage everyone doing 'their own' thing by accommodating them. (Who cares whose "fault" it is..)
Comment 7 Benjamin Otte (Company) 2010-06-02 07:54:14 UTC
It's kinda important to me, because I'd like to avoid the haters. And you get a lot of haters when you're the #1 distro and just decide things. And a "please do what others do" sounds a lot better than "your package is broken".
Comment 8 Sebastian Dröge (slomo) 2010-06-02 08:04:23 UTC
Erm, libvpx didn't have a real "make install" rule with the last release and it was only added recently to their GIT. The "make install" that was there before was more a "make dist".

I didn't decide the /usr/include/vpx location, in fact I've asked Google if that's what they intend to use in the future once they have a correct "make install".
Comment 9 Stefan Sauer (gstreamer, gtkdoc dev) 2010-06-06 17:35:23 UTC
Suse updated the package too. Everything fine now.