All of lore.kernel.org
 help / color / mirror / Atom feed
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Ricardo Ribalda <ribalda@chromium.org>
Cc: Hans Verkuil <hans@jjverkuil.nl>, linux-media@vger.kernel.org
Subject: Re: [PATCH v4l-utils] CONTRIBUTE: Add simple contribution instructions
Date: Thu, 5 Jun 2025 15:46:35 +0300	[thread overview]
Message-ID: <20250605124635.GC29935@pendragon.ideasonboard.com> (raw)
In-Reply-To: <20250605-prefix-v1-1-450aff98308a@chromium.org>

On Thu, Jun 05, 2025 at 12:09:57PM +0000, Ricardo Ribalda wrote:
> In the media summit we discussed the need of a special prefix for
> v4l-utils. This helps patchwork and media-ci.
> 
> Create a new file with a brief introductions on how to contribute and
> make the `v4l-utils` official.
> 
> Signed-off-by: Ricardo Ribalda <ribalda@chromium.org>

I think this is a good idea. I don't know how we'll get users to
actually use the prefix, but documenting it is a necessary first step.

> ---
>  CONTRIBUTE.md | 19 +++++++++++++++++++
>  1 file changed, 19 insertions(+)
> 
> diff --git a/CONTRIBUTE.md b/CONTRIBUTE.md
> new file mode 100644
> index 0000000000000000000000000000000000000000..19a2c5814b45462f3cfeea921e277f8da00ccbd1
> --- /dev/null
> +++ b/CONTRIBUTE.md
> @@ -0,0 +1,19 @@
> +## Repository
> +
> +v4l-utils official repository is hosted at https://git.linuxtv.org/v4l-utils.git
> +
> +## Contributions
> +
> +Patches are welcome! Please send your patches to `linux-media@vger.kernel.org`
> +using the prefix `[PATCH v4l-utils]`.

Nitpicking, I was thinking of '[v4l-utils] [PATCH]' as that's what I
used previously. Looking at the list archives, both (and more) have been
used. I don't object to using '[PATCH v4l-utils]' if that's easier for
tools.

I have this in my .git/config for v4l-utils:

[format]
        subjectPrefix = "v4l-utils] [PATCH"

With your proposal, this would be

[format]
        subjectPrefix = "PATCH v4l-utils"

In either case, let's document this in addition to the b4 configuration.

> +
> +## b4 config
> +
> +If you use b4[1] for your contributor workflow you can use these options:
> +
> +```
> +git config set b4.send-series-cc linux-media@vger.kernel.org
> +git config set b4.send-prefixes v4l-utils
> +```
> +
> +[1] https://b4.docs.kernel.org/en/latest/contributor/overview.html
> 
> ---
> base-commit: 0d4e8e2d4e2d9d87863222e03fd3a6a4a3ff3d86
> change-id: 20250605-prefix-a1a47dbdaa75

-- 
Regards,

Laurent Pinchart

  reply	other threads:[~2025-06-05 12:46 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-06-05 12:09 [PATCH v4l-utils] CONTRIBUTE: Add simple contribution instructions Ricardo Ribalda
2025-06-05 12:46 ` Laurent Pinchart [this message]
2025-06-05 12:54   ` Ricardo Ribalda
2025-06-05 13:17     ` Laurent Pinchart
2025-06-05 13:54       ` Ricardo Ribalda
2025-06-06  7:52       ` Mauro Carvalho Chehab
2025-06-06  8:08         ` Ricardo Ribalda
2025-06-06  8:51           ` Mauro Carvalho Chehab

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=20250605124635.GC29935@pendragon.ideasonboard.com \
    --to=laurent.pinchart@ideasonboard.com \
    --cc=hans@jjverkuil.nl \
    --cc=linux-media@vger.kernel.org \
    --cc=ribalda@chromium.org \
    /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.