Re: [PATCH] isar-bootstrap: Fail on invalid apt source list

[Henning Schild <henning.schild@siemens.com>]

Am Tue, 16 Nov 2021 12:59:07 +0300
schrieb Anton Mikanovich <amikan@ilbers.de>:

> Returning empty list leads to ambiguous errors, so we should fail
> early if distro sources.list is empty or incorrect.
> 
> Fix issue #72

Not sure that will work. In case you want github to close on "master"
merge.

Fixes
Closes

should work
https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue

 
> Signed-off-by: Anton Mikanovich <amikan@ilbers.de>
> ---
>  meta/recipes-core/isar-bootstrap/isar-bootstrap.inc | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/meta/recipes-core/isar-bootstrap/isar-bootstrap.inc
> b/meta/recipes-core/isar-bootstrap/isar-bootstrap.inc index
> e9f92916..20a9afaf 100644 ---
> a/meta/recipes-core/isar-bootstrap/isar-bootstrap.inc +++
> b/meta/recipes-core/isar-bootstrap/isar-bootstrap.inc @@ -163,11
> +163,10 @@ def generate_distro_sources(d): yield parsed
>  
>  def get_distro_primary_source_entry(d):
> -    apt_sources_list = get_aptsources_list(d)

maybe that should be its own patch? Looks like a drive-by fix.

>      for source in generate_distro_sources(d):
>          if source[0] == "deb":
>              return source[2:]
> -    return ["", "", ""]
> +    bb.fatal('Invalid apt sources list')

I did a bit of digging in history. Seems like bbfatal is the right
thing, the empty strings seem to not have had any deeper meaning.

Henning

>  def get_distro_have_https_source(d):
>      return any(source[2].startswith("https://") for source in
> generate_distro_sources(d))