All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Mathieu Dubois-Briand" <mathieu.dubois-briand@bootlin.com>
To: <pascal.eberhard@gmail.com>, <bitbake-devel@lists.openembedded.org>
Cc: "Pascal Eberhard" <pascal.eberhard@se.com>
Subject: Re: [bitbake-devel] [PATCH 1/4] fetch2: add curl method to fetch web content
Date: Mon, 09 Mar 2026 08:40:15 +0100	[thread overview]
Message-ID: <DGY2XTFF7EIK.365FMBM43AWTO@bootlin.com> (raw)
In-Reply-To: <20260305-add_alt_fetch_method_curl-v1-1-0d0220e5fa59@se.com>

On Thu Mar 5, 2026 at 4:32 PM CET, Pascal Eberhard via B4 Relay via lists.openembedded.org wrote:
> From: Pascal Eberhard <pascal.eberhard@se.com>
>
> curl fetch method is an alternative fetch method for web downloads. It
> is based on curl cmdline tool and provides the same http, https, ftp
> and ftps protocols as wget. It supports some new features as well such
> as hostname resolution by the proxy when using SOCKS5 proxy.
>
> Signed-off-by: Pascal Eberhard <pascal.eberhard@se.com>
> ---

Hi Pascal,

Thanks for your patches.

> +
> +    def _runcurl(self, ud: FetchData, d: DataSmart, command: str, quiet: bool, workdir: str | None = None):

Writing type hint unions with a pipe was added recently to Python, 3.10
if I'm correct [1]. This is above the minimal 3.9 [2] version mandated
for bitbake.

And this is indeed failing on some supported distributions, at least on
Debian 11 and Rocky 9:

Traceback (most recent call last):
  File "/srv/pokybuild/yocto-worker/genericarm64/build/repos/bitbake/bin/bitbake-setup", line 27, in <module>
    import bb.msg
  File "/srv/pokybuild/yocto-worker/genericarm64/build/repos/bitbake/lib/bb/__init__.py", line 155, in <module>
    from bb import fetch2 as fetch
  File "/srv/pokybuild/yocto-worker/genericarm64/build/repos/bitbake/lib/bb/fetch2/__init__.py", line 2107, in <module>
    from . import curl
  File "/srv/pokybuild/yocto-worker/genericarm64/build/repos/bitbake/lib/bb/fetch2/curl.py", line 53, in <module>
    class Curl(Wget):
  File "/srv/pokybuild/yocto-worker/genericarm64/build/repos/bitbake/lib/bb/fetch2/curl.py", line 104, in Curl
    def _runcurl(self, ud: FetchData, d: DataSmart, command: str, quiet: bool, workdir: str | None = None):
TypeError: unsupported operand type(s) for |: 'type' and 'NoneType'

https://autobuilder.yoctoproject.org/valkyrie/#/builders/60/builds/3308
https://autobuilder.yoctoproject.org/valkyrie/#/builders/93/builds/3302

[1]: https://peps.python.org/pep-0604/
[2]: https://git.openembedded.org/bitbake/tree/lib/bb/__init__.py

Thanks,
Mathieu

-- 
Mathieu Dubois-Briand, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com



  reply	other threads:[~2026-03-09  7:40 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-03-05 15:32 [PATCH 0/4] fetch2: add alternative fetch method based on curl Pascal Eberhard
2026-03-05 15:32 ` Pascal Eberhard via B4 Relay
2026-03-05 15:32 ` [PATCH 1/4] fetch2: add curl method to fetch web content Pascal Eberhard
2026-03-05 15:32   ` Pascal Eberhard via B4 Relay
2026-03-09  7:40   ` Mathieu Dubois-Briand [this message]
2026-03-05 15:32 ` [PATCH 2/4] fetch2: make curl method activable with BB_FETCH_METHOD_HTTP Pascal Eberhard
2026-03-05 15:32   ` Pascal Eberhard via B4 Relay
2026-03-05 15:32 ` [PATCH 3/4] lib/tests/fetch2: add tests for curl method Pascal Eberhard
2026-03-05 15:32   ` Pascal Eberhard via B4 Relay
2026-03-05 15:32 ` [PATCH 4/4] doc: bitbake-user-manual-ref-variables: describe BB_FETCH_METHOD_HTTP variable Pascal Eberhard
2026-03-05 15:32   ` Pascal Eberhard via B4 Relay
2026-03-05 18:09 ` [bitbake-devel] [PATCH 0/4] fetch2: add alternative fetch method based on curl Paul Barker
2026-03-06  8:40   ` Pascal EBERHARD

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=DGY2XTFF7EIK.365FMBM43AWTO@bootlin.com \
    --to=mathieu.dubois-briand@bootlin.com \
    --cc=bitbake-devel@lists.openembedded.org \
    --cc=pascal.eberhard@gmail.com \
    --cc=pascal.eberhard@se.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.