isar-bootstrap.inc::get_distro_primary_source_entry() should not return ["", "", ""] by default about isar HOT 7 CLOSED

Hello Henning,

we'd like to understand the context, can you provide the steps to reproduce?

If you also have a suggestion in form of a patch, feel free to share.

from isar.

I guess the way to repro would be to use an "invalid" i.e. empty sources.list

If that happens we indeed could probably detect that early instead of returing something that will just break in a weird way further down the line.

Maybe return None, None, None or simply raise/bbfatal

from isar.

@henning-schild I would highly appreciate that. In my case, the list was not empty. However, I made a typo in the distro name. Thus, nothing was read from the list.

It took me a long time to figure out the root cause. I think throwing an error in this function at isar layer would massively ease debugging.

from isar.

In this case i am afraid the example file would be indeed needed. Because you maybe did not even run into the ["", "", ""] but instead the parser returned your invalid distro. Which i think we can not do anything about, because the parser should not know all valid values ... would be too hard to maintain.

from isar.

Just have proposed the patch at [PATCH] isar-bootstrap: Fail on invalid apt source list.
The only inconvenience is that the error during the parsing will be followed by do_bootstrap contents which can be confusing.

from isar.

@amikan Thanks. I will try it and let you know.

from isar.

@amikan It works perfectly. Thanks.

I had to slightly rework you patch as we actually work with this version:
76eac23

from isar.