[libcamera-devel] [PATCH] Fix spelling error

Kieran Bingham kieran.bingham at ideasonboard.com
Thu Apr 14 21:10:41 CEST 2022


Hi Rishikesh,

Quoting Rishikesh Donadkar via libcamera-devel (2022-04-14 19:57:12)
> From: RISHI27-dot <rishikehsdonakdar at gmail.com>

This looks like you need to configure your git author

 git config --global author.name "Rishikesh Donadkar"

And you should add a signed-off by tag on patches you create.
I usually do that by adding a '-s' when I commit

  git commit -s -m "Documentation: Fix spelling error"

Patch titles should have a prefix (in this case 'Documentation') showing
where the commit is making changes before the brief summary.

And ideally - the commit message should say something too, in this case.

"The introduction incorrectly spells the word Compatibility. Fix it"

I know this is a trivial patch that doesn't really need much
explanation, but it's good to get in the habit for all your patches.

--
Kieran


> 
> ---
>  Documentation/guides/introduction.rst | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/guides/introduction.rst b/Documentation/guides/introduction.rst
> index 21880ce7..a4671a07 100644
> --- a/Documentation/guides/introduction.rst
> +++ b/Documentation/guides/introduction.rst
> @@ -67,7 +67,7 @@ through an adaptation layer integrating libcamera into a larger framework.
>      Adaptation Layer                                 |
>       /    +--------------+  +--------------+         |          +--------------+
>       |    |    V4L2      |  |  gstreamer   |         |          |   Android    |
> -     |    | Compatability|  |   element    |         |          |   Camera     |
> +     |    | Compatibility|  |   element    |         |          |   Camera     |

For the change itself,


Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>


You can add that to your next iteration of the patch above your
Signed-off-by: tag.



>       |    |  (preload)   |  |(libcamerasrc)|         |          |     HAL      |
>       \    +--------------+  +--------------+         |          +--------------+
>                                                       |
> -- 
> 2.25.1
>


More information about the libcamera-devel mailing list