<div dir="ltr">Thanks for the feedback! Is there any way to enforce character limit ?<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 22, 2021 at 3:24 PM <<a href="mailto:paul.elder@ideasonboard.com">paul.elder@ideasonboard.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Vedant,<br>
<br>
Thank you for the patch.<br>
<br>
The subject should be more concise, limited to 75 characters (you can go<br>
over by one word, maybe). I think Jacopo had a pretty good suggestion in<br>
his review of v1.<br>
<br>
Also for v2 the patch subject should have [PATCH v2], which you can put<br>
in automatically with git format-patch -v 2<br>
<br>
On Mon, Mar 22, 2021 at 03:01:03PM +0530, Vedant Paranjape wrote:<br>
> While installing meson using pip3 install --user meson, due to python path issues, build.ninja can't be located by ninja.<br>
<br>
All lines in the changelog should be line wrapped to 75 characters per<br>
line (with some exceptions allowed, for compiler error messages, for<br>
example).<br>
<br>
As for the content I think you have the gist. I would want some more<br>
elaboration on what kind of python path issues, though.<br>
<br>
> It gives the following error on ninja -C build install:<br>
> <br>
> ninja: Entering directory `build'<br>
> ninja: error: loading 'build.ninja': No such file or directory<br>
> <br>
> After uninstalling meson and installing it without --user argument solved the issue, as discussed with pinchartl on irc.<br>
<br>
Same here on line length.<br>
<br>
Uninstalling meson where? From pip, or from the system package manager?<br>
<br>
> <br>
> Signed-off-by: Vedant Paranjape <<a href="mailto:vedantparanjape160201@gmail.com" target="_blank">vedantparanjape160201@gmail.com</a>><br>
> ---<br>
>  README.rst | 4 +++-<br>
>  1 file changed, 3 insertions(+), 1 deletion(-)<br>
> <br>
> diff --git a/README.rst b/README.rst<br>
> index 1427c714..0bfd39d9 100644<br>
> --- a/README.rst<br>
> +++ b/README.rst<br>
> @@ -56,7 +56,9 @@ Meson Build system: [required]<br>
>  <br>
>              pip3 install --user meson<br>
>              pip3 install --user --upgrade meson<br>
> -<br>
> +        <br>
> +        If this fails, retry with `pip3 install meson`<br>
> +        <br>
<br>
I assume you'll upgrade this in v3 :)<br>
<br>
<br>
Paul<br>
<br>
>  for the libcamera core: [required]<br>
>          python3-yaml python3-ply python3-jinja2<br>
>  <br>
> -- <br>
> 2.25.1<br>
</blockquote></div>