lsblk command does not return model name as output in some specific cases, specially when you have a micro SD in your computer (perhaps it should happen with a normal SD card as well, but I haven't tested it).
So I included a verification in SfdiskBackend::scanDevice to check for kname in lsblk and use it as the name in these cases.
| Lint Skipped |
| Unit Tests Skipped |
It fixes the problem for me. So in principle it can go in. I would probably put on command line parameter per line, easier to read.
Although, now I was reading your comment, I thought that we can run just one command
lsblk --nodeps --noheadings --output model,kname --json deviceNode
and do JSON parsing. I'm not sure if it's worth it. In vast majority of cases device comes with a name, so json would slow it down a tiny bit.
But in the cases where device has no name, we would save one external command call which is a big win. What do you think? Is it worth it or not?
Actually now I'm thinking, maybe we should always read KNAME for devices and partitions and store it. We need it in quite a few places in the code and I think right now we just follow symlinks.
There are a few places where we call
QFileInfo kernelPath(deviceNode); kernelPath.canonicalFilePath()
In any case this patch can go in and we can improve on top of this.