[Bug 1025716] Re: Package cedarview-graphics-drivers

[Steve Langasek]

Hi Jani,

Minor nit with the packaging:

override_dh_auto_configure:
        echo "/usr/lib/intel-cdv" > usr/lib/ld.so.conf

This is never cleaned up; so if you try to build the binary package and
then build the source package from the same directory, the source build
will fail with this error:

dpkg-source: info: local changes detected, the modified files are:
 cedarview-graphics-drivers-20120608/usr/lib/ld.so.conf

Could you add usr/lib/ld.so.conf to debian/clean in a subsequent upload
to quantal?

There's a bigger bug with the dh_shlibdeps override, though:

E: cedarview-graphics-drivers: missing-dependency-on-libc needed by
usr/lib/i386-linux-gnu/dri/pvr_dri.so and 21 others

I think the dh_shlibdeps invocation you want in debian/rules is this:

override_dh_shlibdeps:
        dh_shlibdeps -l$(CURDIR)/debian/cedarview-graphics-drivers/usr/lib/intel-cdv/ -- --warnings=6

+ a couple extra build-dependencies on the runtime libs.

(btw, the original file had buggy whitespace, so dh_shlibdeps wasn't
being called at all...)

however, even with this, dpkg-shlibdeps is returning errors:
dpkg-shlibdeps: error: couldn't find library libGL.so.1 needed by debian/cedarview-graphics-drivers/usr/lib/intel-cdv/dri/pvr_dri.so (ELF format: 'elf32-i386'; RPATH: '').
[...]
dpkg-shlibdeps: error: couldn't find library libGL.so.1 needed by debian/cedarview-graphics-drivers/usr/lib/i386-linux-gnu/dri/pvr_dri.so (ELF format: 'elf32-i386'; RPATH: '').

(which also points to a duplicated file within the package)

This seems very strange that a package providing GLES should be linked
to a libGL.so that it doesn't provide.  Do you know if this is correct?
If it is, what package should the build-dependency be against?

If I -X the pvr_dri.so, I get the following set of shlibdeps:

libc6 (>= 2.15), libdrm2 (>= 2.4.16), libgcc1 (>= 1:4.1.1),
libpciaccess0, libpixman-1-0, libstdc++6 (>= 4.1.1), libudev0 (>= 147),
libwsbm1, libx11-6, libxext6, libxfixes3,

That's a non-trivial list, so I think the package should really be fixed
to call dh_shlibdeps correctly.

Attached is a prospective fix for this.  Can you look this over and see
if this makes sense to apply to the package?  If so, please reupload
with these changes included.

** Patch added: "cedarview-graphics-drivers-fixes.diff"
   https://bugs.launchpad.net/ubuntu/+bug/1025716/+attachment/3239146/+files/cedarview-graphics-drivers-fixes.diff

-- 
You received this bug notification because you are a member of Ubuntu
Package Archive Administrators, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/1025716

Title:
  Package cedarview-graphics-drivers

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+bug/1025716/+subscriptions