On 2020-04-05 06:44, Michał Górny wrote: >> +EAPI=6 > Is there really a good reason to use an old EAPI here?
None other than me having assumed that there must have been an important reason why such a simple ebuild had not been bumped to EAPI 7 yet. Will change this in the next iteration. >> +RDEPEND="app-eselect/eselect-opencl >> + dev-libs/ocl-icd[khronos-headers,${MULTILIB_USEDEP}]" > > Wouldn't it make sense to remove the virtual and just have stuff depend > on that instead? It would if there only were only one ICD loader in the tree - but there are two, this one and dev-libs/opencl-icd-loader. Overall it seems the latter might be preferable in the long run (official Khronos Group loader, much smaller output library, supports the new unified headers, the last ocl-icd release came out in November 2017 and there has been minimal repo activity since then) but with it having only been officially released in mid-March and with me having only just added it to Gentoo, I feel I'd rather test it for a while before listing it as an alternative in the virtual. Moreover, for the time being we still need eselect-opencl here even if we are no longer to use to switch between implementations because somewhat surprisingly (to me anyway), the package in questions installs OpenCL header files too. > This looks like README.gentoo material. Will do. -- MS
signature.asc
Description: OpenPGP digital signature