From: Julien Olivain via buildroot <buildroot@buildroot.org>
To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Kieran Bingham <kieran.bingham@ideasonboard.com>,
buildroot@buildroot.org,
Marcus Folkesson <marcus.folkesson@gmail.com>,
libcamera-devel@lists.libcamera.org
Subject: Re: [Buildroot] [PATCH 1/1] package/libcamera: update source URL
Date: Fri, 10 Jul 2026 18:50:55 +0200 [thread overview]
Message-ID: <ab26ffc6635e829f0026b2fe4688e325@free.fr> (raw)
In-Reply-To: <20260709132313.GA1007225@killaraus.ideasonboard.com>
Hi Laurent, Kieran,
On 09/07/2026 15:23, Laurent Pinchart wrote:
> On Thu, Jul 09, 2026 at 11:41:03AM +0100, Kieran Bingham wrote:
>> Quoting Laurent Pinchart (2026-07-09 11:26:24)
>> > On Thu, Jul 09, 2026 at 11:19:58AM +0100, Kieran Bingham wrote:
>> > > Quoting Laurent Pinchart (2026-07-08 23:35:32)
>> > > > libcamera has moved for a while now to gitlab.freedesktop.org. The
>> > > > mirror on git.linuxtv.org is still active, but it isn't the canonical
>> > > > repository and has less bandwidth than freedesktop.org. Switch the
>> > > > libcamera source URL to gitlab.freedesktop.org.
>> > > >
>> > > > Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
>> > > > ---
>> > > > package/libcamera/libcamera.mk | 2 +-
>> > > > 1 file changed, 1 insertion(+), 1 deletion(-)
>> > > >
>> > > > diff --git a/package/libcamera/libcamera.mk b/package/libcamera/libcamera.mk
>> > > > index 945207888bab..ba7510daf8e1 100644
>> > > > --- a/package/libcamera/libcamera.mk
>> > > > +++ b/package/libcamera/libcamera.mk
>> > > > @@ -4,7 +4,7 @@
>> > > > #
>> > > > ################################################################################
>> > > >
>> > > > -LIBCAMERA_SITE = https://git.linuxtv.org/libcamera.git
>> > > > +LIBCAMERA_SITE = https://gitlab.freedesktop.org/camera/libcamera.git/
>> > >
>> > > Why the trailing slash?
>> > >
>> > > Though I've just checked and :
>> > > git clone https://gitlab.freedesktop.org/camera/libcamera.git/
>> > >
>> > > works as expected,
>> > >
>> > > Viewing the webpage, drops the .git but cloneing without .git reports
>> > > the following:
>> > >
>> > > kbingham@Monstersaurus:/tmp$ git clone https://gitlab.freedesktop.org/camera/libcamera
>> > > Cloning into 'libcamera'...
>> > > warning: redirecting to https://gitlab.freedesktop.org/camera/libcamera.git/
>> > >
>> > > Which has the trailing slash so :
>> >
>> > That's exactly why I added a trailing slash :-)
>>
>> The web interface does however report this as the URL:
>>
>> https://gitlab.freedesktop.org/camera/libcamera.git
>>
>> without the slash, and cloning that doesn't redirect to one /with/ a
>> slash.
>>
>> Personally I wouldn't add it - but it's harmless either way.
>
> I'm fine dropping it. Can this be done when applying the patch ? I can
> send a v2 otherwise.
I removed the trailing slash and applied to master, thanks.
We usually don't put those in Buildroot. Also, it will remain
consistent with other packages using gitlab.freedesktop.org
(piglit, waffle).
>> > > Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
>> > >
>> > > > LIBCAMERA_VERSION = v0.7.1
>> > > > LIBCAMERA_SITE_METHOD = git
>> > > > LIBCAMERA_DEPENDENCIES = \
>> > > >
>> > > > base-commit: 6854ee9aa3826d5036a7d3cc19056d43c0452320
>
> --
> Regards,
>
> Laurent Pinchart
Best regards,
Julien.
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
prev parent reply other threads:[~2026-07-10 16:51 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-08 22:35 [Buildroot] [PATCH 1/1] package/libcamera: update source URL Laurent Pinchart
2026-07-09 10:19 ` Kieran Bingham
2026-07-09 10:26 ` Laurent Pinchart
2026-07-09 10:41 ` Kieran Bingham
2026-07-09 13:23 ` Laurent Pinchart
2026-07-10 16:50 ` Julien Olivain via buildroot [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ab26ffc6635e829f0026b2fe4688e325@free.fr \
--to=buildroot@buildroot.org \
--cc=ju.o@free.fr \
--cc=kieran.bingham@ideasonboard.com \
--cc=laurent.pinchart@ideasonboard.com \
--cc=libcamera-devel@lists.libcamera.org \
--cc=marcus.folkesson@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox