<div dir="auto">Sure, will do that. Should I add the error message in commit message or the readme?</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 22 Mar, 2021, 13:41 Sebastian Fricke, <<a href="mailto:sebastian.fricke@posteo.net">sebastian.fricke@posteo.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hey Vedant,<br>
<br>
Thank you for the patch.<br>
<br>
I believe this patch would benefit from a description, that makes clear<br>
why this change is needed. The current version feels like you've hit a<br>
quite specific problem and you provide a quite generic solution to it.<br>
<br>
On 22.03.2021 13:10, Vedant Paranjape wrote:<br>
>Signed-off-by: Vedant Paranjape <<a href="mailto:vedantparanjape160201@gmail.com" target="_blank" rel="noreferrer">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>
This feels a little too broad as there are multiple reasons for this<br>
command to fail. Can you show the error message that you encountered?<br>
<br>
> for the libcamera core: [required]<br>
>         python3-yaml python3-ply python3-jinja2<br>
><br>
>-- <br>
>2.25.1<br>
<br>
Greetings,<br>
Sebastian<br>
<br>
><br>
>_______________________________________________<br>
>libcamera-devel mailing list<br>
><a href="mailto:libcamera-devel@lists.libcamera.org" target="_blank" rel="noreferrer">libcamera-devel@lists.libcamera.org</a><br>
><a href="https://lists.libcamera.org/listinfo/libcamera-devel" rel="noreferrer noreferrer" target="_blank">https://lists.libcamera.org/listinfo/libcamera-devel</a><br>
</blockquote></div>