I get; QA: other The ebuild is installing to one or more unexpected paths: /tmp Please fix the ebuild to use correct FHS/Gentoo policy paths. Thanks
Nothing to fix: This is a false positive. Yes, we store files in /tmp in src_install. But this is required to allow binary package user to only install ucode(s) supported by the real system (see USE=hostonly). Later in pkg_preinst we will make sure that "$ED/tmp" won't be merged to user's system. Of course we could move this temporary directory somewhere else to silence the QA warning but this wouldn't be correct because this would hide the fact that we use a temporary location on purpose in the image. Maybe the QA check should offer a variable to override this check like we can do for pre compiled or stripped binaries.
Leaving the ebuild as-is will result in a steady stream of duplicate bug reports. I don't really understand the need for a temporary location at all. I suggest installing the files to ${ED}/lib/firmware in src_install, and removing them if necessary in pkg_preinst.
(In reply to Mike Gilbert from comment #2) > I don't really understand the need for a temporary location at all. I > suggest installing the files to ${ED}/lib/firmware in src_install, and > removing them if necessary in pkg_preinst. iucode_tool needs one folder to read from and one folder to write to. This cannot be the same folder.
The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=e6ade8189bb6037cf298ed70889f15cfcd0a0a00 commit e6ade8189bb6037cf298ed70889f15cfcd0a0a00 Author: Mike Gilbert <floppym@gentoo.org> AuthorDate: 2018-12-16 23:17:46 +0000 Commit: Thomas Deutschmann <whissi@gentoo.org> CommitDate: 2018-12-16 23:18:13 +0000 sys-firmware/intel-microcode: bump - Updated microcodes: sig 0x00050653, pf_mask 0x97, 2018-04-20, rev 0x1000144 -> 2018-08-24, rev 0x1000146 sig 0x00050654, pf_mask 0xb7, 2018-08-09, rev 0x2000050 -> 2018-10-08, rev 0x2000055 sig 0x00050655, pf_mask 0xb7, 2018-04-27, rev 0x300000b -> 2018-06-15, rev 0x300000d - Ebuild changes: In addition to the updated microcodes, the package now avoids using a temp directory for install. Instead it creates a temp directory in pkg_preinst if necessary. This should only be needed when the 'hostonly' USE flag is enabled. Closes: https://github.com/gentoo/gentoo/pull/10461 Closes: https://bugs.gentoo.org/671436 Package-Manager: Portage-2.3.52, Repoman-2.3.12 Signed-off-by: Thomas Deutschmann <whissi@gentoo.org> sys-firmware/intel-microcode/Manifest | 1 + .../intel-microcode-20180807a_p20181215.ebuild | 239 +++++++++++++++++++++ 2 files changed, 240 insertions(+)
