Why the [1:]? The flatpak list does not start with a header, the very first line already contains a package - at least in my case..

It does in my case, but if I recall correctly, the output is different when called from sos, and there's no header there. I've fixed this in the next version.

Why not simple yield (pkg[0], pkg[1], pkg[2]) ? (maybe worth checking len(pkg)>2, just in case of truncated output (or that is over-paranoic..?))

Let me try this and see if it covers the empty version case.

Checking the length of pkg and then yielding either pkg[1] or None seems better than just yielding pkg[1] regardless of it being there or not:

        if len(pkg) > 2:
            yield (pkg[0], pkg[1], pkg[2])
        else:
            yield (pkg[0], None, pkg[2])

Unless yielding pkg[1] when doesn't exist changes it to None on the yield, which then may simply the code a lot. But I saw in the other sources in package_managers that we explicitly set to None.

Ah right, I am not sure now if the split returns None or empty string. But can we pass None as version? Does not we expect a string value later on?

Later on I'm not sure, but if that's the case, we can fail back to "" instead of None. Let me see if I can test with packages with empty value and see what happens with None.

OK, split returns empty string and not None (which is in fact obvious, it returns list of strings):

>>> "name\tvers\tbranch".split("\t")
['name', 'vers', 'branch']
>>> "name\t\tbranch".split("\t")
['name', '', 'branch']
>>> 

And as far as I checked, we assume strings there.

And I just checked what we get with yield (pkg[0], pkg[1], pkg[2]) without checking pkg lenght, from _generate_pkg_list, and it is an empty string. So just that line should be enough.