[{"id":15782,"web_url":"https://patchwork.libcamera.org/comment/15782/","msgid":"<20210322081155.kbnclpfyey6r6g2m@basti-TUXEDO-Book-XA1510>","date":"2021-03-22T08:11:55","subject":"Re: [libcamera-devel] [PATCH] Added alternative to meson install\n\tcommand, if it fails with --user argument","submitter":{"id":78,"url":"https://patchwork.libcamera.org/api/people/78/","name":"Sebastian Fricke","email":"sebastian.fricke@posteo.net"},"content":"Hey Vedant,\n\nThank you for the patch.\n\nI believe this patch would benefit from a description, that makes clear\nwhy this change is needed. The current version feels like you've hit a\nquite specific problem and you provide a quite generic solution to it.\n\nOn 22.03.2021 13:10, Vedant Paranjape wrote:\n>Signed-off-by: Vedant Paranjape \n>---\n> README.rst | 4 +++-\n> 1 file changed, 3 insertions(+), 1 deletion(-)\n>\n>diff --git a/README.rst b/README.rst\n>index 1427c714..0bfd39d9 100644\n>--- a/README.rst\n>+++ b/README.rst\n>@@ -56,7 +56,9 @@ Meson Build system: [required]\n>\n> pip3 install --user meson\n> pip3 install --user --upgrade meson\n>-\n>+\n>+ If this fails, retry with `pip3 install meson`\n>+\n\nThis feels a little too broad as there are multiple reasons for this\ncommand to fail. Can you show the error message that you encountered?\n\n> for the libcamera core: [required]\n> python3-yaml python3-ply python3-jinja2\n>\n>-- \n>2.25.1\n\nGreetings,\nSebastian\n\n>\n>_______________________________________________\n>libcamera-devel mailing list\n>libcamera-devel@lists.libcamera.org\n>https://lists.libcamera.org/listinfo/libcamera-devel","headers":{"Return-Path":"","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id 21929C32E1\n\tfor ;\n\tMon, 22 Mar 2021 08:12:01 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 5643068D58;\n\tMon, 22 Mar 2021 09:12:00 +0100 (CET)","from mout01.posteo.de (mout01.posteo.de [185.67.36.65])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 1039C6051E\n\tfor ;\n\tMon, 22 Mar 2021 09:11:58 +0100 (CET)","from submission (posteo.de [89.146.220.130]) \n\tby mout01.posteo.de (Postfix) with ESMTPS id 4B711160062\n\tfor ;\n\tMon, 22 Mar 2021 09:11:57 +0100 (CET)","from customer (localhost [127.0.0.1])\n\tby submission (posteo.de) with ESMTPSA id 4F3nK82z60z9rxn;\n\tMon, 22 Mar 2021 09:11:56 +0100 (CET)"],"Authentication-Results":"lancelot.ideasonboard.com;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=posteo.net header.i=@posteo.net\n\theader.b=\"dSMlJwt/\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=posteo.net; s=2017;\n\tt=1616400717; bh=iMbyU4HWkxaMASCdyuF2TTxDRIN+of7sdX3u6oE64io=;\n\th=Date:From:To:Cc:Subject:From;\n\tb=dSMlJwt/tndT/w8OMJJuf/1Re5jPcqadPbQ3I1GM+kR9qnfNECSxz5nNYSIUziaKx\n\t/DvkH8TqYhIXy+qPSEyv1DkU4IPExU1OPaHwc2HC6RY5oFIV71J6vF2E9py7dhuKuS\n\tgGhLMBvRT2wLqBbom2wLiST0ImSMfs+P6lEPhK6MjNzSMBJPjwsuFcpMP+9G7Uw0RK\n\t/ZncRRL0vsj6RKn9t5STQG1XqxRJ5Ss/pZ3BCh8Y8FS666U+YWhih12pwMCfYmHCJy\n\tcBna2odLpfHcx9KZdX/JaKmYu7IuefWj+HKox7/7yMuItluzrH3BLjjY0+eyWDrEyR\n\tE32SlHwysFkPQ==","Date":"Mon, 22 Mar 2021 09:11:55 +0100","From":"Sebastian Fricke ","To":"Vedant Paranjape ","Message-ID":"<20210322081155.kbnclpfyey6r6g2m@basti-TUXEDO-Book-XA1510>","References":"<20210322074056.26330-1-vedantparanjape160201@gmail.com>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20210322074056.26330-1-vedantparanjape160201@gmail.com>","Subject":"Re: [libcamera-devel] [PATCH] Added alternative to meson install\n\tcommand, if it fails with --user argument","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"libcamera-devel@lists.libcamera.org","Content-Transfer-Encoding":"7bit","Content-Type":"text/plain; charset=\"us-ascii\"; Format=\"flowed\"","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}},{"id":15783,"web_url":"https://patchwork.libcamera.org/comment/15783/","msgid":"","date":"2021-03-22T08:18:41","subject":"Re: [libcamera-devel] [PATCH] Added alternative to meson install\n\tcommand, if it fails with --user argument","submitter":{"id":85,"url":"https://patchwork.libcamera.org/api/people/85/","name":"Vedant Paranjape","email":"vedantparanjape160201@gmail.com"},"content":"Sure, will do that. Should I add the error message in commit message or the\nreadme?\n\nOn Mon, 22 Mar, 2021, 13:41 Sebastian Fricke, \nwrote:\n\n> Hey Vedant,\n>\n> Thank you for the patch.\n>\n> I believe this patch would benefit from a description, that makes clear\n> why this change is needed. The current version feels like you've hit a\n> quite specific problem and you provide a quite generic solution to it.\n>\n> On 22.03.2021 13:10, Vedant Paranjape wrote:\n> >Signed-off-by: Vedant Paranjape \n> >---\n> > README.rst | 4 +++-\n> > 1 file changed, 3 insertions(+), 1 deletion(-)\n> >\n> >diff --git a/README.rst b/README.rst\n> >index 1427c714..0bfd39d9 100644\n> >--- a/README.rst\n> >+++ b/README.rst\n> >@@ -56,7 +56,9 @@ Meson Build system: [required]\n> >\n> > pip3 install --user meson\n> > pip3 install --user --upgrade meson\n> >-\n> >+\n> >+ If this fails, retry with `pip3 install meson`\n> >+\n>\n> This feels a little too broad as there are multiple reasons for this\n> command to fail. Can you show the error message that you encountered?\n>\n> > for the libcamera core: [required]\n> > python3-yaml python3-ply python3-jinja2\n> >\n> >--\n> >2.25.1\n>\n> Greetings,\n> Sebastian\n>\n> >\n> >_______________________________________________\n> >libcamera-devel mailing list\n> >libcamera-devel@lists.libcamera.org\n> >https://lists.libcamera.org/listinfo/libcamera-devel\n>","headers":{"Return-Path":"","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id 5EA91BD80C\n\tfor ;\n\tMon, 22 Mar 2021 08:18:57 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 9530668D58;\n\tMon, 22 Mar 2021 09:18:56 +0100 (CET)","from mail-yb1-xb2e.google.com (mail-yb1-xb2e.google.com\n\t[IPv6:2607:f8b0:4864:20::b2e])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 42E206051E\n\tfor ;\n\tMon, 22 Mar 2021 09:18:55 +0100 (CET)","by mail-yb1-xb2e.google.com with SMTP id i9so5636354ybp.4\n\tfor ;\n\tMon, 22 Mar 2021 01:18:55 -0700 (PDT)"],"Authentication-Results":"lancelot.ideasonboard.com;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"R5edrbdg\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=mime-version:references:in-reply-to:from:date:message-id:subject:to\n\t:cc; bh=54uKZbQU4otJ2IeUYm5S/mZh+xusqyFEKnaU5TpyQB0=;\n\tb=R5edrbdgiAIGQ6xDqW5J/fum5ZDeAMg0rbKrYNR+FcDnKN15UStv00V19m63FRRzQp\n\tJj3k1TYOxo07+vrlJSC4+08mKOxS+IJFnBCUrIZKJ2tGBdxRGAf0H9jND0m7Xd9YDS6E\n\tqI++cFFs0vjcpAmuyulUvrTsvmxcTqEkIdfZw7DkiqnyPishpmhaGQuAr7qnS0gWkp19\n\tXQFsZlNIcPKtjDz6rK7x0nukW+QS32oEIerToRDBlkvDrqgQdhijpbkaMoJjsxaYmgzv\n\taFGGQNCcBBLZf4pmFEi+jP4ZjYIje3Uk81rHxhK1RpyRotFZVR+8YOjr4Gea3xdMf6lR\n\tGf+A==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:references:in-reply-to:from:date\n\t:message-id:subject:to:cc;\n\tbh=54uKZbQU4otJ2IeUYm5S/mZh+xusqyFEKnaU5TpyQB0=;\n\tb=NYUPwpFaboWF30O7Z5jvULhuWySraVt4Evq8VKhvR4PUqYRek6ENr4e7I9vNpoG7YU\n\tRyVe0klObw6KWVZ+eYxukCJKyw+o8JYbSKemjOoqBMwGDZlewnSaC/SopfZfrRy+Z9Vi\n\tL7Zy+WcBje2PJ5sKMfWjzHB9E94E3AHljENcomCSJk/5nQaV/EOjO60+edAtc59F5ken\n\t5HTCzqD21Jqm+aprNuoZs3m4hjf6f5VAyhrolydfj6gTbpgRXiGnWNwczM5izk18T4Kd\n\tkyhmQLMyDh8hCmV2O5sXq2hlRwn8DHYwTHE1jCAZu10B/o9YSeGjhd7NZ80OPIIRkqLd\n\tApIQ==","X-Gm-Message-State":"AOAM531kbR3V5QCWkZZcZ8tjQmAGZRSiGHiAIYqpyBFXouyKGvbKKJPY\n\t12Io/XXdB+q72CkzckzFu77UUDL/RKRfEgcmtOc=","X-Google-Smtp-Source":"ABdhPJxM/OsDMyZykRlNp925LNI6/MGE+kfDywRMbXOGQasaKvfutjun6jqOpYjdMnjobJ98yrHzJCa7fOgg1j9zv+k=","X-Received":"by 2002:a25:1008:: with SMTP id 8mr22916685ybq.21.1616401133805; \n\tMon, 22 Mar 2021 01:18:53 -0700 (PDT)","MIME-Version":"1.0","References":"<20210322074056.26330-1-vedantparanjape160201@gmail.com>\n\t<20210322081155.kbnclpfyey6r6g2m@basti-TUXEDO-Book-XA1510>","In-Reply-To":"<20210322081155.kbnclpfyey6r6g2m@basti-TUXEDO-Book-XA1510>","From":"Vedant Paranjape ","Date":"Mon, 22 Mar 2021 13:48:41 +0530","Message-ID":"","To":"Sebastian Fricke ","Subject":"Re: [libcamera-devel] [PATCH] Added alternative to meson install\n\tcommand, if it fails with --user argument","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"libcamera-devel@lists.libcamera.org","Content-Type":"multipart/mixed;\n\tboundary=\"===============5767633016841643930==\"","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}},{"id":15784,"web_url":"https://patchwork.libcamera.org/comment/15784/","msgid":"<20210322082814.wyipzqmnj5x5twja@basti-TUXEDO-Book-XA1510>","date":"2021-03-22T08:28:14","subject":"Re: [libcamera-devel] [PATCH] Added alternative to meson install\n\tcommand, if it fails with --user argument","submitter":{"id":78,"url":"https://patchwork.libcamera.org/api/people/78/","name":"Sebastian Fricke","email":"sebastian.fricke@posteo.net"},"content":"Hey Vedant,\n\nOn 22.03.2021 13:48, Vedant Paranjape wrote:\n>Sure, will do that. Should I add the error message in commit message or the\n>readme?\n\nI think you should place the error message into the commit message so\nthat it becomes clear which problem is tackled by this patch. But if it\nis a very long error message, I would write a short version into the\ncommit message and the long version into this mail-series.\n\n>\n>On Mon, 22 Mar, 2021, 13:41 Sebastian Fricke, \n>wrote:\n>\n>> Hey Vedant,\n>>\n>> Thank you for the patch.\n>>\n>> I believe this patch would benefit from a description, that makes clear\n>> why this change is needed. The current version feels like you've hit a\n>> quite specific problem and you provide a quite generic solution to it.\n>>\n>> On 22.03.2021 13:10, Vedant Paranjape wrote:\n>> >Signed-off-by: Vedant Paranjape \n>> >---\n>> > README.rst | 4 +++-\n>> > 1 file changed, 3 insertions(+), 1 deletion(-)\n>> >\n>> >diff --git a/README.rst b/README.rst\n>> >index 1427c714..0bfd39d9 100644\n>> >--- a/README.rst\n>> >+++ b/README.rst\n>> >@@ -56,7 +56,9 @@ Meson Build system: [required]\n>> >\n>> > pip3 install --user meson\n>> > pip3 install --user --upgrade meson\n>> >-\n>> >+\n>> >+ If this fails, retry with `pip3 install meson`\n>> >+\n>>\n>> This feels a little too broad as there are multiple reasons for this\n>> command to fail. Can you show the error message that you encountered?\n>>\n>> > for the libcamera core: [required]\n>> > python3-yaml python3-ply python3-jinja2\n>> >\n>> >--\n>> >2.25.1\n>>\n>> Greetings,\n>> Sebastian\n>>\n>> >\n>> >_______________________________________________\n>> >libcamera-devel mailing list\n>> >libcamera-devel@lists.libcamera.org\n>> >https://lists.libcamera.org/listinfo/libcamera-devel\n>>","headers":{"Return-Path":"","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id 6B350C32E1\n\tfor ;\n\tMon, 22 Mar 2021 08:28:19 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id D85D9605B1;\n\tMon, 22 Mar 2021 09:28:18 +0100 (CET)","from mout02.posteo.de (mout02.posteo.de [185.67.36.66])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 79F806051E\n\tfor ;\n\tMon, 22 Mar 2021 09:28:17 +0100 (CET)","from submission (posteo.de [89.146.220.130]) \n\tby mout02.posteo.de (Postfix) with ESMTPS id 688C524010F\n\tfor ;\n\tMon, 22 Mar 2021 09:28:16 +0100 (CET)","from customer (localhost [127.0.0.1])\n\tby submission (posteo.de) with ESMTPSA id 4F3ngz3HQ9z6tmM;\n\tMon, 22 Mar 2021 09:28:15 +0100 (CET)"],"Authentication-Results":"lancelot.ideasonboard.com;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=posteo.net header.i=@posteo.net\n\theader.b=\"REdQgoOq\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=posteo.net; s=2017;\n\tt=1616401696; bh=5E4TL1vXq6KkRFzikUTmtszYdnzc53kjLNqWb0s4yas=;\n\th=Date:From:To:Cc:Subject:From;\n\tb=REdQgoOq2PpAOX/E9hnVYva9lFNcHIB/v+5YlN2rPkJjO4Dt1rjKY9lr+B8dx50j+\n\tiCdvC419y9nDFAd3sOL1ccCMYtIbml46xL6GeygKY8sOX+Y1TPQY/vWSu8DnpfSYaE\n\thMm/bycnIvQ3zagUmB7bOAZOcFXw43QgMXO8PilFbkbB3IzSogbxHj0hWE3deaiea4\n\t4Knz125nVGCCEbm4dIwvSiokUXFGijLdzBdJRpY79QYJ7FQu+x92Rsxl5jx3biwS/3\n\thUJOF/7bYzzKln9lR2wQiH2ZJdDEcKpH7csPnlGZGGKFqmAfWT8QfsztdWyf8PLqol\n\tjsRqIdfNH83Jg==","Date":"Mon, 22 Mar 2021 09:28:14 +0100","From":"Sebastian Fricke ","To":"Vedant Paranjape ","Message-ID":"<20210322082814.wyipzqmnj5x5twja@basti-TUXEDO-Book-XA1510>","References":"<20210322074056.26330-1-vedantparanjape160201@gmail.com>\n\t<20210322081155.kbnclpfyey6r6g2m@basti-TUXEDO-Book-XA1510>\n\t","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"","Subject":"Re: [libcamera-devel] [PATCH] Added alternative to meson install\n\tcommand, if it fails with --user argument","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"libcamera-devel@lists.libcamera.org","Content-Transfer-Encoding":"7bit","Content-Type":"text/plain; charset=\"us-ascii\"; Format=\"flowed\"","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}},{"id":15785,"web_url":"https://patchwork.libcamera.org/comment/15785/","msgid":"<20210322083332.grbhf4cy2nvmg3ej@uno.localdomain>","date":"2021-03-22T08:33:32","subject":"Re: [libcamera-devel] [PATCH] Added alternative to meson install\n\tcommand, if it fails with --user argument","submitter":{"id":3,"url":"https://patchwork.libcamera.org/api/people/3/","name":"Jacopo Mondi","email":"jacopo@jmondi.org"},"content":"Hello\n\nOne nit: please keep the subject line (as close as possible) to the 75\ncolumns limit. I know sometimes it's hard, but in your case a simpler\n\nlibcamera: Add alternative meson install command\n\nwould be enough.\n\nOn Mon, Mar 22, 2021 at 01:48:41PM +0530, Vedant Paranjape wrote:\n> Sure, will do that. Should I add the error message in commit message or the\n> readme?\n\nI feel this is documenting an issue not related to libcamera but to\nthe user setup. If installing --user fails, one should go and look why\nin general pip3 fails to install in the user directory, that's not\nspecific to libcamera. Or have I missed something trivial ?\n\nThanks\n j\n\n>\n> On Mon, 22 Mar, 2021, 13:41 Sebastian Fricke, \n> wrote:\n>\n> > Hey Vedant,\n> >\n> > Thank you for the patch.\n> >\n> > I believe this patch would benefit from a description, that makes clear\n> > why this change is needed. The current version feels like you've hit a\n> > quite specific problem and you provide a quite generic solution to it.\n> >\n> > On 22.03.2021 13:10, Vedant Paranjape wrote:\n> > >Signed-off-by: Vedant Paranjape \n> > >---\n> > > README.rst | 4 +++-\n> > > 1 file changed, 3 insertions(+), 1 deletion(-)\n> > >\n> > >diff --git a/README.rst b/README.rst\n> > >index 1427c714..0bfd39d9 100644\n> > >--- a/README.rst\n> > >+++ b/README.rst\n> > >@@ -56,7 +56,9 @@ Meson Build system: [required]\n> > >\n> > > pip3 install --user meson\n> > > pip3 install --user --upgrade meson\n> > >-\n> > >+\n> > >+ If this fails, retry with `pip3 install meson`\n> > >+\n> >\n> > This feels a little too broad as there are multiple reasons for this\n> > command to fail. Can you show the error message that you encountered?\n> >\n> > > for the libcamera core: [required]\n> > > python3-yaml python3-ply python3-jinja2\n> > >\n> > >--\n> > >2.25.1\n> >\n> > Greetings,\n> > Sebastian\n> >\n> > >\n> > >_______________________________________________\n> > >libcamera-devel mailing list\n> > >libcamera-devel@lists.libcamera.org\n> > >https://lists.libcamera.org/listinfo/libcamera-devel\n> >\n> _______________________________________________\n> libcamera-devel mailing list\n> libcamera-devel@lists.libcamera.org\n> https://lists.libcamera.org/listinfo/libcamera-devel","headers":{"Return-Path":"","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id 2C22ABD80C\n\tfor ;\n\tMon, 22 Mar 2021 08:33:03 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 8B3C768D58;\n\tMon, 22 Mar 2021 09:33:02 +0100 (CET)","from relay8-d.mail.gandi.net (relay8-d.mail.gandi.net\n\t[217.70.183.201])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 27EDF6051E\n\tfor ;\n\tMon, 22 Mar 2021 09:33:02 +0100 (CET)","from uno.localdomain (host-82-63-7-72.business.telecomitalia.it\n\t[82.63.7.72]) (Authenticated sender: jacopo@jmondi.org)\n\tby relay8-d.mail.gandi.net (Postfix) with ESMTPSA id 8CEF91BF207;\n\tMon, 22 Mar 2021 08:33:00 +0000 (UTC)"],"X-Originating-IP":"82.63.7.72","Date":"Mon, 22 Mar 2021 09:33:32 +0100","From":"Jacopo Mondi ","To":"Vedant Paranjape ","Message-ID":"<20210322083332.grbhf4cy2nvmg3ej@uno.localdomain>","References":"<20210322074056.26330-1-vedantparanjape160201@gmail.com>\n\t<20210322081155.kbnclpfyey6r6g2m@basti-TUXEDO-Book-XA1510>\n\t","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"","Subject":"Re: [libcamera-devel] [PATCH] Added alternative to meson install\n\tcommand, if it fails with --user argument","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"libcamera-devel@lists.libcamera.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}},{"id":15786,"web_url":"https://patchwork.libcamera.org/comment/15786/","msgid":"","date":"2021-03-22T08:35:38","subject":"Re: [libcamera-devel] [PATCH] Added alternative to meson install\n\tcommand, if it fails with --user argument","submitter":{"id":85,"url":"https://patchwork.libcamera.org/api/people/85/","name":"Vedant Paranjape","email":"vedantparanjape160201@gmail.com"},"content":"Ok, I will do that.\n\nOn Mon, 22 Mar, 2021, 14:03 Jacopo Mondi, wrote:\n\n> Hello\n>\n> One nit: please keep the subject line (as close as possible) to the 75\n> columns limit. I know sometimes it's hard, but in your case a simpler\n>\n> libcamera: Add alternative meson install command\n>\n> would be enough.\n>\n> On Mon, Mar 22, 2021 at 01:48:41PM +0530, Vedant Paranjape wrote:\n> > Sure, will do that. Should I add the error message in commit message or\n> the\n> > readme?\n>\n> I feel this is documenting an issue not related to libcamera but to\n> the user setup. If installing --user fails, one should go and look why\n> in general pip3 fails to install in the user directory, that's not\n> specific to libcamera. Or have I missed something trivial ?\n>\n> Thanks\n> j\n>\n> >\n> > On Mon, 22 Mar, 2021, 13:41 Sebastian Fricke, <\n> sebastian.fricke@posteo.net>\n> > wrote:\n> >\n> > > Hey Vedant,\n> > >\n> > > Thank you for the patch.\n> > >\n> > > I believe this patch would benefit from a description, that makes clear\n> > > why this change is needed. The current version feels like you've hit a\n> > > quite specific problem and you provide a quite generic solution to it.\n> > >\n> > > On 22.03.2021 13:10, Vedant Paranjape wrote:\n> > > >Signed-off-by: Vedant Paranjape \n> > > >---\n> > > > README.rst | 4 +++-\n> > > > 1 file changed, 3 insertions(+), 1 deletion(-)\n> > > >\n> > > >diff --git a/README.rst b/README.rst\n> > > >index 1427c714..0bfd39d9 100644\n> > > >--- a/README.rst\n> > > >+++ b/README.rst\n> > > >@@ -56,7 +56,9 @@ Meson Build system: [required]\n> > > >\n> > > > pip3 install --user meson\n> > > > pip3 install --user --upgrade meson\n> > > >-\n> > > >+\n> > > >+ If this fails, retry with `pip3 install meson`\n> > > >+\n> > >\n> > > This feels a little too broad as there are multiple reasons for this\n> > > command to fail. Can you show the error message that you encountered?\n> > >\n> > > > for the libcamera core: [required]\n> > > > python3-yaml python3-ply python3-jinja2\n> > > >\n> > > >--\n> > > >2.25.1\n> > >\n> > > Greetings,\n> > > Sebastian\n> > >\n> > > >\n> > > >_______________________________________________\n> > > >libcamera-devel mailing list\n> > > >libcamera-devel@lists.libcamera.org\n> > > >https://lists.libcamera.org/listinfo/libcamera-devel\n> > >\n> > _______________________________________________\n> > libcamera-devel mailing list\n> > libcamera-devel@lists.libcamera.org\n> > https://lists.libcamera.org/listinfo/libcamera-devel\n>\n>","headers":{"Return-Path":"","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id DAD0FBD80C\n\tfor ;\n\tMon, 22 Mar 2021 08:35:54 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 09E8A68D61;\n\tMon, 22 Mar 2021 09:35:54 +0100 (CET)","from mail-yb1-xb2d.google.com (mail-yb1-xb2d.google.com\n\t[IPv6:2607:f8b0:4864:20::b2d])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 8FAA46051E\n\tfor ;\n\tMon, 22 Mar 2021 09:35:51 +0100 (CET)","by mail-yb1-xb2d.google.com with SMTP id w8so5680909ybt.3\n\tfor ;\n\tMon, 22 Mar 2021 01:35:51 -0700 (PDT)"],"Authentication-Results":"lancelot.ideasonboard.com;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"N0s6JGQ7\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=mime-version:references:in-reply-to:from:date:message-id:subject:to\n\t:cc; bh=oWIWU0ZOzYImQgxCrkJu7ZPURPLQfJlojNDX7dzTR2Q=;\n\tb=N0s6JGQ7Att4pEen8O4wZPogJRqqo2k1ZxH8QU0alamcrie0jy3Nh4KDG4+NNS5nwe\n\tEb0zZI6b0WD0Km7SfVzrVAzSdIE4txL0xW4BqZB0TQMOsyoUxwNLSMMMafkwlsGJoY4Z\n\tk7qtwX9rXS8zH/tlhXGz/36xgW7s+ARyEMyu88z4JMammg049RJ18qzxnkXJ2P84svma\n\t8/IOmJcky2DMYK+uwNxt0MabYRuTY2XGsgYY6BJm6e2niKEBBhQPWPt6FZXHqDny7yf4\n\tqxZhspuD5J+NCOwKCTyNTWaUFtnmPQEpuJihhAGDW61SZXpkwVsX0b27938JX+gNUDaX\n\tju9A==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:references:in-reply-to:from:date\n\t:message-id:subject:to:cc;\n\tbh=oWIWU0ZOzYImQgxCrkJu7ZPURPLQfJlojNDX7dzTR2Q=;\n\tb=TGS/jHVwj5mpECTuF2EPTAC6ioIy39oFSgIHzJqj+h8T89aaFP8yWeVrzURNR9REG2\n\t+qp8BW7FXVupiuFto1hIMT3Br12n3gHvvextjGKTWQf0hvHInSlJkfLaCHNeWajyxq56\n\tHE6pd2CkxkjRmJJ2GJMeKKuKy1wjkbiMkad+b8w0Rfmy9niGibDBTE722VHB/m4lLHrj\n\tH2eScKf/dOwKe8vGgmrlHLuS2HH6DoPWsT2A3CGzdaykO6NCvUtdWBgkINSD47Z1ZYXE\n\t3NuM31yYzxIl0FURD5uDv+xYE9uqazhdj6VKb/amm81mHaxVeFD7+zX6A0UrVMs5GRZT\n\thNNg==","X-Gm-Message-State":"AOAM531iH6sv1nV8nbVo80RbaxLHbFf8hFOVFXdFvmj9lnc69LEeJkgk\n\tJJpjP3mCBg2bDVHmZ6c7PBfLWV24Z6InV/wi2HZo0DdBQlY=","X-Google-Smtp-Source":"ABdhPJw6vOh31JqvPAkpa2pj09uUVO0+4h3BiOcnkLJUjI8onRr66UWjr53Vvg8bFaQKajTLorkzG//ASDBm+esM/Kc=","X-Received":"by 2002:a25:ca52:: with SMTP id\n\ta79mr23867461ybg.248.1616402150313; \n\tMon, 22 Mar 2021 01:35:50 -0700 (PDT)","MIME-Version":"1.0","References":"<20210322074056.26330-1-vedantparanjape160201@gmail.com>\n\t<20210322081155.kbnclpfyey6r6g2m@basti-TUXEDO-Book-XA1510>\n\t\n\t<20210322083332.grbhf4cy2nvmg3ej@uno.localdomain>","In-Reply-To":"<20210322083332.grbhf4cy2nvmg3ej@uno.localdomain>","From":"Vedant Paranjape ","Date":"Mon, 22 Mar 2021 14:05:38 +0530","Message-ID":"","To":"Jacopo Mondi ","Subject":"Re: [libcamera-devel] [PATCH] Added alternative to meson install\n\tcommand, if it fails with --user argument","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"libcamera-devel@lists.libcamera.org","Content-Type":"multipart/mixed;\n\tboundary=\"===============8568352004809573573==\"","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}},{"id":15788,"web_url":"https://patchwork.libcamera.org/comment/15788/","msgid":"<20210322092929.GC9888@pyrite.rasen.tech>","date":"2021-03-22T09:29:29","subject":"Re: [libcamera-devel] [PATCH] Added alternative to meson install\n\tcommand, if it fails with --user argument","submitter":{"id":17,"url":"https://patchwork.libcamera.org/api/people/17/","name":"Paul Elder","email":"paul.elder@ideasonboard.com"},"content":"Hello Vedant,\n\nThank you for the patch.\n\nAs Jacopo pointed out, the subject should be more concise. You're also\nmissing a changelog.\n\nOn Mon, Mar 22, 2021 at 01:10:56PM +0530, Vedant Paranjape wrote:\n> Signed-off-by: Vedant Paranjape \n> ---\n> README.rst | 4 +++-\n> 1 file changed, 3 insertions(+), 1 deletion(-)\n> \n> diff --git a/README.rst b/README.rst\n> index 1427c714..0bfd39d9 100644\n> --- a/README.rst\n> +++ b/README.rst\n> @@ -56,7 +56,9 @@ Meson Build system: [required]\n> \n> pip3 install --user meson\n> pip3 install --user --upgrade meson\n> -\n> + \n> + If this fails, retry with `pip3 install meson`\n> + \n\nI don't think this is the right place, and I don't think there's enough\ninformation.\n\nThis is the dependencies section, so I think your supplemental\ninformation should go before \"Dependencies\", and maybe after the code\nthat's after \"To fetch the sources, build and install\".\n\niirc the issue was about a potential version mismatch beteween the\nversion that the root user uses, and the version that your user uses. So\nyou should outline this issue, and then provide a solution to it. Maybe\nrecommending not to use --user if meson is already installed\nsystem-wide?\n\n\nPaul\n\n> for the libcamera core: [required]\n> python3-yaml python3-ply python3-jinja2\n> \n> -- \n> 2.25.1","headers":{"Return-Path":"","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id A7A73BD80C\n\tfor ;\n\tMon, 22 Mar 2021 09:29:56 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id B325468D58;\n\tMon, 22 Mar 2021 10:29:55 +0100 (CET)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 6DC806051E\n\tfor ;\n\tMon, 22 Mar 2021 10:29:39 +0100 (CET)","from pyrite.rasen.tech (unknown\n\t[IPv6:2400:4051:61:600:2c71:1b79:d06d:5032])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 62C33AD6;\n\tMon, 22 Mar 2021 10:29:36 +0100 (CET)"],"Authentication-Results":"lancelot.ideasonboard.com;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"s0+7F0nl\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1616405377;\n\tbh=6dgQX/Uo0Ly+GqhKXfKFjL/DP0L43y0YIojXWUnErHE=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=s0+7F0nlqLeBOlq6tjwqRstPo2BNfOQRncSXayzL0KLrTd++heatEOw8HHDMabY6w\n\twh4EcBVdodBKueLmkdrKSbwAhYqto3mhBg7m1pchbOtUDWo+Y8gX5SomR5/Y4Vedi5\n\tl3UlWv6WlewRVE5TyFJw6gLSGmVAKMPZuZQgWCTE=","Date":"Mon, 22 Mar 2021 18:29:29 +0900","From":"paul.elder@ideasonboard.com","To":"Vedant Paranjape ","Message-ID":"<20210322092929.GC9888@pyrite.rasen.tech>","References":"<20210322074056.26330-1-vedantparanjape160201@gmail.com>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20210322074056.26330-1-vedantparanjape160201@gmail.com>","Subject":"Re: [libcamera-devel] [PATCH] Added alternative to meson install\n\tcommand, if it fails with --user argument","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"libcamera-devel@lists.libcamera.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}}]