[{"id":19583,"web_url":"https://patchwork.libcamera.org/comment/19583/","msgid":"","date":"2021-09-09T15:07:43","subject":"Re: [libcamera-devel] [PATCH] README: Fix android dependencies","submitter":{"id":2,"url":"https://patchwork.libcamera.org/api/people/2/","name":"Laurent Pinchart","email":"laurent.pinchart@ideasonboard.com"},"content":"Hi Kieran,\n\nThank you for the patch.\n\nOn Thu, Sep 09, 2021 at 04:01:37PM +0100, Kieran Bingham wrote:\n> While the distribution is unspecified, The README.rst gives\n> debian/ubuntu named dependencies for all except the android\n> requirements.\n> \n> Update the android dependencies to match the others, by showing that the\n> -dev package is required.\n> \n> Signed-off-by: Kieran Bingham \n> ---\n> README.rst | 2 +-\n> 1 file changed, 1 insertion(+), 1 deletion(-)\n> \n> diff --git a/README.rst b/README.rst\n> index cf81eca42149..6eb25c708180 100644\n> --- a/README.rst\n> +++ b/README.rst\n> @@ -88,7 +88,7 @@ for tracing with lttng: [optional]\n> liblttng-ust-dev python3-jinja2 lttng-tools\n> \n> for android: [optional]\n> - libexif libjpeg libyaml\n> + libexif-dev libjpeg-dev libyaml-dev\n\nI'm curious to know how you'll use the Android camera HAL on a\nDebian-based distribution :-) It's coherent with the rest of the file\nthough, so\n\nReviewed-by: Laurent Pinchart \n\n> Using GStreamer plugin\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 B1844BDB1D\n\tfor ;\n\tThu, 9 Sep 2021 15:08:06 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 78CE169170;\n\tThu, 9 Sep 2021 17:08:06 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 8120F6916B\n\tfor ;\n\tThu, 9 Sep 2021 17:08:04 +0200 (CEST)","from pendragon.ideasonboard.com (62-78-145-57.bb.dnainternet.fi\n\t[62.78.145.57])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 1742C883;\n\tThu, 9 Sep 2021 17:08:04 +0200 (CEST)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"is1LXsU0\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1631200084;\n\tbh=Z3xL/egGQf6A/zYUlAyCBncispnOaOEsd2VjtAB1i9M=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=is1LXsU0ph3RCL3YM960N5ZcJ0fex+JnwxDhKwpB6V9HWbHFcXgdbmfSR5AWn9wwh\n\tKknfoUX4IEgxCZfaCdR/xU2cXPe+/Si3RtB0fzEmgu8GKnADCtMEj8FvG4Saw8CXbv\n\tlUR/co17/06oa2mvelEKFvKz/hkQfWWxrIuIt+ZM=","Date":"Thu, 9 Sep 2021 18:07:43 +0300","From":"Laurent Pinchart ","To":"Kieran Bingham ","Message-ID":"","References":"<20210909150137.4011590-1-kieran.bingham@ideasonboard.com>","MIME-Version":"1.0","Content-Type":"text/plain; charset=utf-8","Content-Disposition":"inline","In-Reply-To":"<20210909150137.4011590-1-kieran.bingham@ideasonboard.com>","Subject":"Re: [libcamera-devel] [PATCH] README: Fix android dependencies","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 ","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}},{"id":19584,"web_url":"https://patchwork.libcamera.org/comment/19584/","msgid":"<2a074eb6-6c7d-7724-3c8a-8519a39eda63@ideasonboard.com>","date":"2021-09-09T15:12:34","subject":"Re: [libcamera-devel] [PATCH] README: Fix android dependencies","submitter":{"id":4,"url":"https://patchwork.libcamera.org/api/people/4/","name":"Kieran Bingham","email":"kieran.bingham@ideasonboard.com"},"content":"On 09/09/2021 16:07, Laurent Pinchart wrote:\n> Hi Kieran,\n> \n> Thank you for the patch.\n> \n> On Thu, Sep 09, 2021 at 04:01:37PM +0100, Kieran Bingham wrote:\n>> While the distribution is unspecified, The README.rst gives\n>> debian/ubuntu named dependencies for all except the android\n>> requirements.\n>>\n>> Update the android dependencies to match the others, by showing that the\n>> -dev package is required.\n>>\n>> Signed-off-by: Kieran Bingham \n>> ---\n>> README.rst | 2 +-\n>> 1 file changed, 1 insertion(+), 1 deletion(-)\n>>\n>> diff --git a/README.rst b/README.rst\n>> index cf81eca42149..6eb25c708180 100644\n>> --- a/README.rst\n>> +++ b/README.rst\n>> @@ -88,7 +88,7 @@ for tracing with lttng: [optional]\n>> liblttng-ust-dev python3-jinja2 lttng-tools\n>> \n>> for android: [optional]\n>> - libexif libjpeg libyaml\n>> + libexif-dev libjpeg-dev libyaml-dev\n> \n> I'm curious to know how you'll use the Android camera HAL on a\n> Debian-based distribution :-) It's coherent with the rest of the file\n> though, so\n\nTrue ... but I compile test with everything enabled, including Android,\nand saw this while trying to dockerise the build so the whole build\nenvironment was specified.\n\nIt's mostly the consistency indeed here, as you're right. We wouldn't\nnormally expect the android HAL to be compiled otherwise.\n\nIt even makes me doubt the patch entirely ...\n\n> Reviewed-by: Laurent Pinchart \n> \n>> Using GStreamer plugin\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 D9B55BDC71\n\tfor ;\n\tThu, 9 Sep 2021 15:12:38 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 4D5636916E;\n\tThu, 9 Sep 2021 17:12:38 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id C2E306916B\n\tfor ;\n\tThu, 9 Sep 2021 17:12:37 +0200 (CEST)","from [192.168.0.20]\n\t(cpc89244-aztw30-2-0-cust3082.18-1.cable.virginm.net [86.31.172.11])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 1D9D6883;\n\tThu, 9 Sep 2021 17:12:37 +0200 (CEST)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"d7KgzlEF\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1631200357;\n\tbh=AfIrFwbWa22zaJ4lKcpLdzl1WxlVONGlxhq3qIdgXdY=;\n\th=From:Subject:To:Cc:References:Date:In-Reply-To:From;\n\tb=d7KgzlEFS374UTFcpMKSG9GcFhXAq0x224wPIF6P3g7SC1/fQrppHx8NzEkBs6d3k\n\tKLwVhksZ+/X+HvmsuEhoyF6NPOSrpb9J/q1vqTJOmlwiwDKswGTnATaiSneAjifodj\n\t3D1hJeq1QWi7M9CtDvPw6HBasDLwfdveHViMEI8A=","From":"Kieran Bingham ","To":"Laurent Pinchart ","References":"<20210909150137.4011590-1-kieran.bingham@ideasonboard.com>\n\t","Message-ID":"<2a074eb6-6c7d-7724-3c8a-8519a39eda63@ideasonboard.com>","Date":"Thu, 9 Sep 2021 16:12:34 +0100","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101\n\tThunderbird/78.11.0","MIME-Version":"1.0","In-Reply-To":"","Content-Type":"text/plain; charset=utf-8","Content-Language":"en-GB","Content-Transfer-Encoding":"7bit","Subject":"Re: [libcamera-devel] [PATCH] README: Fix android dependencies","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 ","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}},{"id":19588,"web_url":"https://patchwork.libcamera.org/comment/19588/","msgid":"<20210910004141.GB4137293@pyrite.rasen.tech>","date":"2021-09-10T00:41:41","subject":"Re: [libcamera-devel] [PATCH] README: Fix android dependencies","submitter":{"id":17,"url":"https://patchwork.libcamera.org/api/people/17/","name":"Paul Elder","email":"paul.elder@ideasonboard.com"},"content":"Hi Kieran,\n\nOn Thu, Sep 09, 2021 at 04:01:37PM +0100, Kieran Bingham wrote:\n> While the distribution is unspecified, The README.rst gives\n> debian/ubuntu named dependencies for all except the android\n> requirements.\n> \n> Update the android dependencies to match the others, by showing that the\n> -dev package is required.\n> \n> Signed-off-by: Kieran Bingham \n\nReviewed-by: Paul Elder \n\n> ---\n> README.rst | 2 +-\n> 1 file changed, 1 insertion(+), 1 deletion(-)\n> \n> diff --git a/README.rst b/README.rst\n> index cf81eca42149..6eb25c708180 100644\n> --- a/README.rst\n> +++ b/README.rst\n> @@ -88,7 +88,7 @@ for tracing with lttng: [optional]\n> liblttng-ust-dev python3-jinja2 lttng-tools\n> \n> for android: [optional]\n> - libexif libjpeg libyaml\n> + libexif-dev libjpeg-dev libyaml-dev\n> \n> Using GStreamer plugin\n> ~~~~~~~~~~~~~~~~~~~~~~\n> -- \n> 2.30.2\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 2A1C2BDC71\n\tfor ;\n\tFri, 10 Sep 2021 00:41:53 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 6267E6916E;\n\tFri, 10 Sep 2021 02:41:52 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id F2E1D60503\n\tfor ;\n\tFri, 10 Sep 2021 02:41:49 +0200 (CEST)","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 87890883;\n\tFri, 10 Sep 2021 02:41:48 +0200 (CEST)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"UmjAgHw6\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1631234509;\n\tbh=l2uvHeFpzuBhBB5xQ5jMjUA0W4A60XVBU1h98L1rTXA=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=UmjAgHw660oBvh/pRNinjAKdbM+z8pZ5aR3lig/ke9q78Xs29OkY3UJs+WyFsycgt\n\tZEUi6A7K9wJUWIFJPc6p7Ou0qsL+NaVCHxG6hSDVVanTawEvW37uVAvBZUH4mojrAV\n\tMYXJ1DaIY+sB1TjQicQnpz7IalMB2de+2L7ObxQY=","Date":"Fri, 10 Sep 2021 09:41:41 +0900","From":"paul.elder@ideasonboard.com","To":"Kieran Bingham ","Message-ID":"<20210910004141.GB4137293@pyrite.rasen.tech>","References":"<20210909150137.4011590-1-kieran.bingham@ideasonboard.com>","MIME-Version":"1.0","Content-Type":"text/plain; charset=us-ascii","Content-Disposition":"inline","In-Reply-To":"<20210909150137.4011590-1-kieran.bingham@ideasonboard.com>","Subject":"Re: [libcamera-devel] [PATCH] README: Fix android dependencies","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 ","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}},{"id":19597,"web_url":"https://patchwork.libcamera.org/comment/19597/","msgid":"","date":"2021-09-10T08:25:42","subject":"Re: [libcamera-devel] [PATCH] README: Fix android dependencies","submitter":{"id":63,"url":"https://patchwork.libcamera.org/api/people/63/","name":"Hirokazu Honda","email":"hiroh@chromium.org"},"content":"Hi Kieran, thank you for the patch.\n\nOn Fri, Sep 10, 2021 at 9:41 AM wrote:\n>\n> Hi Kieran,\n>\n> On Thu, Sep 09, 2021 at 04:01:37PM +0100, Kieran Bingham wrote:\n> > While the distribution is unspecified, The README.rst gives\n> > debian/ubuntu named dependencies for all except the android\n> > requirements.\n> >\n> > Update the android dependencies to match the others, by showing that the\n> > -dev package is required.\n> >\n> > Signed-off-by: Kieran Bingham \n>\n> Reviewed-by: Paul Elder \n>\n\nI thought the direction was opposite. I would rather drop \"-dev\" of\nother dependencies.\nA user shall look for proper packages in their platform for dependencies.\nHaving \"-dev\" seems to centralize debian/ubuntu package systems. How\nabout other distributions..?\n\n-Hiro\n> > ---\n> > README.rst | 2 +-\n> > 1 file changed, 1 insertion(+), 1 deletion(-)\n> >\n> > diff --git a/README.rst b/README.rst\n> > index cf81eca42149..6eb25c708180 100644\n> > --- a/README.rst\n> > +++ b/README.rst\n> > @@ -88,7 +88,7 @@ for tracing with lttng: [optional]\n> > liblttng-ust-dev python3-jinja2 lttng-tools\n> >\n> > for android: [optional]\n> > - libexif libjpeg libyaml\n> > + libexif-dev libjpeg-dev libyaml-dev\n> >\n> > Using GStreamer plugin\n> > ~~~~~~~~~~~~~~~~~~~~~~\n> > --\n> > 2.30.2\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 EB3F0BDB1D\n\tfor ;\n\tFri, 10 Sep 2021 08:25:55 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 2EC0C6916E;\n\tFri, 10 Sep 2021 10:25:55 +0200 (CEST)","from mail-ed1-x52b.google.com (mail-ed1-x52b.google.com\n\t[IPv6:2a00:1450:4864:20::52b])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 754746024A\n\tfor ;\n\tFri, 10 Sep 2021 10:25:53 +0200 (CEST)","by mail-ed1-x52b.google.com with SMTP id z19so1333162edi.9\n\tfor ;\n\tFri, 10 Sep 2021 01:25:53 -0700 (PDT)"],"Authentication-Results":"lancelot.ideasonboard.com;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=chromium.org header.i=@chromium.org\n\theader.b=\"VVehPplD\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org;\n\ts=google; \n\th=mime-version:references:in-reply-to:from:date:message-id:subject:to\n\t:cc; bh=1rP+M5lY7ILZQYNFaY8wU2cKUCDVyxak93gYqe8IAFU=;\n\tb=VVehPplDrDIQaTuQA4iLYgc9L3pWfJ8/+Ra9pSX02N7Z6gtEg2mGuUT5OAplvG500U\n\trTtEQzUIouJ/zRNvAvtzD6Fj7YZhPEYtupp8kTXMu3ctfO7AZeZ9aNErHbh4ftVJsajM\n\tij35VI/CXdVLzIbpB67hDSCyXoxFkde5x/cOo=","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20210112;\n\th=x-gm-message-state:mime-version:references:in-reply-to:from:date\n\t:message-id:subject:to:cc;\n\tbh=1rP+M5lY7ILZQYNFaY8wU2cKUCDVyxak93gYqe8IAFU=;\n\tb=AdS6iDA72Nf7xZOWW3i6KNm/LJ7ytZH7w4thOL81WlXjCV4MUb2/+8MnEVbfADDwUb\n\tFokupnWMeC+F668M3VyIsGtH+c7A3/bswkOQmnXHd19Z9rX9or0PrV1ij80kBCJ+zlIp\n\t2LbV0cPbYwHCnboxFIdCl0T1eXDQQHZYZob26GMj90GZA/kZgunJVV71K9KQtLk3sei6\n\tUkp6X74ojtPGkkXd0dxzDY7lZOGCb0/GGb3MwnhFZ0CVqNX6bYTw+7pB5JaXiqxZnHgN\n\tLSRcLtF2OkkKDfdYGImAN8tWJGzNATy7x43LgOeOU1TTA0BQ7SzH/5cI0lIqjRIlo5b9\n\thLlA==","X-Gm-Message-State":"AOAM5333HdzqOwQalMddvDnNQJjGyEwZ4vL52fAZCoSOacXnp2mQoavp\n\t49pAKKKN71FLfNoXbw8MZJU3mpsE5ZcrVF4t+jl56F13E4A=","X-Google-Smtp-Source":"ABdhPJzLlsHwGXD7NbonrbUFsEs47WwCaUbjDKLqJRZKXuH5iwdamxJ6kQcI0C5CUKYpubG/cznor768EwezBayFFaQ=","X-Received":"by 2002:aa7:da93:: with SMTP id\n\tq19mr7841970eds.206.1631262352949; \n\tFri, 10 Sep 2021 01:25:52 -0700 (PDT)","MIME-Version":"1.0","References":"<20210909150137.4011590-1-kieran.bingham@ideasonboard.com>\n\t<20210910004141.GB4137293@pyrite.rasen.tech>","In-Reply-To":"<20210910004141.GB4137293@pyrite.rasen.tech>","From":"Hirokazu Honda ","Date":"Fri, 10 Sep 2021 17:25:42 +0900","Message-ID":"","To":"Paul Elder ","Content-Type":"text/plain; charset=\"UTF-8\"","Subject":"Re: [libcamera-devel] [PATCH] README: Fix android dependencies","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 ","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}},{"id":19606,"web_url":"https://patchwork.libcamera.org/comment/19606/","msgid":"","date":"2021-09-10T15:00:08","subject":"Re: [libcamera-devel] [PATCH] README: Fix android dependencies","submitter":{"id":2,"url":"https://patchwork.libcamera.org/api/people/2/","name":"Laurent Pinchart","email":"laurent.pinchart@ideasonboard.com"},"content":"Hi Hiro,\n\nOn Fri, Sep 10, 2021 at 05:25:42PM +0900, Hirokazu Honda wrote:\n> On Fri, Sep 10, 2021 at 9:41 AM wrote:\n> > On Thu, Sep 09, 2021 at 04:01:37PM +0100, Kieran Bingham wrote:\n> > > While the distribution is unspecified, The README.rst gives\n> > > debian/ubuntu named dependencies for all except the android\n> > > requirements.\n> > >\n> > > Update the android dependencies to match the others, by showing that the\n> > > -dev package is required.\n> > >\n> > > Signed-off-by: Kieran Bingham \n> >\n> > Reviewed-by: Paul Elder \n> \n> I thought the direction was opposite. I would rather drop \"-dev\" of\n> other dependencies.\n> A user shall look for proper packages in their platform for dependencies.\n> Having \"-dev\" seems to centralize debian/ubuntu package systems. How\n> about other distributions..?\n\nMost of us, and it seems a large part of our users, use Debian-based\ndistributions (I use Gentoo personally ;-)). I think it would be nice to\nlist depednencies for other distributions too, if someone wants to\nsubmit a patch. I'm not sure how to best format the data though, we may\nneed to turn it into a table.\n\n> > > ---\n> > > README.rst | 2 +-\n> > > 1 file changed, 1 insertion(+), 1 deletion(-)\n> > >\n> > > diff --git a/README.rst b/README.rst\n> > > index cf81eca42149..6eb25c708180 100644\n> > > --- a/README.rst\n> > > +++ b/README.rst\n> > > @@ -88,7 +88,7 @@ for tracing with lttng: [optional]\n> > > liblttng-ust-dev python3-jinja2 lttng-tools\n> > >\n> > > for android: [optional]\n> > > - libexif libjpeg libyaml\n> > > + libexif-dev libjpeg-dev libyaml-dev\n> > >\n> > > Using GStreamer plugin\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 1A16EBDB1D\n\tfor ;\n\tFri, 10 Sep 2021 15:00:33 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 6A89469170;\n\tFri, 10 Sep 2021 17:00:32 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 9734E69169\n\tfor ;\n\tFri, 10 Sep 2021 17:00:30 +0200 (CEST)","from pendragon.ideasonboard.com (62-78-145-57.bb.dnainternet.fi\n\t[62.78.145.57])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 05107DD;\n\tFri, 10 Sep 2021 17:00:29 +0200 (CEST)"],"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=\"dB0uHfPY\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1631286030;\n\tbh=NJwRGY1KROme69bRmLDt7o6+CCts++vW3VsIyugDuVg=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=dB0uHfPYTTo9gDP99Ab/b2gsiRC5cfSZ8psZZiO51K//WQTC0OgoCl3QglyWwsK3M\n\t0D6bWzeEdMrTZidsXwKXivsFPVuvfBVw6XaCzFwUdFKCESTP/9F9TbXGbcojiE6LXl\n\tGGg8PWdlV9HFaDn13PeXZjl/+P72llInkYre8zmI=","Date":"Fri, 10 Sep 2021 18:00:08 +0300","From":"Laurent Pinchart ","To":"Hirokazu Honda ","Message-ID":"","References":"<20210909150137.4011590-1-kieran.bingham@ideasonboard.com>\n\t<20210910004141.GB4137293@pyrite.rasen.tech>\n\t","MIME-Version":"1.0","Content-Type":"text/plain; charset=utf-8","Content-Disposition":"inline","In-Reply-To":"","Subject":"Re: [libcamera-devel] [PATCH] README: Fix android dependencies","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 ","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}},{"id":19612,"web_url":"https://patchwork.libcamera.org/comment/19612/","msgid":"","date":"2021-09-10T16:07:23","subject":"Re: [libcamera-devel] [PATCH] README: Fix android dependencies","submitter":{"id":63,"url":"https://patchwork.libcamera.org/api/people/63/","name":"Hirokazu Honda","email":"hiroh@chromium.org"},"content":"Hi Laurent,\n\nOn Sat, Sep 11, 2021 at 12:00 AM Laurent Pinchart\n wrote:\n>\n> Hi Hiro,\n>\n> On Fri, Sep 10, 2021 at 05:25:42PM +0900, Hirokazu Honda wrote:\n> > On Fri, Sep 10, 2021 at 9:41 AM wrote:\n> > > On Thu, Sep 09, 2021 at 04:01:37PM +0100, Kieran Bingham wrote:\n> > > > While the distribution is unspecified, The README.rst gives\n> > > > debian/ubuntu named dependencies for all except the android\n> > > > requirements.\n> > > >\n> > > > Update the android dependencies to match the others, by showing that the\n> > > > -dev package is required.\n> > > >\n> > > > Signed-off-by: Kieran Bingham \n> > >\n> > > Reviewed-by: Paul Elder \n> >\n> > I thought the direction was opposite. I would rather drop \"-dev\" of\n> > other dependencies.\n> > A user shall look for proper packages in their platform for dependencies.\n> > Having \"-dev\" seems to centralize debian/ubuntu package systems. How\n> > about other distributions..?\n>\n> Most of us, and it seems a large part of our users, use Debian-based\n> distributions (I use Gentoo personally ;-)). I think it would be nice to\n> list depednencies for other distributions too, if someone wants to\n> submit a patch. I'm not sure how to best format the data though, we may\n> need to turn it into a table.\n>\n\nI sometimes see the command lines to install dependent packages in\neach systems like\nDebian/Ubuntu\n`apt install ....`\nArch Linux\n`pacman -S`\n\nAnyway, I am not against this patch of fixing the consistency.\n\nReviewed-by: Hirokazu Honda \n\n-Hiro\n> > > > ---\n> > > > README.rst | 2 +-\n> > > > 1 file changed, 1 insertion(+), 1 deletion(-)\n> > > >\n> > > > diff --git a/README.rst b/README.rst\n> > > > index cf81eca42149..6eb25c708180 100644\n> > > > --- a/README.rst\n> > > > +++ b/README.rst\n> > > > @@ -88,7 +88,7 @@ for tracing with lttng: [optional]\n> > > > liblttng-ust-dev python3-jinja2 lttng-tools\n> > > >\n> > > > for android: [optional]\n> > > > - libexif libjpeg libyaml\n> > > > + libexif-dev libjpeg-dev libyaml-dev\n> > > >\n> > > > Using GStreamer plugin\n> > > > ~~~~~~~~~~~~~~~~~~~~~~\n>\n> --\n> Regards,\n>\n> Laurent Pinchart","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 3EC6FBDC71\n\tfor ;\n\tFri, 10 Sep 2021 16:07:37 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 971DE69170;\n\tFri, 10 Sep 2021 18:07:36 +0200 (CEST)","from mail-ed1-x536.google.com (mail-ed1-x536.google.com\n\t[IPv6:2a00:1450:4864:20::536])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 0263669169\n\tfor ;\n\tFri, 10 Sep 2021 18:07:34 +0200 (CEST)","by mail-ed1-x536.google.com with SMTP id r7so3190731edd.6\n\tfor ;\n\tFri, 10 Sep 2021 09:07:34 -0700 (PDT)"],"Authentication-Results":"lancelot.ideasonboard.com;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=chromium.org header.i=@chromium.org\n\theader.b=\"Wsc/KyQK\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org;\n\ts=google; \n\th=mime-version:references:in-reply-to:from:date:message-id:subject:to\n\t:cc; bh=vh31sgeqnNN0o0BWetGYqqkXHUAH7u6lEMf6ODXmw6Q=;\n\tb=Wsc/KyQKMPy4xnJgCCh68RR0tcJCnPIwrMj1H88BoxBkD8EhrF2fjOw0573ndapueL\n\tEM0BH2PjB7Px7JbC8uXeZnk5WYcOTxg5GXg3WV7c3sZBSOVCdcGZNuqsi3PE2VvzScpT\n\t/enVgbsVlU0PSnHVX/XqPlzuuFUm11KsnePak=","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20210112;\n\th=x-gm-message-state:mime-version:references:in-reply-to:from:date\n\t:message-id:subject:to:cc;\n\tbh=vh31sgeqnNN0o0BWetGYqqkXHUAH7u6lEMf6ODXmw6Q=;\n\tb=b8DRJppyE2wF6/uHDrCYfpTt9cTXrwlCkRcOmp+y5aEinUzmOMtOvhIybfOiVXS9h9\n\tlxUW8vLjnlf+zC2lOteswVxJBdN+eA7Y0f7FdIEC5C08IrSQbSUycWfzfrUyL4Ak2yL/\n\tzSiZyYbSnpn9GN4DYzmSJq371wJDlox+qvQIp1Z1OVJdazoTCkm2rqYwDDmgsZHSds0s\n\teTgCxG6cplM52TbXuuOVX3Db+Z6UxyN8Q2KfXT/E/kwBmSlGGA1VW/6bE73xibpll3A/\n\thZZIcNQHWr72rawgGL0YOl6RqYr34qs1jsZPoS5yCD3TaHfP96jmleMpMOrfZPzZaN21\n\ttL3Q==","X-Gm-Message-State":"AOAM531vL+RwLjnfb+t/UiP0ajdUquRqUXjcey4JvlkLJ7ym2cvVUxAc\n\tUayXu7TooCBaUE6MUD5qIT0OBvxIHhStK7HdSfUArj0QV+Y=","X-Google-Smtp-Source":"ABdhPJwYN0+sxMJmcIbmZO16V9UFIyk3YTqzm3r7L3xRkcRED5hj3/Ss9lFy9xO1dG69KCRGqJILvb3zxqeqHeEyQEo=","X-Received":"by 2002:aa7:da93:: with SMTP id\n\tq19mr9968044eds.206.1631290054580; \n\tFri, 10 Sep 2021 09:07:34 -0700 (PDT)","MIME-Version":"1.0","References":"<20210909150137.4011590-1-kieran.bingham@ideasonboard.com>\n\t<20210910004141.GB4137293@pyrite.rasen.tech>\n\t\n\t","In-Reply-To":"","From":"Hirokazu Honda ","Date":"Sat, 11 Sep 2021 01:07:23 +0900","Message-ID":"","To":"Laurent Pinchart ","Content-Type":"text/plain; charset=\"UTF-8\"","Subject":"Re: [libcamera-devel] [PATCH] README: Fix android dependencies","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 ","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}},{"id":19614,"web_url":"https://patchwork.libcamera.org/comment/19614/","msgid":"<13824789-3850-e8fb-2689-9f1ab6bf307c@ideasonboard.com>","date":"2021-09-10T16:17:59","subject":"Re: [libcamera-devel] [PATCH] README: Fix android dependencies","submitter":{"id":4,"url":"https://patchwork.libcamera.org/api/people/4/","name":"Kieran Bingham","email":"kieran.bingham@ideasonboard.com"},"content":"On 10/09/2021 17:07, Hirokazu Honda wrote:\n> Hi Laurent,\n> \n> On Sat, Sep 11, 2021 at 12:00 AM Laurent Pinchart\n> wrote:\n>>\n>> Hi Hiro,\n>>\n>> On Fri, Sep 10, 2021 at 05:25:42PM +0900, Hirokazu Honda wrote:\n>>> On Fri, Sep 10, 2021 at 9:41 AM wrote:\n>>>> On Thu, Sep 09, 2021 at 04:01:37PM +0100, Kieran Bingham wrote:\n>>>>> While the distribution is unspecified, The README.rst gives\n>>>>> debian/ubuntu named dependencies for all except the android\n>>>>> requirements.\n>>>>>\n>>>>> Update the android dependencies to match the others, by showing that the\n>>>>> -dev package is required.\n>>>>>\n>>>>> Signed-off-by: Kieran Bingham \n>>>>\n>>>> Reviewed-by: Paul Elder \n>>>\n>>> I thought the direction was opposite. I would rather drop \"-dev\" of\n>>> other dependencies.\n>>> A user shall look for proper packages in their platform for dependencies.\n>>> Having \"-dev\" seems to centralize debian/ubuntu package systems. How\n>>> about other distributions..?\n>>\n>> Most of us, and it seems a large part of our users, use Debian-based\n>> distributions (I use Gentoo personally ;-)). I think it would be nice to\n>> list depednencies for other distributions too, if someone wants to\n>> submit a patch. I'm not sure how to best format the data though, we may\n>> need to turn it into a table.\n>>\n> \n> I sometimes see the command lines to install dependent packages in\n> each systems like\n> Debian/Ubuntu\n> `apt install ....`\n> Arch Linux\n> `pacman -S`\n\nOne of my half baked ideas was to put these dependencies in a table in a\nscript and allow:\n\n ./utils/bootstrap-deps.sh gcc cam qcam gstreamer hotplug\n\nor\n\n ./utils/bootstrap-deps.sh all\n\nwhich would install the relevant dependencies for those categories.\n\nIt's stashed on one of my development branches waiting for some\ninspiration or motivation to make it more generic.\n\nIf anyone is interested in having a go at continuing it ...\n https://gist.github.com/kbingham/667e2b95acfbbdd726764fcee632c8b0\n\n\n> Anyway, I am not against this patch of fixing the consistency.\n> \n> Reviewed-by: Hirokazu Honda \n\nThanks, I'd rather merge this for consistency, and if we later decide to\nuse just the library names adapt then. But somehow we might want to\ndistinguish between runtime and compile time dependencies if we make it\ngeneric...\n\n\n\n> \n> -Hiro\n>>>>> ---\n>>>>> README.rst | 2 +-\n>>>>> 1 file changed, 1 insertion(+), 1 deletion(-)\n>>>>>\n>>>>> diff --git a/README.rst b/README.rst\n>>>>> index cf81eca42149..6eb25c708180 100644\n>>>>> --- a/README.rst\n>>>>> +++ b/README.rst\n>>>>> @@ -88,7 +88,7 @@ for tracing with lttng: [optional]\n>>>>> liblttng-ust-dev python3-jinja2 lttng-tools\n>>>>>\n>>>>> for android: [optional]\n>>>>> - libexif libjpeg libyaml\n>>>>> + libexif-dev libjpeg-dev libyaml-dev\n>>>>>\n>>>>> Using GStreamer plugin\n>>>>> ~~~~~~~~~~~~~~~~~~~~~~\n>>\n>> --\n>> Regards,\n>>\n>> Laurent Pinchart","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 31C80BDB1D\n\tfor ;\n\tFri, 10 Sep 2021 16:18:03 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id AE5276917C;\n\tFri, 10 Sep 2021 18:18:02 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 24E5769169\n\tfor ;\n\tFri, 10 Sep 2021 18:18:02 +0200 (CEST)","from [192.168.0.20]\n\t(cpc89244-aztw30-2-0-cust3082.18-1.cable.virginm.net [86.31.172.11])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 929D8883;\n\tFri, 10 Sep 2021 18:18:01 +0200 (CEST)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"SMMf0EZQ\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1631290681;\n\tbh=YL4FplCUnEJFuNcNuL3KHpfYZRZOMj3o2XtgP82A+MQ=;\n\th=To:Cc:References:From:Subject:Date:In-Reply-To:From;\n\tb=SMMf0EZQzoqtpRzLULNQxrnb08iGtabhSKrnu+TyqBzIwBY6BmPe78jXuWdVcNVB6\n\tl9HC5ZqpaFHnKQNmzGJmcx9opbekHYu/JUjJ/e+eZ0TpIzcy4lo9lCRVSmNdcH6aIX\n\tMpyX1yTX1ocExb0W0wI1B8cXVxfRa6f9NE6SGN4o=","To":"Hirokazu Honda ,\n\tLaurent Pinchart ","References":"<20210909150137.4011590-1-kieran.bingham@ideasonboard.com>\n\t<20210910004141.GB4137293@pyrite.rasen.tech>\n\t\n\t\n\t","From":"Kieran Bingham ","Message-ID":"<13824789-3850-e8fb-2689-9f1ab6bf307c@ideasonboard.com>","Date":"Fri, 10 Sep 2021 17:17:59 +0100","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101\n\tThunderbird/78.11.0","MIME-Version":"1.0","In-Reply-To":"","Content-Type":"text/plain; charset=utf-8","Content-Language":"en-GB","Content-Transfer-Encoding":"8bit","Subject":"Re: [libcamera-devel] [PATCH] README: Fix android dependencies","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 ","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}}]