Re: [yocto] State of Yocto styleguide and oe-stylize.py script
On Fri, Mar 17, 2023 at 5:51 AM VIVAVIS AG via lists.yoctoproject.org wrote: > 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 > This is an open source community project, so as an interested user, you count as a maintainer :) I say this because it would be great if you used this valuable time and energy you have spent to help the next person. Your observations are excellent and should be captured. > 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 > *Gesendet:* Montag, 13. März 2023 19:00 > *An:* Alexander Kanavin > *Cc:* embedded (VIVAVIS AG) ; > yocto@lists.yoctoproject.org > *Betreff:* Re: [yocto] State of Yocto styleguide and oe-stylize.py script > > > > On Mon, Mar 13, 2023 at 6:34 PM Alexander Kanavin > 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> > 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_ba
Re: [yocto] State of Yocto styleguide and oe-stylize.py script
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 Gesendet: Montag, 13. März 2023 19:00 An: Alexander Kanavin Cc: embedded (VIVAVIS AG) ; yocto@lists.yoctoproject.org Betreff: Re: [yocto] State of Yocto styleguide and oe-stylize.py script On Mon, Mar 13, 2023 at 6:34 PM Alexander Kanavin mailto:alex.kana...@gmail.com>> 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> mailto:vivavis@lists.yoctoproject.org>> 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; how
Re: [yocto] State of Yocto styleguide and oe-stylize.py script
On Mon, Mar 13, 2023 at 6:34 PM Alexander Kanavin 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 > wrote: > > > > Hi, > > > > I'm wondering whether the styleguide > www.openembedded.org/wiki/Styleguide, > > 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 > > 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 (#59420): https://lists.yoctoproject.org/g/yocto/message/59420 Mute This Topic: https://lists.yoctoproject.org/mt/97581374/21656 Group Owner: yocto+ow...@lists.yoctoproject.org Unsubscribe: https://lists.yoctoproject.org/g/yocto/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [yocto] State of Yocto styleguide and oe-stylize.py script
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. 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 wrote: > > Hi, > > I'm wondering whether the styleguide www.openembedded.org/wiki/Styleguide, > 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 > 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 (#59419): https://lists.yoctoproject.org/g/yocto/message/59419 Mute This Topic: https://lists.yoctoproject.org/mt/97581374/21656 Group Owner: yocto+ow...@lists.yoctoproject.org Unsubscribe: https://lists.yoctoproject.org/g/yocto/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [yocto] State of Yocto styleguide and oe-stylize.py script
On 3/13/23 7:20 AM, VIVAVIS AG via lists.yoctoproject.org wrote: Hi, I'm wondering whether the styleguide www.openembedded.org/wiki/Styleguide, 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 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? oe-stylize.py is still the linter of choice but as you have noted it has been lacking on certain areas and wiki should be updated to match what it produces. It does need some TLC and patches are welcome. Thanks for clarification. Carsten Stelling OpenPGP_0xBB053355919D3314.asc Description: OpenPGP public key OpenPGP_signature Description: OpenPGP digital signature -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#59418): https://lists.yoctoproject.org/g/yocto/message/59418 Mute This Topic: https://lists.yoctoproject.org/mt/97581374/21656 Group Owner: yocto+ow...@lists.yoctoproject.org Unsubscribe: https://lists.yoctoproject.org/g/yocto/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
[yocto] State of Yocto styleguide and oe-stylize.py script
Hi, I'm wondering whether the styleguide www.openembedded.org/wiki/Styleguide, 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 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 (#59408): https://lists.yoctoproject.org/g/yocto/message/59408 Mute This Topic: https://lists.yoctoproject.org/mt/97581374/21656 Group Owner: yocto+ow...@lists.yoctoproject.org Unsubscribe: https://lists.yoctoproject.org/g/yocto/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-