[{"id":19789,"web_url":"https://patchwork.libcamera.org/comment/19789/","msgid":"<6331b56e-cc67-9eaf-ffa1-234e712a0756@ideasonboard.com>","date":"2021-09-22T12:21:57","subject":"Re: [libcamera-devel] [PATCH] libcamera: Standardize URLs to git\n\trepositories","submitter":{"id":86,"url":"https://patchwork.libcamera.org/api/people/86/","name":"Umang Jain","email":"umang.jain@ideasonboard.com"},"content":"Hi Laurent,\n\nThank you for the patch\n\nOn 9/22/21 5:38 PM, Laurent Pinchart wrote:\n> When the libcamera project was started, we had no public git tree hosted\n> on git.libcamera.org. The problem has been addressed a while ago, and\n> the git.linuxtv.org repository is now a mirror of the main git tree. The\n\n\ns/ git.linuxtv.org / git.linuxtv.org/libcamera.git/  maybe? or simply,\n\ns/ git.linuxtv.org / git.linuxtv.org libcamera /\n\n> mirror is useful to benefit from the linuxtv.org automated compile\n> tests, but it can also confuse users who don't know where the official\n> version is. To try and clarify this, use the git.libcamera.org URL\n> consistently through the project.\n>\n> This doesn't the validatity of the linuxtv.org repository which will\n\n\ns/doesn't /doesn't void / maybe?\n\n> continue to mirror the libcamera.org repository.\n>\n> Signed-off-by: Laurent Pinchart\n> ---\n> README.rst | 2 +-\n> package/gentoo/media-libs/libcamera/libcamera-9999.ebuild | 2 +-\n> 2 files changed, 2 insertions(+), 2 deletions(-)\n\n\nShould references to https://git.linuxtv.org/libcamera* inside \nDocumentation/ be replaced as well?\n\nReviewed-by: Umang Jain \n\n> diff --git a/README.rst b/README.rst\n> index 6eb25c708180..a0cc45978b0b 100644\n> --- a/README.rst\n> +++ b/README.rst\n> @@ -32,7 +32,7 @@ To fetch the sources, build and install:\n> \n> ::\n> \n> - git clonegit://linuxtv.org/libcamera.git\n> + git clonehttps://git.libcamera.org/libcamera/libcamera.git\n> cd libcamera\n> meson build\n> ninja -C build install\n> diff --git a/package/gentoo/media-libs/libcamera/libcamera-9999.ebuild b/package/gentoo/media-libs/libcamera/libcamera-9999.ebuild\n> index ca216c0ae909..65619e07d4ae 100644\n> --- a/package/gentoo/media-libs/libcamera/libcamera-9999.ebuild\n> +++ b/package/gentoo/media-libs/libcamera/libcamera-9999.ebuild\n> @@ -8,7 +8,7 @@ inherit git-r3 meson python-any-r1\n> \n> DESCRIPTION=\"Camera support library for Linux\"\n> HOMEPAGE=\"http://libcamera.org\"\n> -EGIT_REPO_URI=\"git://linuxtv.org/libcamera.git\"\n> +EGIT_REPO_URI=\"https://git.libcamera.org/libcamera/libcamera.git\"\n> EGIT_BRANCH=\"master\"\n> \n> LICENSE=\"LGPL-2.1+\"\n>\n> base-commit: a3b1539869ae53a6446a1c31bdade6560f91a46b","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 4689DBDC71\n\tfor ;\n\tWed, 22 Sep 2021 12:22:03 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id B43B46918C;\n\tWed, 22 Sep 2021 14:22: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 2524B6917F\n\tfor ;\n\tWed, 22 Sep 2021 14:22:02 +0200 (CEST)","from [192.168.1.104] (unknown [103.251.226.124])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 3F5C2F1;\n\tWed, 22 Sep 2021 14:22:00 +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=\"RP+foKqI\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1632313321;\n\tbh=j/cPplsK90fIZNHN2cuRy1FfGn+fJx2xUWB5vhp8HhY=;\n\th=From:Subject:To:References:Date:In-Reply-To:From;\n\tb=RP+foKqIUbhs1CjopeXOmHwrnK2m/lvtV3isU6fCvkQl1wRVaE/DAtY9tpbOqsqlI\n\tXh5fQ1vmwa+mXGd4dXdpBj3uNK5SsWZLhs/1hfyKW0j7puZCR7zPVJDq9NVTLj/eA8\n\tvqkv/kdyFtH30EnRvO19hwXy0FoNthzTh7H1TXgs=","From":"Umang Jain ","To":"Laurent Pinchart ,\n\tlibcamera-devel@lists.libcamera.org","References":"<20210922120856.30735-1-laurent.pinchart@ideasonboard.com>","Message-ID":"<6331b56e-cc67-9eaf-ffa1-234e712a0756@ideasonboard.com>","Date":"Wed, 22 Sep 2021 17:51:57 +0530","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101\n\tThunderbird/78.10.2","MIME-Version":"1.0","In-Reply-To":"<20210922120856.30735-1-laurent.pinchart@ideasonboard.com>","Content-Type":"text/plain; charset=utf-8; format=flowed","Content-Transfer-Encoding":"8bit","Content-Language":"en-US","Subject":"Re: [libcamera-devel] [PATCH] libcamera: Standardize URLs to git\n\trepositories","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","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}},{"id":19799,"web_url":"https://patchwork.libcamera.org/comment/19799/","msgid":"<20210923035743.GL4382@pyrite.rasen.tech>","date":"2021-09-23T03:57:44","subject":"Re: [libcamera-devel] [PATCH] libcamera: Standardize URLs to git\n\trepositories","submitter":{"id":17,"url":"https://patchwork.libcamera.org/api/people/17/","name":"Paul Elder","email":"paul.elder@ideasonboard.com"},"content":"Hi Laurent,\n\nOn Wed, Sep 22, 2021 at 03:08:56PM +0300, Laurent Pinchart wrote:\n> When the libcamera project was started, we had no public git tree hosted\n> on git.libcamera.org. The problem has been addressed a while ago, and\n> the git.linuxtv.org repository is now a mirror of the main git tree. The\n> mirror is useful to benefit from the linuxtv.org automated compile\n> tests, but it can also confuse users who don't know where the official\n> version is. To try and clarify this, use the git.libcamera.org URL\n> consistently through the project.\n> \n> This doesn't the validatity of the linuxtv.org repository which will\n\ndoesn't what? :p\n\n> continue to mirror the libcamera.org repository.\n> \n> Signed-off-by: Laurent Pinchart \n\nReviewed-by: Paul Elder \n\n> ---\n> README.rst | 2 +-\n> package/gentoo/media-libs/libcamera/libcamera-9999.ebuild | 2 +-\n> 2 files changed, 2 insertions(+), 2 deletions(-)\n> \n> diff --git a/README.rst b/README.rst\n> index 6eb25c708180..a0cc45978b0b 100644\n> --- a/README.rst\n> +++ b/README.rst\n> @@ -32,7 +32,7 @@ To fetch the sources, build and install:\n> \n> ::\n> \n> - git clone git://linuxtv.org/libcamera.git\n> + git clone https://git.libcamera.org/libcamera/libcamera.git\n> cd libcamera\n> meson build\n> ninja -C build install\n> diff --git a/package/gentoo/media-libs/libcamera/libcamera-9999.ebuild b/package/gentoo/media-libs/libcamera/libcamera-9999.ebuild\n> index ca216c0ae909..65619e07d4ae 100644\n> --- a/package/gentoo/media-libs/libcamera/libcamera-9999.ebuild\n> +++ b/package/gentoo/media-libs/libcamera/libcamera-9999.ebuild\n> @@ -8,7 +8,7 @@ inherit git-r3 meson python-any-r1\n> \n> DESCRIPTION=\"Camera support library for Linux\"\n> HOMEPAGE=\"http://libcamera.org\"\n> -EGIT_REPO_URI=\"git://linuxtv.org/libcamera.git\"\n> +EGIT_REPO_URI=\"https://git.libcamera.org/libcamera/libcamera.git\"\n> EGIT_BRANCH=\"master\"\n> \n> LICENSE=\"LGPL-2.1+\"\n> \n> base-commit: a3b1539869ae53a6446a1c31bdade6560f91a46b","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 E19A5BDC71\n\tfor ;\n\tThu, 23 Sep 2021 03:57:54 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 102F76918C;\n\tThu, 23 Sep 2021 05:57:54 +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 77FD460245\n\tfor ;\n\tThu, 23 Sep 2021 05:57:51 +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 D821845E;\n\tThu, 23 Sep 2021 05:57:49 +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=\"F0+N6+yb\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1632369471;\n\tbh=ZyzPRYhsTSvdDtRzFkCHOs+7vezj0yUKd9xjbSErl+A=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=F0+N6+ybpSmDtVahTZuA4NAkviExS1boHSldCSTL3L3RiW4CeZSGvFPHiPmawgii4\n\t2XUrlUg9MFG04gQ7MxupPDqbZ5Wcb5VGBGM6WC+gVTV6tUGaLrT3bom+VwayRpS/Vu\n\tUhWXw+yU5RzU4l6SxoPyhs6AqbCLepTGtDDhkyuU=","Date":"Thu, 23 Sep 2021 12:57:44 +0900","From":"paul.elder@ideasonboard.com","To":"Laurent Pinchart ","Message-ID":"<20210923035743.GL4382@pyrite.rasen.tech>","References":"<20210922120856.30735-1-laurent.pinchart@ideasonboard.com>","MIME-Version":"1.0","Content-Type":"text/plain; charset=us-ascii","Content-Disposition":"inline","In-Reply-To":"<20210922120856.30735-1-laurent.pinchart@ideasonboard.com>","Subject":"Re: [libcamera-devel] [PATCH] libcamera: Standardize URLs to git\n\trepositories","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","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}},{"id":19809,"web_url":"https://patchwork.libcamera.org/comment/19809/","msgid":"","date":"2021-09-23T10:24:26","subject":"Re: [libcamera-devel] [PATCH] libcamera: Standardize URLs to git\n\trepositories","submitter":{"id":2,"url":"https://patchwork.libcamera.org/api/people/2/","name":"Laurent Pinchart","email":"laurent.pinchart@ideasonboard.com"},"content":"Hi Umang,\n\nOn Wed, Sep 22, 2021 at 05:51:57PM +0530, Umang Jain wrote:\n> On 9/22/21 5:38 PM, Laurent Pinchart wrote:\n> > When the libcamera project was started, we had no public git tree hosted\n> > on git.libcamera.org. The problem has been addressed a while ago, and\n> > the git.linuxtv.org repository is now a mirror of the main git tree. The\n> \n> s/ git.linuxtv.org / git.linuxtv.org/libcamera.git/  maybe? or simply,\n> \n> s/ git.linuxtv.org / git.linuxtv.org libcamera /\n\nI'll go for the latter.\n\n> > mirror is useful to benefit from the linuxtv.org automated compile\n> > tests, but it can also confuse users who don't know where the official\n> > version is. To try and clarify this, use the git.libcamera.org URL\n> > consistently through the project.\n> >\n> > This doesn't the validatity of the linuxtv.org repository which will\n> \n> s/doesn't /doesn't void / maybe?\n\nOops :-) Yes\n\n> > continue to mirror the libcamera.org repository.\n> >\n> > Signed-off-by: Laurent Pinchart\n> > ---\n> > README.rst | 2 +-\n> > package/gentoo/media-libs/libcamera/libcamera-9999.ebuild | 2 +-\n> > 2 files changed, 2 insertions(+), 2 deletions(-)\n> \n> Should references to https://git.linuxtv.org/libcamera* inside \n> Documentation/ be replaced as well?\n\nHow did I miss that ? I'll fix it in a v2.\n\n> Reviewed-by: Umang Jain \n> \n> > diff --git a/README.rst b/README.rst\n> > index 6eb25c708180..a0cc45978b0b 100644\n> > --- a/README.rst\n> > +++ b/README.rst\n> > @@ -32,7 +32,7 @@ To fetch the sources, build and install:\n> > \n> > ::\n> > \n> > - git clonegit://linuxtv.org/libcamera.git\n> > + git clonehttps://git.libcamera.org/libcamera/libcamera.git\n> > cd libcamera\n> > meson build\n> > ninja -C build install\n> > diff --git a/package/gentoo/media-libs/libcamera/libcamera-9999.ebuild b/package/gentoo/media-libs/libcamera/libcamera-9999.ebuild\n> > index ca216c0ae909..65619e07d4ae 100644\n> > --- a/package/gentoo/media-libs/libcamera/libcamera-9999.ebuild\n> > +++ b/package/gentoo/media-libs/libcamera/libcamera-9999.ebuild\n> > @@ -8,7 +8,7 @@ inherit git-r3 meson python-any-r1\n> > \n> > DESCRIPTION=\"Camera support library for Linux\"\n> > HOMEPAGE=\"http://libcamera.org\"\n> > -EGIT_REPO_URI=\"git://linuxtv.org/libcamera.git\"\n> > +EGIT_REPO_URI=\"https://git.libcamera.org/libcamera/libcamera.git\"\n> > EGIT_BRANCH=\"master\"\n> > \n> > LICENSE=\"LGPL-2.1+\"\n> >\n> > base-commit: a3b1539869ae53a6446a1c31bdade6560f91a46b","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 6D19CBDC71\n\tfor ;\n\tThu, 23 Sep 2021 10:24:31 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id EC60F6918A;\n\tThu, 23 Sep 2021 12:24:30 +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 CAC6169189\n\tfor ;\n\tThu, 23 Sep 2021 12:24:28 +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 37D2F45E;\n\tThu, 23 Sep 2021 12:24:28 +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=\"ZrrsRWT7\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1632392668;\n\tbh=aw3/zPL7ZcnfxIUl2yAaG4IvE9xz+KbKK+iBEZgOrPk=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=ZrrsRWT7IXnBb67rhAbpxVKV2QPoWKx4fKSrGeJiUtcmrY0I0tsH8yzdn2pRhuJ+f\n\tqyhI9dXOBhA6Di4wHjOm63rKYmaSXRgHZplCaAl8YvPoiGwguHZL2gdN0CAqxFSfrq\n\txEwoXI07niPci5QD3gmy9fOdznWora8n+pe0Bdbw=","Date":"Thu, 23 Sep 2021 13:24:26 +0300","From":"Laurent Pinchart ","To":"Umang Jain ","Message-ID":"","References":"<20210922120856.30735-1-laurent.pinchart@ideasonboard.com>\n\t<6331b56e-cc67-9eaf-ffa1-234e712a0756@ideasonboard.com>","MIME-Version":"1.0","Content-Type":"text/plain; charset=utf-8","Content-Disposition":"inline","Content-Transfer-Encoding":"8bit","In-Reply-To":"<6331b56e-cc67-9eaf-ffa1-234e712a0756@ideasonboard.com>","Subject":"Re: [libcamera-devel] [PATCH] libcamera: Standardize URLs to git\n\trepositories","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","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}}]