[meta-intel] emgd-driver-dev and libgl-mesa-dev packaging conflict

Fri Apr 19 15:29:04 PDT 2013

On Fri, 2013-04-19 at 17:10 -0500, Kamble, Nitin A wrote:
> 
> > -----Original Message-----
> > From: Zanussi, Tom
> > Sent: Friday, April 19, 2013 2:55 PM
> > To: Kamble, Nitin A
> > Cc: Darren Hart; meta-intel at yoctoproject.org
> > Subject: Re: [meta-intel] emgd-driver-dev and libgl-mesa-dev packaging
> > conflict
> > 
> > On Fri, 2013-04-19 at 16:33 -0500, Kamble, Nitin A wrote:
> > >
> > > > -----Original Message-----
> > > > From: Zanussi, Tom
> > > > Sent: Friday, April 19, 2013 1:51 PM
> > > > To: Darren Hart
> > > > Cc: Kamble, Nitin A; meta-intel at yoctoproject.org
> > > > Subject: Re: [meta-intel] emgd-driver-dev and libgl-mesa-dev
> > > > packaging conflict
> > > >
> > > > On Fri, 2013-04-19 at 13:46 -0700, Darren Hart wrote:
> > > > >
> > > > > On 04/19/2013 12:23 PM, Tom Zanussi wrote:
> > > > > > On Thu, 2013-04-18 at 22:37 -0500, Kamble, Nitin A wrote:
> > > > > >>>>>
> > > > > >>>>> With IPK packaging I am able to reproduce the issue. I think
> > > > > >>>>> this can be easily avoided by reverting this commit
> > > > > >>>>>
> > > > > >>>>> commit 5e314278e4cf37b7fbb2946f4f42a3236071e131
> > > > > >>>>> Author: Laurentiu Palcu <laurentiu.palcu at intel.com>
> > > > > >>>>> Date:   Wed Mar 13 13:05:48 2013 +0200
> > > > > >>>>>
> > > > > >>>>>     emgd-driver-bin: add pkgconfig files
> > > > > >>>>>
> > > > > >>>>>     Add pkgconfig files needed by other projects (for
> > > > > >>>>> example newer
> > > > libva
> > > > > >>>>>     packages look for egl.pc file).
> > > > > >>>>>
> > > > > >>>>>     Pkgconfig files added by this patch:
> > > > > >>>>>      * egl.pc
> > > > > >>>>>      * gl.pc
> > > > > >>>>>      * gles_cm.pc
> > > > > >>>>>      * glesv2.pc
> > > > > >>>>>
> > > > > >>>>>     Signed-off-by: Laurentiu Palcu <laurentiu.palcu at intel.com>
> > > > > >>>>>     Signed-off-by: Tom Zanussi <tom.zanussi at linux.intel.com>
> > > > > >>>>>
> > > > > >>>>>
> > > > > >>>>>
> > > > > >>>>> This commit was needed for newer libva, now we have pinned
> > > > EMGD
> > > > > >>> based
> > > > > >>>>> BSPs to older libva, so this commit is not really needed.
> > > > > >>>>
> > > > > >>>> I verified that this indeed fixes the issue.
> > > > > >>>
> > > > > >>> It more like hides the issue right?
> > > > > >> No, it is not really hiding. This commit adds the conflicting
> > > > > >> files, which are
> > > > not needed anymore.
> > > > > >>
> > > > > >>>
> > > > > >>> The problem seems to be a continued conflict between emgd and
> > > > mesa
> > > > > >>> supplying the same files.
> > > > > >>>
> > > > > >>> Seems to me we need something like the anonymous python in
> > the
> > > > > >>> meta- intel mesa bbappend from:
> > > > > >>>
> > > > > >>> commit 1a5ab65544830dd571a368b9d9f6ec782ffc94ac
> > > > > >>> Author: Nitin A Kamble <nitin.a.kamble at intel.com>
> > > > > >>> Date:   Tue Oct 30 14:38:34 2012 -0700
> > > > > >>>
> > > > > >>>     mesa-dri.bbappend: avoid conflict with emgd-driver-bin
> > > > > >>>
> > > > > >>> Right?
> > > > > >> That can be another way to fix the problem. But the issue is
> > > > > >> not only
> > > > with mesa-dri, I was seeing conflicts with other recipes too as seen
> > below.
> > > > > >>
> > > > > >> | Collected errors:
> > > > > >> |  * check_data_file_clashes: Package libgl-mesa-dev wants to
> > > > > >> | install file
> > > > /srv/home/nitin/build-test-bsps/build-emenlow/tmp/work/emenlow-
> > poky-
> > > > linux/core-image-sato-sdk/1.0-r0/rootfs/usr/lib/pkgconfig/gl.pc
> > > > > >> | 	But that file is already provided by package  *
> > > > > >> | emgd-driver-dev
> > > > > >> |  * opkg_install_cmd: Cannot install package qt4-x11-free-dev.
> > > > > >> |  * check_data_file_clashes: Package libgl-mesa-dev wants to
> > > > > >> | install file
> > > > /srv/home/nitin/build-test-bsps/build-emenlow/tmp/work/emenlow-
> > poky-
> > > > linux/core-image-sato-sdk/1.0-r0/rootfs/usr/lib/pkgconfig/gl.pc
> > > > > >> | 	But that file is already provided by package  *
> > > > > >> | emgd-driver-dev
> > > > > >> |  * opkg_install_cmd: Cannot install package mesa-demos-dev.
> > > > > >> |  * check_data_file_clashes: Package libgl-mesa-dev wants to
> > > > > >> | install file
> > > > /srv/home/nitin/build-test-bsps/build-emenlow/tmp/work/emenlow-
> > poky-
> > > > linux/core-image-sato-sdk/1.0-r0/rootfs/usr/lib/pkgconfig/gl.pc
> > > > > >> | 	But that file is already provided by package  *
> > > > > >> | emgd-driver-dev
> > > > > >> |  * opkg_install_cmd: Cannot install package libva-egl-dev.
> > > > > >> |  * check_data_file_clashes: Package libgl-mesa-dev wants to
> > > > > >> | install file
> > > > /srv/home/nitin/build-test-bsps/build-emenlow/tmp/work/emenlow-
> > poky-
> > > > linux/core-image-sato-sdk/1.0-r0/rootfs/usr/lib/pkgconfig/gl.pc
> > > > > >> | 	But that file is already provided by package  *
> > > > > >> | emgd-driver-dev
> > > > > >> |  * opkg_install_cmd: Cannot install package libva-glx-dev.
> > > > > >> |  * check_data_file_clashes: Package libgl-mesa-dev wants to
> > > > > >> | install file
> > > > /srv/home/nitin/build-test-bsps/build-emenlow/tmp/work/emenlow-
> > poky-
> > > > linux/core-image-sato-sdk/1.0-r0/rootfs/usr/lib/pkgconfig/gl.pc
> > > > > >> | 	But that file is already provided by package  *
> > > > > >> | emgd-driver-dev
> > > > > >> |  * opkg_install_cmd: Cannot install package libgl-mesa-dev.
> > > > > >> | ERROR: Function failed: do_rootfs (see
> > > > > >> | /srv/home/nitin/build-test-bsps/build-
> > > > emenlow/tmp/work/emenlow-po
> > > > > >> | ky-linux/core-image-sato-sdk/1.0-r0/temp/log.do_rootfs.49983
> > > > > >> | for further information)
> > > > > >>
> > > > > >>
> > > > > >> So I think at this point in the 1.4 release, the least
> > > > > >> disturbing solution will
> > > > be to revert the commit adding .pc files to emgd recipe.
> > > > > >>
> > > > > >
> > > > > > OK, so if this is the solution, we need to do test builds and
> > > > > > run-time testing on the potentially affected images and arches?
> > > > > > I guess that will now have to include .ipk now too.
> > > > > >
> > > > >
> > > > > Ross made a suggestion to just drop the offending .pc file from
> > > > > the emgd recipe, it was for something it didn't even provide iirc...
> > > > >
> > > > > I can run some builds over the weekend on rage as well once we
> > > > > have agreed on a patch.
> > > > >
> > > >
> > > > Right but Nitin said that there were other problems that only
> > > > reverting the patch would fix.
> > > >
> > > > Running builds over the weekend is good, but it seems that
> > > > everything would need to be re-run-tested as well.  I can also help
> > > > test once we have a patch, but probably only emgd targets.
> > > >
> > >
> > > This is issue with EMGD bsps only. So we will need to test only sdk
> > > images for EMGD BSPs with ipk packaging. Rest of the BSPs would not
> > > get affected if we just revert the .pc commit for the emgd recipe.
> > >
> > 
> > You would think.
> > 
> > Anyway, can you guarantee that it won't break rpm and/or sato/minimal, etc
> > without testing?  The track record hasn't been great with these kinds of
> > changes.
> 
> Yesterday I tested sato & sdk with rpm for emenlow BSP. And with my previous experience in this space I have high confidence that there are no other side effects here. But as you say it is not possible to guarantee it, and testing more would help remove any doubts. So what do we agree finally? IMO revert of the commit would be least disturbing, and that is what I am testing here so far. and I am also open to any alternate solution. Let's finalize the final solution 1st, so that we can start with the testing accordingly.
> 

If you think that's the right thing to do, fine with me - I can test
crownbay over the weekend.

Tom

> Nitin
> 
> 
> > 
> > Tom
> > 
> > > Right now I am tesgin emenlow here. And I can also test crownbay. I will
> > not be able to test FRI2 & sys940x.
> > >
> > 
> > >
> > > Nitin
> > >
> > >
> > >
> > > > Tom
> > > >
> > > > > --
> > > > > Darren
> > > > >
> > > > > > Tom
> > > > > >
> > > > > >> Nitin
> > > > > >>
> > > > > >>>
> > > > > >>> --
> > > > > >>> Darren Hart
> > > > > >>> Intel Open Source Technology Center Yocto Project - Technical
> > > > > >>> Lead
> > > > > >>> - Linux Kernel
> > > > > >
> > > > > >
> > > > >
> > > >
> > >
> > 
> 