<div dir="auto">Ok, I will do that. </div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 22 Mar, 2021, 14:03 Jacopo Mondi, <<a href="mailto:jacopo@jmondi.org">jacopo@jmondi.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello<br>
<br>
One nit: please keep the subject line (as close as possible) to the 75<br>
columns limit. I know sometimes it's hard, but in your case a simpler<br>
<br>
libcamera: Add alternative meson install command<br>
<br>
would be enough.<br>
<br>
On Mon, Mar 22, 2021 at 01:48:41PM +0530, Vedant Paranjape wrote:<br>
> Sure, will do that. Should I add the error message in commit message or the<br>
> readme?<br>
<br>
I feel this is documenting an issue not related to libcamera but to<br>
the user setup. If installing --user fails, one should go and look why<br>
in general pip3 fails to install in the user directory, that's not<br>
specific to libcamera. Or have I missed something trivial ?<br>
<br>
Thanks<br>
  j<br>
<br>
><br>
> On Mon, 22 Mar, 2021, 13:41 Sebastian Fricke, <<a href="mailto:sebastian.fricke@posteo.net" target="_blank" rel="noreferrer">sebastian.fricke@posteo.net</a>><br>
> wrote:<br>
><br>
> > 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>
> ><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>
<br>
</blockquote></div>