Hi Patrik, We merged this, but the scripts seem overly defensive, would you be able to send a follow-on cleanup? See comments below.
On 2 Feb 2026, at 22:36, Pratik Farkase via lists.openembedded.org <[email protected]> wrote: > +for t in allowuninstall alternative blacklist choicerules choose cleandeps \ > + cplxdeps distupgrade evrcmp excludefromweak favor focus forcebest \ > + lock lockstep multiversion namespace proof recommendations sat \ > + selection strictrepoprio strongrecommends targeted testcase > weakdeps \ > + whatprovideswithdisabled yumobs; do Can this just be for t in testcases/*? > + if [ -x ./runtestcases.sh ] && [ -d testcases/$t ]; then You know these exist as they were installed, and if they were not then we want the tests to fail. Remove the checks. > +do_install_ptest() { > + install -d ${D}${PTEST_PATH}/tools > + install -d ${D}${PTEST_PATH}/test > + > + if [ -f ${B}/tools/testsolv ]; then > + install -m 0755 ${B}/tools/testsolv ${D}${PTEST_PATH}/tools/ > + fi > + > + if [ -f ${S}/test/runtestcases.sh ]; then > + install -m 0755 ${S}/test/runtestcases.sh ${D}${PTEST_PATH}/test/ > + fi > + > + if [ -d ${S}/test/testcases ]; then > + cp -r ${S}/test/testcases ${D}${PTEST_PATH}/test/ > + fi Why all the checks? If the tests rename or move then we _want_ the recipe to fail so the ptest installation can be updated. Cheers, Ross
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#230890): https://lists.openembedded.org/g/openembedded-core/message/230890 Mute This Topic: https://lists.openembedded.org/mt/117606429/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
