Hi, while playing with portage & binpkgs, I noticed gparted has unnecessary deps, the bunch of file utils is only needed at runtime. See patch.
Created attachment 119527 [details, diff] Patch for deps
I fail to see what you are trying to achieve in your patch I'm willing to close as invalid though since fs specific utils are required both at compile time and at runtime because gparted (afaik) dynamically checks for the presence of these utilities when it's launched. If I'm wrong please explain better what you want.
The patch removes the utilities from the compile time deps, "because gparted (afaik) dynamically checks for the presence of these utilities when it's launched" and they don't need to be around at compile time. To quote the README: GParted stands for Gnome Partition Editor. It uses libparted to detect and manipulate devices and partitiontables while several (optional) filesystemtools provide support for filesystems not included in libparted. These optional packages will be detected at runtime and don't require a rebuild of GParted. So, on a host compiling packages on behalf of other hosts, these deps get installed needlessly.
If those tools are not needed at build time at all, there's no need to have useflags for them and it'd be better to consider removing them altogether and use elog to print out the names of those tools for specific FS support (pretty much like file-roller and archivers support). @herd: what do you guys think?
I'm not a huge fan of "You need to install x to get functionality y" as an elog; But, aside from that, there's no point in changing the structure of the ebuild now. It would just confuse users. I've fixed this the same way Thomas suggested.