On Fri, 14 Oct 2022 at 09:59, Jose Quaresma <quaresma.j...@gmail.com> wrote: >> for l in layers: >> - if l[0] == os.path.abspath(args.layerpath): >> + if os.path.abspath(l[0]) == os.path.abspath(args.layerpath): >> targetlayer = l[0] > > > Maybe can be better to have the abspath of layerpath in targetlayer > > layerpath = os.path.abspath(args.layerpath) > if os.path.abspath(l[0]) == layerpath: > targetlayer = layerpath
I prefer Adrian's version. 'layerpath' is not a good name, and targetlayer does not need to be absolute. If you'd like to refactor this, doing this with a boolean variable 'has_targetlayer' would be better. Alex
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#171729): https://lists.openembedded.org/g/openembedded-core/message/171729 Mute This Topic: https://lists.openembedded.org/mt/94321973/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
