[{"id":272,"web_url":"https://patchwork.libcamera.org/comment/272/","msgid":"<5b42d520-552b-0119-d8f4-587939bbe87d@ideasonboard.com>","date":"2019-01-09T17:07:29","subject":"Re: [libcamera-devel] [PATCH v2 02/12] libcamera: add dependency on\n\tlibudev","submitter":{"id":4,"url":"https://patchwork.libcamera.org/api/people/4/","name":"Kieran Bingham","email":"kieran.bingham@ideasonboard.com"},"content":"Hi Niklas, Laurent,\n\nOn 29/12/2018 03:28, Niklas Söderlund wrote:\n> The device enumeration will depend on libudev, add the dependency to the\n> build system. This should be turned into a optional dependency once a\n> device enumerator not using udev is supported.\n> \n> Signed-off-by: Niklas Söderlund \n> Reviewed-by: Laurent Pinchart \n> ---\n> src/libcamera/meson.build | 5 ++++-\n> 1 file changed, 4 insertions(+), 1 deletion(-)\n> \n> diff --git a/src/libcamera/meson.build b/src/libcamera/meson.build\n> index 46591069aa5f8beb..52b556a8ed4050cb 100644\n> --- a/src/libcamera/meson.build\n> +++ b/src/libcamera/meson.build\n> @@ -16,7 +16,10 @@ includes = [\n> libcamera_internal_includes,\n> ]\n> \n> +libudev = dependency('libudev')\n> +\n\nThe website currently states that we will not depend on anything except\nthe standard libraries [0].\n\nI guess this will be updated when we move the website to build from the\nrepo, although I can't see anywhere within Documentation listing\nanything similar. Is there a section missing in our source level docs?\n\n[0] http://www.libcamera.org/docs.html#technical-requirements\n\n\n--\nKieran\n\n\n> libcamera = shared_library('camera',\n> libcamera_sources,\n> install : true,\n> - include_directories : includes)\n> + include_directories : includes,\n> + dependencies : libudev)\n>","headers":{"Return-Path":"","Received":["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 3598A60B2D\n\tfor ;\n\tWed, 9 Jan 2019 18:07:33 +0100 (CET)","from [192.168.0.21]\n\t(cpc89242-aztw30-2-0-cust488.18-1.cable.virginm.net [86.31.129.233])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 57EFA56D;\n\tWed, 9 Jan 2019 18:07:32 +0100 (CET)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1547053652;\n\tbh=7/q5MfWR6dQlZ+VqKVOj29GavFsE4sTINQ7FJG2Uz/U=;\n\th=Reply-To:Subject:To:References:From:Date:In-Reply-To:From;\n\tb=DoFN8Zwg6LLujEJc2PCC/QfxVseYkf+CE9nN4ljzOGCtJ13bBsao7nCJQ5rfIanEc\n\tnKsbB8cuWCgBBGC6mM8GJlh1IfuTTiRthNvCCY9HYu7e8hONi0k1WdpskkvqsEL1m3\n\tRTt5d6iheHd5D6DWJpERZUaxOATFvpOEg6G/ZcdE=","Reply-To":"kieran.bingham@ideasonboard.com","To":"=?utf-8?q?Niklas_S=C3=B6derlund?= ,\n\tlibcamera-devel@lists.libcamera.org, Laurent Pinchart\n\t","References":"<20181229032855.26249-1-niklas.soderlund@ragnatech.se>\n\t<20181229032855.26249-3-niklas.soderlund@ragnatech.se>","From":"Kieran Bingham ","Openpgp":"preference=signencrypt","Autocrypt":"addr=kieran.bingham@ideasonboard.com; keydata=\n\tmQINBFYE/WYBEACs1PwjMD9rgCu1hlIiUA1AXR4rv2v+BCLUq//vrX5S5bjzxKAryRf0uHat\n\tV/zwz6hiDrZuHUACDB7X8OaQcwhLaVlq6byfoBr25+hbZG7G3+5EUl9cQ7dQEdvNj6V6y/SC\n\trRanWfelwQThCHckbobWiQJfK9n7rYNcPMq9B8e9F020LFH7Kj6YmO95ewJGgLm+idg1Kb3C\n\tpotzWkXc1xmPzcQ1fvQMOfMwdS+4SNw4rY9f07Xb2K99rjMwZVDgESKIzhsDB5GY465sCsiQ\n\tcSAZRxqE49RTBq2+EQsbrQpIc8XiffAB8qexh5/QPzCmR4kJgCGeHIXBtgRj+nIkCJPZvZtf\n\tKr2EAbc6tgg6DkAEHJb+1okosV09+0+TXywYvtEop/WUOWQ+zo+Y/OBd+8Ptgt1pDRyOBzL8\n\tRXa8ZqRf0Mwg75D+dKntZeJHzPRJyrlfQokngAAs4PaFt6UfS+ypMAF37T6CeDArQC41V3ko\n\tlPn1yMsVD0p+6i3DPvA/GPIksDC4owjnzVX9kM8Zc5Cx+XoAN0w5Eqo4t6qEVbuettxx55gq\n\t8K8FieAjgjMSxngo/HST8TpFeqI5nVeq0/lqtBRQKumuIqDg+Bkr4L1V/PSB6XgQcOdhtd36\n\tOe9X9dXB8YSNt7VjOcO7BTmFn/Z8r92mSAfHXpb07YJWJosQOQARAQABtDBLaWVyYW4gQmlu\n\tZ2hhbSA8a2llcmFuLmJpbmdoYW1AaWRlYXNvbmJvYXJkLmNvbT6JAkAEEwEKACoCGwMFCwkI\n\tBwIGFQgJCgsCBBYCAwECHgECF4ACGQEFAlnDk/gFCQeA/YsACgkQoR5GchCkYf3X5w/9EaZ7\n\tcnUcT6dxjxrcmmMnfFPoQA1iQXr/MXQJBjFWfxRUWYzjvUJb2D/FpA8FY7y+vksoJP7pWDL7\n\tQTbksdwzagUEk7CU45iLWL/CZ/knYhj1I/+5LSLFmvZ/5Gf5xn2ZCsmg7C0MdW/GbJ8IjWA8\n\t/LKJSEYH8tefoiG6+9xSNp1p0Gesu3vhje/GdGX4wDsfAxx1rIYDYVoX4bDM+uBUQh7sQox/\n\tR1bS0AaVJzPNcjeC14MS226mQRUaUPc9250aj44WmDfcg44/kMsoLFEmQo2II9aOlxUDJ+x1\n\txohGbh9mgBoVawMO3RMBihcEjo/8ytW6v7xSF+xP4Oc+HOn7qebAkxhSWcRxQVaQYw3S9iZz\n\t2iA09AXAkbvPKuMSXi4uau5daXStfBnmOfalG0j+9Y6hOFjz5j0XzaoF6Pln0jisDtWltYhP\n\tX9LjFVhhLkTzPZB/xOeWGmsG4gv2V2ExbU3uAmb7t1VSD9+IO3Km4FtnYOKBWlxwEd8qOFpS\n\tjEqMXURKOiJvnw3OXe9MqG19XdeENA1KyhK5rqjpwdvPGfSn2V+SlsdJA0DFsobUScD9qXQw\n\tOvhapHe3XboK2+Rd7L+g/9Ud7ZKLQHAsMBXOVJbufA1AT+IaOt0ugMcFkAR5UbBg5+dZUYJj\n\t1QbPQcGmM3wfvuaWV5+SlJ+WeKIb8ta5Ag0EVgT9ZgEQAM4o5G/kmruIQJ3K9SYzmPishRHV\n\tDcUcvoakyXSX2mIoccmo9BHtD9MxIt+QmxOpYFNFM7YofX4lG0ld8H7FqoNVLd/+a0yru5Cx\n\tadeZBe3qr1eLns10Q90LuMo7/6zJhCW2w+HE7xgmCHejAwuNe3+7yt4QmwlSGUqdxl8cgtS1\n\tPlEK93xXDsgsJj/bw1EfSVdAUqhx8UQ3aVFxNug5OpoX9FdWJLKROUrfNeBE16RLrNrq2ROc\n\tiSFETpVjyC/oZtzRFnwD9Or7EFMi76/xrWzk+/b15RJ9WrpXGMrttHUUcYZEOoiC2lEXMSAF\n\tSSSj4vHbKDJ0vKQdEFtdgB1roqzxdIOg4rlHz5qwOTynueiBpaZI3PHDudZSMR5Fk6QjFooE\n\tXTw3sSl/km/lvUFiv9CYyHOLdygWohvDuMkV/Jpdkfq8XwFSjOle+vT/4VqERnYFDIGBxaRx\n\tkoBLfNDiiuR3lD8tnJ4A1F88K6ojOUs+jndKsOaQpDZV6iNFv8IaNIklTPvPkZsmNDhJMRHH\n\tIu60S7BpzNeQeT4yyY4dX9lC2JL/LOEpw8DGf5BNOP1KgjCvyp1/KcFxDAo89IeqljaRsCdP\n\t7WCIECWYem6pLwaw6IAL7oX+tEqIMPph/G/jwZcdS6Hkyt/esHPuHNwX4guqTbVEuRqbDzDI\n\t2DJO5FbxABEBAAGJAiUEGAEKAA8CGwwFAlnDlGsFCQeA/gIACgkQoR5GchCkYf1yYRAAq+Yo\n\tnbf9DGdK1kTAm2RTFg+w9oOp2Xjqfhds2PAhFFvrHQg1XfQR/UF/SjeUmaOmLSczM0s6XMeO\n\tVcE77UFtJ/+hLo4PRFKm5X1Pcar6g5m4xGqa+Xfzi9tRkwC29KMCoQOag1BhHChgqYaUH3yo\n\tUzaPwT/fY75iVI+yD0ih/e6j8qYvP8pvGwMQfrmN9YB0zB39YzCSdaUaNrWGD3iCBxg6lwSO\n\tLKeRhxxfiXCIYEf3vwOsP3YMx2JkD5doseXmWBGW1U0T/oJF+DVfKB6mv5UfsTzpVhJRgee7\n\t4jkjqFq4qsUGxcvF2xtRkfHFpZDbRgRlVmiWkqDkT4qMA+4q1y/dWwshSKi/uwVZNycuLsz+\n\t+OD8xPNCsMTqeUkAKfbD8xW4LCay3r/dD2ckoxRxtMD9eOAyu5wYzo/ydIPTh1QEj9SYyvp8\n\tO0g6CpxEwyHUQtF5oh15O018z3ZLztFJKR3RD42VKVsrnNDKnoY0f4U0z7eJv2NeF8xHMuiU\n\tRCIzqxX1GVYaNkKTnb/Qja8hnYnkUzY1Lc+OtwiGmXTwYsPZjjAaDX35J/RSKAoy5wGo/YFA\n\tJxB1gWThL4kOTbsqqXj9GLcyOImkW0lJGGR3o/fV91Zh63S5TKnf2YGGGzxki+ADdxVQAm+Q\n\tsbsRB8KNNvVXBOVNwko86rQqF9drZuw=","Organization":"Ideas on Board","Message-ID":"<5b42d520-552b-0119-d8f4-587939bbe87d@ideasonboard.com>","Date":"Wed, 9 Jan 2019 17:07:29 +0000","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101\n\tThunderbird/60.2.1","MIME-Version":"1.0","In-Reply-To":"<20181229032855.26249-3-niklas.soderlund@ragnatech.se>","Content-Type":"text/plain; charset=utf-8","Content-Language":"en-GB","Content-Transfer-Encoding":"8bit","Subject":"Re: [libcamera-devel] [PATCH v2 02/12] libcamera: add dependency on\n\tlibudev","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.23","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","X-List-Received-Date":"Wed, 09 Jan 2019 17:07:33 -0000"}},{"id":273,"web_url":"https://patchwork.libcamera.org/comment/273/","msgid":"<1553348.vTvO4ahPAO@avalon>","date":"2019-01-10T13:39:34","subject":"Re: [libcamera-devel] [PATCH v2 02/12] libcamera: add dependency on\n\tlibudev","submitter":{"id":2,"url":"https://patchwork.libcamera.org/api/people/2/","name":"Laurent Pinchart","email":"laurent.pinchart@ideasonboard.com"},"content":"Hi Kieran,\n\nOn Wednesday, 9 January 2019 19:07:29 EET Kieran Bingham wrote:\n> On 29/12/2018 03:28, Niklas Söderlund wrote:\n> > The device enumeration will depend on libudev, add the dependency to the\n> > build system. This should be turned into a optional dependency once a\n> > device enumerator not using udev is supported.\n> > \n> > Signed-off-by: Niklas Söderlund \n> > Reviewed-by: Laurent Pinchart \n> > ---\n> > \n> > src/libcamera/meson.build | 5 ++++-\n> > 1 file changed, 4 insertions(+), 1 deletion(-)\n> > \n> > diff --git a/src/libcamera/meson.build b/src/libcamera/meson.build\n> > index 46591069aa5f8beb..52b556a8ed4050cb 100644\n> > --- a/src/libcamera/meson.build\n> > +++ b/src/libcamera/meson.build\n> > @@ -16,7 +16,10 @@ includes = [\n> > libcamera_internal_includes,\n> > ]\n> > \n> > +libudev = dependency('libudev')\n> > +\n> \n> The website currently states that we will not depend on anything except\n> the standard libraries [0].\n\nIt also states \"If other dependencies are deemed to be useful during \ndevelopment they shall be proposed and reviewed.\" :-) Note that we should make \nthis dependency optional.\n\n> I guess this will be updated when we move the website to build from the\n> repo, although I can't see anywhere within Documentation listing\n> anything similar. Is there a section missing in our source level docs?\n\nThe rest of the section has been added to the coding style document, but this \nparticular bullet point is missing. It doesn't belong to the coding style in \nmy opinion. Any idea on who to structure the documentation to give it a home ?\n\n> [0] http://www.libcamera.org/docs.html#technical-requirements\n> \n> > libcamera = shared_library('camera',\n> > libcamera_sources,\n> > install : true,\n> > - include_directories : includes)\n> > + include_directories : includes,\n> > + dependencies : libudev)","headers":{"Return-Path":"","Received":["from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 5BD7360B2C\n\tfor ;\n\tThu, 10 Jan 2019 14:38:25 +0100 (CET)","from avalon.localnet (dfj612ybrt5fhg77mgycy-3.rev.dnainternet.fi\n\t[IPv6:2001:14ba:21f5:5b00:2e86:4862:ef6a:2804])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id AC43C53E;\n\tThu, 10 Jan 2019 14:38:24 +0100 (CET)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1547127504;\n\tbh=0g7J3JPWEaVN7uvuJ6+GYj3TJ+rfaiOYFTy+Y2HvMvc=;\n\th=From:To:Cc:Subject:Date:In-Reply-To:References:From;\n\tb=QU3lGLGl+RLdFabUlaorxEnvQurKMVa3ckvu4j0bdzqzyjeS73wvDefBq/wfiyz4B\n\t3NpBFussMtGJrcgyVK/hIAqSpFjtsvgjCqsamyPu28m56WohHvfy9PdVjKldbFOhgF\n\thyZnhAq1MRTmOEAztJR/Bzw/qGndkFKfRyl7vdF0=","From":"Laurent Pinchart ","To":"kieran.bingham@ideasonboard.com","Cc":"Niklas =?iso-8859-1?q?S=F6derlund?= ,\n\tlibcamera-devel@lists.libcamera.org","Date":"Thu, 10 Jan 2019 15:39:34 +0200","Message-ID":"<1553348.vTvO4ahPAO@avalon>","Organization":"Ideas on Board Oy","In-Reply-To":"<5b42d520-552b-0119-d8f4-587939bbe87d@ideasonboard.com>","References":"<20181229032855.26249-1-niklas.soderlund@ragnatech.se>\n\t<20181229032855.26249-3-niklas.soderlund@ragnatech.se>\n\t<5b42d520-552b-0119-d8f4-587939bbe87d@ideasonboard.com>","MIME-Version":"1.0","Content-Transfer-Encoding":"quoted-printable","Content-Type":"text/plain; charset=\"iso-8859-1\"","Subject":"Re: [libcamera-devel] [PATCH v2 02/12] libcamera: add dependency on\n\tlibudev","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.23","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","X-List-Received-Date":"Thu, 10 Jan 2019 13:38:25 -0000"}},{"id":274,"web_url":"https://patchwork.libcamera.org/comment/274/","msgid":"","date":"2019-01-10T16:34:26","subject":"Re: [libcamera-devel] [PATCH v2 02/12] libcamera: add dependency on\n\tlibudev","submitter":{"id":4,"url":"https://patchwork.libcamera.org/api/people/4/","name":"Kieran Bingham","email":"kieran.bingham@ideasonboard.com"},"content":"Hi Laurent,\n\nOn 10/01/2019 13:39, Laurent Pinchart wrote:\n> Hi Kieran,\n> \n> On Wednesday, 9 January 2019 19:07:29 EET Kieran Bingham wrote:\n>> On 29/12/2018 03:28, Niklas Söderlund wrote:\n>>> The device enumeration will depend on libudev, add the dependency to the\n>>> build system. This should be turned into a optional dependency once a\n>>> device enumerator not using udev is supported.\n>>>\n>>> Signed-off-by: Niklas Söderlund \n>>> Reviewed-by: Laurent Pinchart \n>>> ---\n>>>\n>>> src/libcamera/meson.build | 5 ++++-\n>>> 1 file changed, 4 insertions(+), 1 deletion(-)\n>>>\n>>> diff --git a/src/libcamera/meson.build b/src/libcamera/meson.build\n>>> index 46591069aa5f8beb..52b556a8ed4050cb 100644\n>>> --- a/src/libcamera/meson.build\n>>> +++ b/src/libcamera/meson.build\n>>> @@ -16,7 +16,10 @@ includes = [\n>>> libcamera_internal_includes,\n>>> ]\n>>>\n>>> +libudev = dependency('libudev')\n>>> +\n>>\n>> The website currently states that we will not depend on anything except\n>> the standard libraries [0].\n> \n> It also states \"If other dependencies are deemed to be useful during \n> development they shall be proposed and reviewed.\" :-) Note that we should make \n> this dependency optional.\n\nAh yes - I'm not disagreeing on the inclusion of the dependency - just\nthat it leaves the documentation out-dated.\n\nAgreed on making it optional at some point but we can call that an\noptimisation at the moment :) (and I don't think it can be optional\nuntil there is a sysfs fallback option)\n\n>> I guess this will be updated when we move the website to build from the\n>> repo, although I can't see anywhere within Documentation listing\n>> anything similar. Is there a section missing in our source level docs?\n> \n> The rest of the section has been added to the coding style document, but this \n> particular bullet point is missing. It doesn't belong to the coding style in \n> my opinion. Any idea on who to structure the documentation to give it a home ?\n\nWe have a top level README.md which describes how to build the project.\nThat file itself could do with some attention at somepoint, but I think\nproject dependencies should go with build instructions. I think we\nshould keep a top-level README.md - but we might want to integrate that\nsomehow into the website documentation as well in a manner which doesn't\nduplicate the text if possible.\n\n\n\nCan we progress your patches that build the website from the source tree?\n\nI think at least bcd64a88e307...e164951f08 inclusive from your\ndocumentation branch could be posted and integrated already...\n\nWould you like to do so ? or shall I steal the patches ? :)\nPersonally I'd say just push them:\n Documentation: Add custom theme\n Documentation: Make the toctree more web-friendly\n Documentation: Link to the API documentation generated by Doxygen\n\nCan have an Acked-by: tag from me if you like.\n\nI realise there is a bit of a hack/workaround in that last patch - but\nit works and if we dislike it so much we can solve it later, along with\nlooking at the sphinx/breathe integration at some point.\n\nI'd rather see the patches progress upstream as they are as it will\nallow us to tie in the website documentation directly and improve\nincrementally.\n\n--\nKieran\n\n\n>> [0] http://www.libcamera.org/docs.html#technical-requirements\n>>\n>>> libcamera = shared_library('camera',\n>>> libcamera_sources,\n>>> install : true,\n>>> - include_directories : includes)\n>>> + include_directories : includes,\n>>> + dependencies : libudev)\n>","headers":{"Return-Path":"","Received":["from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id AD86360B2C\n\tfor ;\n\tThu, 10 Jan 2019 17:34:29 +0100 (CET)","from [192.168.0.21]\n\t(cpc89242-aztw30-2-0-cust488.18-1.cable.virginm.net [86.31.129.233])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 1892156D;\n\tThu, 10 Jan 2019 17:34:29 +0100 (CET)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1547138069;\n\tbh=lZiTebS9HtZHqI5qT7bpFxFTt8YDM+FZm6d/2KrWDyU=;\n\th=Reply-To:Subject:To:Cc:References:From:Date:In-Reply-To:From;\n\tb=MBOf/IzqXuHz2xG68u9hlA6vnyRb1goVMQqCBWx/r8RVDhH+DqernL9PhqCHl5TCd\n\t5R1HQAdtiaq89xFJ/k4D+31HJapRQzdEHjb6AQ0ezQXTvgqDr/lnO0G8I1Dj9NEDYu\n\t0xRX5GWe1GcS/6qi6F2uPpZ3rYZwThJ7iGL5uPvA=","Reply-To":"kieran.bingham@ideasonboard.com","To":"Laurent Pinchart ","Cc":"=?utf-8?q?Niklas_S=C3=B6derlund?= ,\n\tlibcamera-devel@lists.libcamera.org","References":"<20181229032855.26249-1-niklas.soderlund@ragnatech.se>\n\t<20181229032855.26249-3-niklas.soderlund@ragnatech.se>\n\t<5b42d520-552b-0119-d8f4-587939bbe87d@ideasonboard.com>\n\t<1553348.vTvO4ahPAO@avalon>","From":"Kieran Bingham ","Openpgp":"preference=signencrypt","Autocrypt":"addr=kieran.bingham@ideasonboard.com; keydata=\n\tmQINBFYE/WYBEACs1PwjMD9rgCu1hlIiUA1AXR4rv2v+BCLUq//vrX5S5bjzxKAryRf0uHat\n\tV/zwz6hiDrZuHUACDB7X8OaQcwhLaVlq6byfoBr25+hbZG7G3+5EUl9cQ7dQEdvNj6V6y/SC\n\trRanWfelwQThCHckbobWiQJfK9n7rYNcPMq9B8e9F020LFH7Kj6YmO95ewJGgLm+idg1Kb3C\n\tpotzWkXc1xmPzcQ1fvQMOfMwdS+4SNw4rY9f07Xb2K99rjMwZVDgESKIzhsDB5GY465sCsiQ\n\tcSAZRxqE49RTBq2+EQsbrQpIc8XiffAB8qexh5/QPzCmR4kJgCGeHIXBtgRj+nIkCJPZvZtf\n\tKr2EAbc6tgg6DkAEHJb+1okosV09+0+TXywYvtEop/WUOWQ+zo+Y/OBd+8Ptgt1pDRyOBzL8\n\tRXa8ZqRf0Mwg75D+dKntZeJHzPRJyrlfQokngAAs4PaFt6UfS+ypMAF37T6CeDArQC41V3ko\n\tlPn1yMsVD0p+6i3DPvA/GPIksDC4owjnzVX9kM8Zc5Cx+XoAN0w5Eqo4t6qEVbuettxx55gq\n\t8K8FieAjgjMSxngo/HST8TpFeqI5nVeq0/lqtBRQKumuIqDg+Bkr4L1V/PSB6XgQcOdhtd36\n\tOe9X9dXB8YSNt7VjOcO7BTmFn/Z8r92mSAfHXpb07YJWJosQOQARAQABtDBLaWVyYW4gQmlu\n\tZ2hhbSA8a2llcmFuLmJpbmdoYW1AaWRlYXNvbmJvYXJkLmNvbT6JAkAEEwEKACoCGwMFCwkI\n\tBwIGFQgJCgsCBBYCAwECHgECF4ACGQEFAlnDk/gFCQeA/YsACgkQoR5GchCkYf3X5w/9EaZ7\n\tcnUcT6dxjxrcmmMnfFPoQA1iQXr/MXQJBjFWfxRUWYzjvUJb2D/FpA8FY7y+vksoJP7pWDL7\n\tQTbksdwzagUEk7CU45iLWL/CZ/knYhj1I/+5LSLFmvZ/5Gf5xn2ZCsmg7C0MdW/GbJ8IjWA8\n\t/LKJSEYH8tefoiG6+9xSNp1p0Gesu3vhje/GdGX4wDsfAxx1rIYDYVoX4bDM+uBUQh7sQox/\n\tR1bS0AaVJzPNcjeC14MS226mQRUaUPc9250aj44WmDfcg44/kMsoLFEmQo2II9aOlxUDJ+x1\n\txohGbh9mgBoVawMO3RMBihcEjo/8ytW6v7xSF+xP4Oc+HOn7qebAkxhSWcRxQVaQYw3S9iZz\n\t2iA09AXAkbvPKuMSXi4uau5daXStfBnmOfalG0j+9Y6hOFjz5j0XzaoF6Pln0jisDtWltYhP\n\tX9LjFVhhLkTzPZB/xOeWGmsG4gv2V2ExbU3uAmb7t1VSD9+IO3Km4FtnYOKBWlxwEd8qOFpS\n\tjEqMXURKOiJvnw3OXe9MqG19XdeENA1KyhK5rqjpwdvPGfSn2V+SlsdJA0DFsobUScD9qXQw\n\tOvhapHe3XboK2+Rd7L+g/9Ud7ZKLQHAsMBXOVJbufA1AT+IaOt0ugMcFkAR5UbBg5+dZUYJj\n\t1QbPQcGmM3wfvuaWV5+SlJ+WeKIb8ta5Ag0EVgT9ZgEQAM4o5G/kmruIQJ3K9SYzmPishRHV\n\tDcUcvoakyXSX2mIoccmo9BHtD9MxIt+QmxOpYFNFM7YofX4lG0ld8H7FqoNVLd/+a0yru5Cx\n\tadeZBe3qr1eLns10Q90LuMo7/6zJhCW2w+HE7xgmCHejAwuNe3+7yt4QmwlSGUqdxl8cgtS1\n\tPlEK93xXDsgsJj/bw1EfSVdAUqhx8UQ3aVFxNug5OpoX9FdWJLKROUrfNeBE16RLrNrq2ROc\n\tiSFETpVjyC/oZtzRFnwD9Or7EFMi76/xrWzk+/b15RJ9WrpXGMrttHUUcYZEOoiC2lEXMSAF\n\tSSSj4vHbKDJ0vKQdEFtdgB1roqzxdIOg4rlHz5qwOTynueiBpaZI3PHDudZSMR5Fk6QjFooE\n\tXTw3sSl/km/lvUFiv9CYyHOLdygWohvDuMkV/Jpdkfq8XwFSjOle+vT/4VqERnYFDIGBxaRx\n\tkoBLfNDiiuR3lD8tnJ4A1F88K6ojOUs+jndKsOaQpDZV6iNFv8IaNIklTPvPkZsmNDhJMRHH\n\tIu60S7BpzNeQeT4yyY4dX9lC2JL/LOEpw8DGf5BNOP1KgjCvyp1/KcFxDAo89IeqljaRsCdP\n\t7WCIECWYem6pLwaw6IAL7oX+tEqIMPph/G/jwZcdS6Hkyt/esHPuHNwX4guqTbVEuRqbDzDI\n\t2DJO5FbxABEBAAGJAiUEGAEKAA8CGwwFAlnDlGsFCQeA/gIACgkQoR5GchCkYf1yYRAAq+Yo\n\tnbf9DGdK1kTAm2RTFg+w9oOp2Xjqfhds2PAhFFvrHQg1XfQR/UF/SjeUmaOmLSczM0s6XMeO\n\tVcE77UFtJ/+hLo4PRFKm5X1Pcar6g5m4xGqa+Xfzi9tRkwC29KMCoQOag1BhHChgqYaUH3yo\n\tUzaPwT/fY75iVI+yD0ih/e6j8qYvP8pvGwMQfrmN9YB0zB39YzCSdaUaNrWGD3iCBxg6lwSO\n\tLKeRhxxfiXCIYEf3vwOsP3YMx2JkD5doseXmWBGW1U0T/oJF+DVfKB6mv5UfsTzpVhJRgee7\n\t4jkjqFq4qsUGxcvF2xtRkfHFpZDbRgRlVmiWkqDkT4qMA+4q1y/dWwshSKi/uwVZNycuLsz+\n\t+OD8xPNCsMTqeUkAKfbD8xW4LCay3r/dD2ckoxRxtMD9eOAyu5wYzo/ydIPTh1QEj9SYyvp8\n\tO0g6CpxEwyHUQtF5oh15O018z3ZLztFJKR3RD42VKVsrnNDKnoY0f4U0z7eJv2NeF8xHMuiU\n\tRCIzqxX1GVYaNkKTnb/Qja8hnYnkUzY1Lc+OtwiGmXTwYsPZjjAaDX35J/RSKAoy5wGo/YFA\n\tJxB1gWThL4kOTbsqqXj9GLcyOImkW0lJGGR3o/fV91Zh63S5TKnf2YGGGzxki+ADdxVQAm+Q\n\tsbsRB8KNNvVXBOVNwko86rQqF9drZuw=","Organization":"Ideas on Board","Message-ID":"","Date":"Thu, 10 Jan 2019 16:34:26 +0000","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101\n\tThunderbird/60.2.1","MIME-Version":"1.0","In-Reply-To":"<1553348.vTvO4ahPAO@avalon>","Content-Type":"text/plain; charset=utf-8","Content-Language":"en-GB","Content-Transfer-Encoding":"8bit","Subject":"Re: [libcamera-devel] [PATCH v2 02/12] libcamera: add dependency on\n\tlibudev","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.23","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","X-List-Received-Date":"Thu, 10 Jan 2019 16:34:29 -0000"}},{"id":275,"web_url":"https://patchwork.libcamera.org/comment/275/","msgid":"<4205920.DLueB1ChmD@avalon>","date":"2019-01-10T23:40:13","subject":"Re: [libcamera-devel] [PATCH v2 02/12] libcamera: add dependency on\n\tlibudev","submitter":{"id":2,"url":"https://patchwork.libcamera.org/api/people/2/","name":"Laurent Pinchart","email":"laurent.pinchart@ideasonboard.com"},"content":"Hi Kieran,\n\nOn Thursday, 10 January 2019 18:34:26 EET Kieran Bingham wrote:\n> On 10/01/2019 13:39, Laurent Pinchart wrote:\n> > On Wednesday, 9 January 2019 19:07:29 EET Kieran Bingham wrote:\n> >> On 29/12/2018 03:28, Niklas Söderlund wrote:\n> >>> The device enumeration will depend on libudev, add the dependency to the\n> >>> build system. This should be turned into a optional dependency once a\n> >>> device enumerator not using udev is supported.\n> >>> \n> >>> Signed-off-by: Niklas Söderlund \n> >>> Reviewed-by: Laurent Pinchart \n> >>> ---\n> >>> \n> >>> src/libcamera/meson.build | 5 ++++-\n> >>> 1 file changed, 4 insertions(+), 1 deletion(-)\n> >>> \n> >>> diff --git a/src/libcamera/meson.build b/src/libcamera/meson.build\n> >>> index 46591069aa5f8beb..52b556a8ed4050cb 100644\n> >>> --- a/src/libcamera/meson.build\n> >>> +++ b/src/libcamera/meson.build\n> >>> @@ -16,7 +16,10 @@ includes = [\n> >>> libcamera_internal_includes,\n> >>> ]\n> >>> \n> >>> +libudev = dependency('libudev')\n> >>> +\n> >> \n> >> The website currently states that we will not depend on anything except\n> >> the standard libraries [0].\n> > \n> > It also states \"If other dependencies are deemed to be useful during\n> > development they shall be proposed and reviewed.\" :-) Note that we should\n> > make this dependency optional.\n> \n> Ah yes - I'm not disagreeing on the inclusion of the dependency - just\n> that it leaves the documentation out-dated.\n> \n> Agreed on making it optional at some point but we can call that an\n> optimisation at the moment :) (and I don't think it can be optional\n> until there is a sysfs fallback option)\n\nAgreed, that's on the todo list.\n\n> >> I guess this will be updated when we move the website to build from the\n> >> repo, although I can't see anywhere within Documentation listing\n> >> anything similar. Is there a section missing in our source level docs?\n> > \n> > The rest of the section has been added to the coding style document, but\n> > this particular bullet point is missing. It doesn't belong to the coding\n> > style in my opinion. Any idea on who to structure the documentation to\n> > give it a home ?\n> \n> We have a top level README.md which describes how to build the project.\n> That file itself could do with some attention at somepoint, but I think\n> project dependencies should go with build instructions.\n\nDon't they also belong to a design document that would explain the rationale ?\n\n> I think we should keep a top-level README.md - but we might want to\n> integrate that somehow into the website documentation as well in a manner\n> which doesn't duplicate the text if possible.\n\nI'm all for de-duplicating :-)\n\n> Can we progress your patches that build the website from the source tree?\n> \n> I think at least bcd64a88e307...e164951f08 inclusive from your\n> documentation branch could be posted and integrated already...\n> \n> Would you like to do so ? or shall I steal the patches ? :)\n\nDone :-)\n\n> Personally I'd say just push them:\n> Documentation: Add custom theme\n> Documentation: Make the toctree more web-friendly\n> Documentation: Link to the API documentation generated by Doxygen\n> \n> Can have an Acked-by: tag from me if you like.\n\nI've updated those patches to add support for a search box and to remove the \ntoc tree on the first page. Could you have a look ?\n\n> I realise there is a bit of a hack/workaround in that last patch - but\n> it works and if we dislike it so much we can solve it later, along with\n> looking at the sphinx/breathe integration at some point.\n\nI also believe we will at some point need to move documentation to a \nsubsection of the website, and generate the rest using a different tool. \nThere's no urgency, but if we want to add other services (such as a bugtracker \nfor instance), or extend the site with more dynamic content (such as blogging) \nwe won't be able to generate it all using sphinx.\n\n> I'd rather see the patches progress upstream as they are as it will\n> allow us to tie in the website documentation directly and improve\n> incrementally.\n\nI agree.\n\n> >> [0] http://www.libcamera.org/docs.html#technical-requirements\n> >> \n> >>> libcamera = shared_library('camera',\n> >>> libcamera_sources,\n> >>> install : true,\n> >>> - include_directories : includes)\n> >>> + include_directories : includes,\n> >>> + dependencies : libudev)","headers":{"Return-Path":"","Received":["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 6266F60B2C\n\tfor ;\n\tFri, 11 Jan 2019 00:39:03 +0100 (CET)","from avalon.localnet (dfj612ybrt5fhg77mgycy-3.rev.dnainternet.fi\n\t[IPv6:2001:14ba:21f5:5b00:2e86:4862:ef6a:2804])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id E28E156D;\n\tFri, 11 Jan 2019 00:39:02 +0100 (CET)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1547163543;\n\tbh=mdHF19sDC/Kru6I6ZDOKEFxNd0cO94kF+TP4eKT1x1Q=;\n\th=From:To:Cc:Subject:Date:In-Reply-To:References:From;\n\tb=CGPtxb6Fv9tfZ1lodUGCKyTjCwDaLaj2rckWXn5cOEcw5WpIH7xsHLfuBuhwJfOgw\n\tybEETNJU3Hf21gQGxqT74jehD/7aicd/siUnrnpDqj+JbAvbLhCMKGWW11Hy4CyVbp\n\tq2nkWFmDhojNCmZzqV7i2mNlQOnFTZ1mJCW5hX8g=","From":"Laurent Pinchart ","To":"kieran.bingham@ideasonboard.com","Cc":"Niklas =?iso-8859-1?q?S=F6derlund?= ,\n\tlibcamera-devel@lists.libcamera.org","Date":"Fri, 11 Jan 2019 01:40:13 +0200","Message-ID":"<4205920.DLueB1ChmD@avalon>","Organization":"Ideas on Board Oy","In-Reply-To":"","References":"<20181229032855.26249-1-niklas.soderlund@ragnatech.se>\n\t<1553348.vTvO4ahPAO@avalon>\n\t","MIME-Version":"1.0","Content-Transfer-Encoding":"quoted-printable","Content-Type":"text/plain; charset=\"iso-8859-1\"","Subject":"Re: [libcamera-devel] [PATCH v2 02/12] libcamera: add dependency on\n\tlibudev","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.23","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","X-List-Received-Date":"Thu, 10 Jan 2019 23:39:03 -0000"}}]