On Tue, 2 Sept 2025 at 17:20, Antonios Christidis <[email protected]> wrote:
> Moreover, having the tests in a separate package allows users to easily > include them in their filesystem without having to rebuild the entire > package with |DISTRO_FEATURES| including |ptest|. This is particularly > important since rebuilding with |ptest| enabled in my experience can > cause a significant increase in build time, as it often triggers the > rebuild of basic package dependencies. You can enable ptests per-recipe by setting PTEST_ENABLED = "1" for that particular recipe only (via bbappend or distro config). > I do agree with the placement of tests being non-standard, do you have > any suggestions on elsewhere to place them ? Aside from > |/usr/lib/libsdl2/ptest/tests| . Once again, if you want to install the tests, and upstream does not currently install them, then you need to write a patch for sdl upstream that does it and get it accepted. Otherwise, what this change is doing (installing the tests ad hoc) is entirely non-standard and deviates from what upstream expects, and as such the tests would belong only in ptest package, but not in the test package. Openembedded, as a rule, does not cherry-pick items from the build tree in do_install because they might be useful to someone, with ptest the only exception. > > This should stay as PACKAGECONFIG, and be enabled subject to ptest in > > DISTRO_FEATURES. > > Looking over other packages that have ptest support, they often > automatically build tests without the need for setting PACKAGECONFIGS. This lengthens the build times and increases disk usage, and is wasteful if the tests aren't actually installed and packaged. Other recipes aren't always perfect, or they build components that do not have a way to disable tests. Alex
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#222858): https://lists.openembedded.org/g/openembedded-core/message/222858 Mute This Topic: https://lists.openembedded.org/mt/114926113/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
