State of Yocto styleguide and oe-stylize.py script

State of Yocto styleguide and oe-stylize.py script

[embedded (VIVAVIS AG)]

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

Re: [yocto] State of Yocto styleguide and oe-stylize.py script

[Khem Raj]

[-- Attachment #1.1.1: Type: text/plain, Size: 2103 bytes --]

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
> 
> 
> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> You automatically follow any topics you start or reply to.
> View/Reply Online (#59408): https://lists.yoctoproject.org/g/yocto/message/59408
> Mute This Topic: https://lists.yoctoproject.org/mt/97581374/1997914
> Group Owner: yocto+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/yocto/unsub [raj.khem@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
> 

[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 2613 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 203 bytes --]

Re: [yocto] State of Yocto styleguide and oe-stylize.py script

[Alexander Kanavin]

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
<embedded=vivavis.com@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 (#59408): https://lists.yoctoproject.org/g/yocto/message/59408
> Mute This Topic: https://lists.yoctoproject.org/mt/97581374/1686489
> Group Owner: yocto+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/yocto/unsub [alex.kanavin@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>

Re: [yocto] State of Yocto styleguide and oe-stylize.py script

[Martin Jansa]

[-- Attachment #1: Type: text/plain, Size: 3842 bytes --]

On Mon, Mar 13, 2023 at 6:34 PM Alexander Kanavin <alex.kanavin@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
> <embedded=vivavis.com@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/3617156
> Group Owner: yocto+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/yocto/unsub [
> Martin.Jansa@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
>

[-- Attachment #2: Type: text/html, Size: 5349 bytes --]

AW: [yocto] State of Yocto styleguide and oe-stylize.py script

[embedded (VIVAVIS AG)]

[-- Attachment #1: Type: text/plain, Size: 7383 bytes --]

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 <martin.jansa@gmail.com>
Gesendet: Montag, 13. März 2023 19:00
An: Alexander Kanavin <alex.kanavin@gmail.com>
Cc: embedded (VIVAVIS AG) <embedded@vivavis.com>; 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 <alex.kanavin@gmail.com<mailto:alex.kanavin@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>
<embedded=vivavis.com@lists.yoctoproject.org<mailto:vivavis.com@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; 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 (#59419): https://lists.yoctoproject.org/g/yocto/message/59419<https://atpscan.global.hornetsecurity.com/index.php?atp_str=FpQT2EiH51fObDN7f3ZfphzN9csC8vs4QqLTiruLkhli47GZt8jed0F7geUCiAqtwVlr0L1NWPSfgkhKPiZZzKCICDJ4Fyfl_aWfh3YaU-GswBsZUm2SFeDk2jKhkG-z27_gVCrSW5usAsSFv_p93zdRyRex_h2CXB8Cs2rJ5XQI-m2mhFTSmjI_fa54HrCyBw24UdtEeqdLrQCQt52OmLliUB573eSwCHhbUkddIfbmm9YPmVKWtBO6DjQsLTbo3-oxJysHr8uNZotz-oe5fYnCKUGqL91VrM5zbKneK2F_BNN6rc6tIC33RBR1u1IAyxc8Izo6I8txAfSjWHZbOyUZwSM6OiNBF5Yyr1CyDSg--U0hWQ0p>
Mute This Topic: https://lists.yoctoproject.org/mt/97581374/3617156<https://atpscan.global.hornetsecurity.com/index.php?atp_str=of09i-vFC_4ibqswQzjZ_3NQ9MYcDRMnlASAekpDPXZWA69PiKh13QX41yFic86Ywc05z3vB8qhDw1NpLyufSVztUQmNGyHe3yHo-CY0fVBXFhPzppZ-ere_axDSGAfyzOafE3lxc3cRwf-fI5-rRtpXTOFXdQU0sarzhI6m5q7SmzBngi8HOJq2H0jW8AojSnrsJZJj_GTPQYm45_nbIPVh3-INpMvFJP9UsjnwbPWqxOQAmCsu5lGS-b_9tWWgdZOAzwUdKG75AKzsMDnTEUI9xzlLPHVDmufMRCHLsLf9ZZyiucWc-3o0EiPK4OCF-yM6OiM2mYwKF-dVLdbhKUUjOjojb0p9hxhA2hKkS_tijk2z-g>
Group Owner: yocto+owner@lists.yoctoproject.org<mailto:yocto%2Bowner@lists.yoctoproject.org>
Unsubscribe: https://lists.yoctoproject.org/g/yocto/unsub<https://atpscan.global.hornetsecurity.com/index.php?atp_str=61jDnS-21z5P-11mgqsEbSHQa2lxazdXB-qWhbHqIBGzvmC9b4IOyz16D1u3G8htGfF7m9izxBu_pa2FUdll8oXeL9B-_hmN8nX85c6si_MyHbU7FFfe2uWBmHQC4aZFT4i8a9c9jFctjRDYxygnL6WoMRKs-8H3ftgqqNbNyQeMdtVMXEwzedTFFtJDxvLKPaJmZFJLjaKQo6cI4YNLNT8u6KdC5oJdN8kBM9Yq3TYMQaS6MZBGsx2WH7YT4fiZoCRpLevFkhdo5GTagfuwAwlhqk3e6v9c7kLHJOjJfpJZvCl6M8rrIAbKRyM6OiPY_Rq5qqIEiJqUwk0jOjojVdkTj3tgw5OdxX0aL5oz0A> [Martin.Jansa@gmail.com<mailto:Martin.Jansa@gmail.com>]
-=-=-=-=-=-=-=-=-=-=-=-

[-- Attachment #2: Type: text/html, Size: 57835 bytes --]

Re: [yocto] State of Yocto styleguide and oe-stylize.py script

[Tim Orling]

[-- Attachment #1: Type: text/plain, Size: 6704 bytes --]

On Fri, Mar 17, 2023 at 5:51 AM VIVAVIS AG via lists.yoctoproject.org
<embedded=vivavis.com@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 <martin.jansa@gmail.com>
> *Gesendet:* Montag, 13. März 2023 19:00
> *An:* Alexander Kanavin <alex.kanavin@gmail.com>
> *Cc:* embedded (VIVAVIS AG) <embedded@vivavis.com>;
> 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 <alex.kanavin@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>
> <embedded=vivavis.com@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; 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 (#59447):
> https://lists.yoctoproject.org/g/yocto/message/59447
> Mute This Topic: https://lists.yoctoproject.org/mt/97581374/924729
> Group Owner: yocto+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/yocto/unsub [
> ticotimo@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
>

[-- Attachment #2: Type: text/html, Size: 22124 bytes --]