mbox series

[v2,0/2] Describe AUTOREV and SRCREV_FORMAT roles in fetching

Message ID 20251022175913.13007-1-adam.blank.g@gmail.com
Headers show
Series Describe AUTOREV and SRCREV_FORMAT roles in fetching | expand

Message

Adam Blank Oct. 22, 2025, 5:59 p.m. UTC 
Fix encountered style issues, add missing variable
descriptions and references.
---
Changes in v2:
- abandoned the commit for the UNPACKDIR
- reworded and fixed typos

---
Adam Blank (2):
  doc: bitbake-user-manual: AUTOREV and SRCREV_FORMAT roles in fetching
  doc: bitbake-user-manual: style and completeness corrections

 .../bitbake-user-manual-fetching.rst          | 112 ++++++++++++------
 .../bitbake-user-manual-ref-variables.rst     |  18 ++-
 2 files changed, 91 insertions(+), 39 deletions(-)

Comments

Antonin Godard Oct. 23, 2025, 8:09 a.m. UTC | #1 
Hi,

On Wed Oct 22, 2025 at 7:59 PM CEST, Adam Blank via lists.openembedded.org wrote:
> Fix encountered style issues, add missing variable
> descriptions and references.
> ---
> Changes in v2:
> - abandoned the commit for the UNPACKDIR
> - reworded and fixed typos
>
> ---
> Adam Blank (2):
>   doc: bitbake-user-manual: AUTOREV and SRCREV_FORMAT roles in fetching
>   doc: bitbake-user-manual: style and completeness corrections
>
>  .../bitbake-user-manual-fetching.rst          | 112 ++++++++++++------
>  .../bitbake-user-manual-ref-variables.rst     |  18 ++-
>  2 files changed, 91 insertions(+), 39 deletions(-)

Please, put other people's Reviewed-by in your patch body _after_ said people
have sent these kind of trailers themselves.

Some details:
https://docs.kernel.org/process/submitting-patches.html#using-reported-by-tested-by-reviewed-by-suggested-by-and-fixes

I would also suggest using b4 which has the `b4 trailers -u` command to
automatically add these to your patches:
https://b4.docs.kernel.org/en/latest/

With that said, for the two patches:

Reviewed-by: Antonin Godard <antonin.godard@bootlin.com>

Thanks!
Antonin