Hi Denys,

On Tue, 19 Oct 2021 18:52:12 -0400 Denys Dmytriyenko <[email protected]> wrote:
> On Tue, Oct 19, 2021 at 05:30:10PM -0400, Vivien Didelot wrote:
> > Hi,
> > 
> > With latest master(s), ti-sgx-ddk-km fails with:
> > 
> >     ERROR: ti-sgx-ddk-km-1.17.4948957-r22x do_package: QA Issue: 
> > ti-sgx-ddk-km: Files/directories were installed but not shipped in any 
> > package:
> >       /lib
> >       /lib/modules
> >       /lib/modules/5.10.59-g04b02a7e8e
> >       /lib/modules/5.10.59-g04b02a7e8e/extra
> >       /lib/modules/5.10.59-g04b02a7e8e/extra/pvrsrvkm.ko
> >     Please set FILES such that these items are packaged. Alternatively if 
> > they are unneeded, avoid installing them or delete them within do_install.
> >     ti-sgx-ddk-km: 5 installed and not shipped files. [installed-vs-shipped]
> > 
> > But adding a ti-sgx-ddk-km_1.17.4948957.bbappend recipe containing:
> > 
> >     FILES:${PN} += "/lib/modules/5.10.59-g04b02a7e8e/extra/pvrsrvkm.ko"
> > 
> > won't fix the problem if DISTRO_FEATURES contains "usrmerge":
> > 
> >     ERROR: ti-sgx-ddk-km-1.17.4948957-r22x do_package_qa: QA Issue: 
> > ti-sgx-ddk-km package is not obeying usrmerge distro feature. /lib should 
> > be relocated to /usr. [usrmerge]
> > 
> > Instead this unrecommended workaround fixes the task:
> > 
> >     do_install:append () {
> >         mkdir ${D}/usr
> >         mv ${D}/lib ${D}/usr
> >     }
> > 
> > This seems like a regression with the prefix passed to the module install
> > step, forcing ${D}/lib instead of ${D}/usr/lib when usrmerge is enabled.
> 
> No regressions - this part hasn't change forever. The internal Makefile 
> framework in SGX and RGX packages is very generic and convoluted, plus 
> somewhat old and outdated - it has no knowledge of "usrmerge" concept 
> and simply installs modules under /lib/modules under rootfs hierarchy. 
> The recipe doesn't dictate that, it just passes ${D} down as the rootfs 
> hierarchy. But I think I have a workaround for this...

True, the package "used to work" for me but I wasn't using the distro
feature back then. I'll give the patch a test if that can help.


Thanks,

        Vivien
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#14065): 
https://lists.yoctoproject.org/g/meta-ti/message/14065
Mute This Topic: https://lists.yoctoproject.org/mt/86451285/21656
Group Owner: [email protected]
Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to