Ah, thanks for reverting this. I am not familiar with snapd and Linux tooling. As it's in draft mode, I won't otherwise comment on it right now.

Yeah, let's wait to see if this gets approved before making changes to the docs. Thanks!

SG

@sfshaza2 this can merge now. Thanks!

We should probably temporarily revert this change; I didn't think about it at the time, but many users are now using desktop in the beta and stable channels, where the other libraries are still necessary. I've seen a few issues come in where prior followed these instructions, but then we're missing libraries, because they were in stable.

This should be changed only once the Flutter changes that go with it reach stable.

We should probably temporarily revert this change

The only library that legitimately should be on this list is liblzma (indefinitely - not temporarily). This is because some distros (such as Fedora 33+ & Ubuntu 20.04+) have since removed liblzma from their default installed packages.

I've reverted this here and here.

The others (libblkid & libgcrypt) should never have been added upstream as those only ever showed up in snap usage.

Sorry for all the back and forth on this!

We should probably temporarily revert this change

The only library that legitimately should be on this list is liblzma

Well, now that 2.2 has shipped my comment is moot.

The others (libblkid & libgcrypt) should never have been added upstream as those only ever showed up in snap usage.

This statement is not correct, which is why the instructions should not actually have been changed while 2.0 was the stable release. The motivation to add them was for snap usage, but they were made requirements in every single developer's app templates regardless of whether they were using the snap or not. Which means that not having them installed prevented compilation.

Yes, they were made requirements. This is what I was referring to when I said they “should never have been added”. As in, they should never have been added to the CMake, nor the docs.