On Mon, 2024-09-30 at 12:33 +0000, Ross Burton wrote: > On 26 Sep 2024, at 21:40, Claus Stovgaard via lists.openembedded.org > <[email protected]> wrote: > > > > Return early when globs is either None or an empty string. If globs > > is > > an empty string from the self.d.getVar, we should skip the reset of > > install_complementary, as the result from processing with empty > > glob in > > oe-pkgdata-util will always be 0 packages to install. > > This isn’t right: if the globs are empty then we can skip the > processing of the globs, but this function also contains the locale > archive generation which is then skipped. >
Oh yes - you are correct. I focused to much on the top part, and the with loop, so somehow it slipped. Nicely catch. I see 3 options. 1) drop this patch, and keep doing the extra work. 2) invert the condition and move the with loop in under this condition 3) split the archiving out in seperate method, and then call it after install_complementary the places where install_complementary is called from. This option is what I belive you refering to below. What do you think is the best options? /Claus > Ideally this function is tidied up a little as locale archive > generation isn’t really related to complementary installation. > > Ross
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#205133): https://lists.openembedded.org/g/openembedded-core/message/205133 Mute This Topic: https://lists.openembedded.org/mt/108675632/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
