Right, I guess I was missing the context where the patch is applied: "if delayed_postinst is None" inside uninstall_unneeded(). Perhaps you could expand the commit message a bit, and say specifically what is the scenario when this is executed ("not needed" is somewhat vague).
Then no separate test is necessary. Alex On Mon, 22 Mar 2021 at 14:41, Awais Belal <awais_be...@mentor.com> wrote: > Hi Alex, > > Why isn’t this seen in the tests? Can you add a test that shows the issue > please? (Fails without the patch, passes with) > > I'm not very well versed with the oe test infrastructure, I'd be happy to > create a test for this scenario if you can point me to the material that I > need to go through in order to be able to create a test for this. However, > if you want me to summarize this tends to fix the issue where the systemd > run-postinsts.service is run at the target even when not needed (no pending > postinsts are found during build). > > I do need to send a v2 for this as there's a slight issue in the patch. > > >
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#149780): https://lists.openembedded.org/g/openembedded-core/message/149780 Mute This Topic: https://lists.openembedded.org/mt/81518683/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-