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@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@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.pa...@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.pa...@intel.com> > > > > > >>>>> Signed-off-by: Tom Zanussi <tom.zanu...@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.kam...@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 > > > > > > > > > > > > > > > > > > > > > > > > > > > _______________________________________________ meta-intel mailing list meta-intel@yoctoproject.org https://lists.yoctoproject.org/listinfo/meta-intel