Thank you for all your answers on this topic. My realization from the discussion is, that there’s currently no definitive guide on how to order variables and tasks. So we’re going to proceed with our BCP and write recipes following the Styleguide and based on recipes in meta and meta-openembedded. If the style differs within these layers too, we do some sort of majority decision.
Therefore, the first line of our bbappend files would normally start with the FILESEXTRAPATHS variable for instance, like in most recipes of these public layers. oe-stylize.py places this at the end of the file, which looks a little bit strange for us. I understand Alexander, saying it must look good to the tools not for human eyes, but maybe the truth lies in between. Maybe, or hopefully the maintainers of the YP will maintain and update the styleguide (in my opinion the canonical reference) and adapt the tools to minimize the gap between specifications. As formerly stated, this may reduce discussions and overhead when contributing to the YP relating to recipes. Carsten Von: Martin Jansa <[email protected]> Gesendet: Montag, 13. März 2023 19:00 An: Alexander Kanavin <[email protected]> Cc: embedded (VIVAVIS AG) <[email protected]>; [email protected] Betreff: Re: [yocto] State of Yocto styleguide and oe-stylize.py script On Mon, Mar 13, 2023 at 6:34 PM Alexander Kanavin <[email protected]<mailto:[email protected]>> wrote: There is not a lot of interest in maintaining style guides, and associated tooling. My personal feeling is that they don't really help with the truly problematic things in recipes that need a human eye (or chatgpt level intelligence :) - by and large people do follow reasonable order of entries for example, and nitpicking on an exact order would just be wasting precious maintainer time. I agree that there are other more important things, but I also like style (only in code - human looks are overrated :)). If the tool is executed locally before sending the contribution (and Carsten does seem interested in doing just that - not asking Khem to run it on incoming patches) then I believe it can save maintainers time by sending better patches on first try not waste it and our styleguide really needs some TLC as Carsten found out. The same does apply to e.g. scripts/contrib/patchreview.py. If the missing/malformed Upstream-Status is reported to the original contributor before he/she/it sends the patch, by some at least well documented work flow how to submit patches, then it saves more maintainer time, than asking for it during e-mail review or worse after the change is merged or built by maintainer and QA check reports it's missing or malformed. If someone is willing to help with the tooling, we should endorse that, not discourage them even when there are bigger-greater goals like yocto-check-layer script you mentioned. Cheers, If you want to ensure good quality, making your layer pass the yocto compatibility script, and ensuring there are no warnings of any kind from bitbake when built with latest master revisions of everything is a better first step in my opinion. Alex On Mon, 13 Mar 2023 at 15:20, VIVAVIS AG via lists.yoctoproject.org<http://atpscan.global.hornetsecurity.com/index.php?atp_str=6ii-lFB9x9qquaocUdwq3KPDfo4Ve-R9P7eR1RKQNoX0PPCb3_6yhFGC3x1GwZole7KBAvT-xxm_pGgLpG845IRXpTidBDAwEBQPD0x1b9WvbYpe4wJDh9rF3J5hWsTF-vhrwi4K0NOlANabME2YpC0J56gr8jh5aobqidN7LZ3W0kK7aL0Z77nIwfLTXeCe3DDEwwHpEQEm4E24jDBcBjPn0lR1licMMlOMuN0Ku2UJWuY5EZKYj2MDhWPES4ORfGQwTBKJ2_2SQbNCp3HiqKXuKLNA0uZZjDSnSSM6OiPSxJs_FTatAw_qwRMjOjojnNbHc4NY9ija7K0-XewMfA> <[email protected]<mailto:[email protected]>> wrote: > > Hi, > > I'm wondering whether the styleguide > www.openembedded.org/wiki/Styleguide<http://atpscan.global.hornetsecurity.com/index.php?atp_str=iIIqKiBtWyLK_Z31_K3uvMZBrXb_PFR2od9-FtXKXz0RMKEw5ucbEAeGpGronR5UoCkmMUNPDZPduCLN_Vls4-Q0YnC6_L0lK97-bh_8QIu0wG0kBftp7nxSdc2LDqs-ozpYL075CxoglKgEi_TnHSDIzWL8eNhFmR_miaWimb3E36UKxgJ6gCP7ksibOI0OBY2xn1BVElTVgJIPnB8cl2tUoif82pqqfLdtjn4PJPbitDP7HoboaHzVlcG-2DiPo9YxhYRcE9YQNsNJZ0ulKd9OnTTyIftKKOOkgcYTV9AUa5gQwkk1rFoFIzo6I0vpagG38uAaDiMMySM6OiM4QIzYwtTY-Kax_baGBvUa>, > meta-openembedded/contrib/oe-stylize.py or none of them is the source of > truth for writing a good recipe. > > E.g., if you run oe-stylize.py, the SRCREV variable is placed above SRC_URI, > or RDEPENDS is placed above FILES, which is not what the Wiki documents. > And there are more discrepancies of this type. > > Furthermore, the script doesn't know the FILESEXTRAPATHS variable in bbappend > files > and moves it to the end of my recipe. Well, this is not decribed in the Wiki, > but a look > into append files in meta-openembedded shows, that it is common pactice to put > FILESEXTRAPATHS in the first line of the recipe. > > The Wiki has an interesting note and the end: "You can run > contrib/oe-stylize.py from > meta-oe on your recipes before submitting them; however it is not necessarily > up-to-date > with all current style conventions. This page should be considered the > canonical reference." > > Furthermore, there's > github.com/openembedded/meta-openembedded/pull/465<http://github.com/openembedded/meta-openembedded/pull/465> > providing another ruleset for the new linter. > > So, my question to the Yocto maintainers is, what is the current state of the > styleguide > and oe-stylize.py? Are there plans to synchronize them? > > Thanks for clarification. > > Carsten Stelling > > >
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#59447): https://lists.yoctoproject.org/g/yocto/message/59447 Mute This Topic: https://lists.yoctoproject.org/mt/97581374/21656 Group Owner: [email protected] Unsubscribe: https://lists.yoctoproject.org/g/yocto/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
