All of lore.kernel.org
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: rpjday@crashcourse.ca, antonin.godard@bootlin.com
Cc: YP docs mailing list <docs@lists.yoctoproject.org>
Subject: Re: [docs] "Required Variables" aren't really "required"
Date: Fri, 05 Jun 2026 15:43:51 +0100	[thread overview]
Message-ID: <059b9f24b250f12dfd262d2b44d3afab53d1bdce.camel@linuxfoundation.org> (raw)
In-Reply-To: <3091fc6f-a221-8af6-c844-4ffca0bd0ecd@crashcourse.ca>

On Fri, 2026-06-05 at 07:22 -0700, Robert P. J. Day via lists.yoctoproject.org wrote:
> On Fri, 5 Jun 2026, Antonin Godard via lists.yoctoproject.org wrote:
> 
> > On Fri Jun 5, 2026 at 1:40 PM CEST, Robert P. J. Day wrote:
> > > 
> > > here:
> > > 
> > > https://docs.yoctoproject.org/contributor-guide/recipe-style-guide.html#required-variables
> > > 
> > > the section "Required Variables" goes on to say that four variables
> > > "should" be included in all recipes ("should" does not equal "must"),
> > > but a quick test build showed that none of those four variables seemed
> > > to be required for successful parsing -- only a missing "LICENSE"
> > > triggered a parse error during my testing.
> > 
> > What do you suggest as an alternative word for "Required"?
> 
>   Well, it makes no sense to first say that short list of variables is
> "Required", then say they "should" be included in all recipes. It's
> common usage that "should" means only that something is *recommended*
> or *encouraged*, not mandatory.
> 
>   Then based on my testing, none of those variables are actually
> required since leaving them out didn't cause a parse error. So I don't
> know how people would want to word that or what it should say.

LICENSE is effectively required and should be in the required section.
SUMMARY/DESCRIPTION are also strongly encouraged else they get defaults
which aren't particularly useful. I thought there were QA checks for
those? Perhaps the QA tests are limited just to oe-core by default?

I think saying they're required is fine to be honest. LICENSE should be
moved to the required section.

HOMEPAGE is also strongly recommended, again. Perhaps the section
should be:

"""
The following variables should be included in all recipes:

LICENSE, DESCRIPTION, SUMMARY

It is also strongly recommended to set:

HOMEPAGE, BUGTRACKER
"""

Would that be more acceptable to you?

Cheers,

Richard


  reply	other threads:[~2026-06-05 14:44 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-05 11:40 "Required Variables" aren't really "required" Robert P. J. Day
2026-06-05 14:03 ` [docs] " Antonin Godard
2026-06-05 14:22   ` Robert P. J. Day
2026-06-05 14:43     ` Richard Purdie [this message]
2026-06-05 15:12       ` Robert P. J. Day
2026-06-05 15:35         ` Richard Purdie
2026-06-05 15:38           ` Robert P. J. Day
2026-06-05 15:49             ` Richard Purdie
2026-06-05 16:32               ` Robert P. J. Day
2026-06-05 16:44                 ` Richard Purdie
2026-06-05 16:47                   ` Robert P. J. Day
2026-06-17 11:24       ` Robert P. J. Day

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=059b9f24b250f12dfd262d2b44d3afab53d1bdce.camel@linuxfoundation.org \
    --to=richard.purdie@linuxfoundation.org \
    --cc=antonin.godard@bootlin.com \
    --cc=docs@lists.yoctoproject.org \
    --cc=rpjday@crashcourse.ca \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.