Bug#739269: Please move libwayland-egl out of libegl1-mesa-drivers
On Sun, 2014-03-02 at 20:34 +0100, Julien Cristau wrote:
> On Sun, Mar 2, 2014 at 19:09:50 +0100, Julien Cristau wrote:
>
> > On Mon, Feb 17, 2014 at 14:50:48 +0100, Sjoerd Simons wrote:
> >
> > > diff --git a/debian/control b/debian/control
> > > index 4577639..de02e96 100644
> > > --- a/debian/control
> > > +++ b/debian/control
> > > @@ -308,6 +308,7 @@ Depends:
> > > ${misc:Depends},
> > > libegl1-mesa (= ${binary:Version}),
> > > libglapi-mesa (= ${binary:Version}),
> > > + libwayland-egl1-mesa (= ${binary:Version}),
> > > # for libllvmradeon
> > > libgl1-mesa-dri (= ${binary:Version}) [any-i386 any-amd64],
> > > Pre-Depends: ${misc:Pre-Depends}
> > > @@ -320,6 +321,41 @@ Description: free implementation of the EGL API -- hardware drivers
> > > This package contains the drivers required for hardware accelerated rendering
> > > of EGL-based graphics libraries, such as OpenGL|ES and OpenVG.
> > >
> > > +Package: libwayland-egl1-mesa
> > > +Section: libs
> > > +Architecture: linux-any
> > > +Depends:
> > > + ${shlibs:Depends},
> > > + ${misc:Depends},
> > > + libegl1-mesa (= ${binary:Version})
> > > +Recommends: libegl1-mesa-drivers
> > > +Provides: libwayland-egl
> > > +Conflicts: libwayland-egl1
> > > +Replaces: libwayland-egl1
> >
> > This looks broken. If this new package contains files split out of
> > libegl1-mesa-drivers then it needs to replace its old versions, at the
> > very least. Can somebody please fix this (and test the upgrade path)?
> >
> > > +Pre-Depends: ${misc:Pre-Depends}
> > > +Multi-Arch: same
> > > +Description: free implementation of the EGL API -- runtime
> > > + This package contains the EGL native platform graphics interface library.
> > > + EGL provides a platform-agnostic mechanism for creating rendering surfaces
> > > + for use with other graphics libraries, such as OpenGL|ES and OpenVG.
> > > + .
> > > + This package contains wayland specific interface for use with EGL.
> > > +
>
> Also the short description is the one from libegl, which is not
> appropriate here.
>
> And I'm thinking splitting out a -dev package might be useful if this
> lib is supposed to be used externally. If not, we shouldn't ship a .pc
> file and a .so symlink.
It's meant to be used externally indeed. Any Wayland application that
uses EGL/GLES2 should use it to e.g. transform a wayland surface into an
EGLNativeWindow for use with the EGL stack.
I did keep the .so/.pc etc in libegl1-mesa-dev (as they are currently)
as libwayland-egl is both conceptually part of the EGL stack and tiny
(the .h is not even 60 lines). I'm happy to update the patch to split it
out into its own package as well though (as well as correct the other
issues you've noted)
--
Sjoerd Simons <sjoerd@debian.org>
Reply to: