[Linux] Place icons in proper dirs, according to icon size.

[Fisiu]

Signed-off-by: Mariusz Fik mariusz@fidano.pl

[krzemin]

I wonder if target icon files should have size suffixes when they are copied to size-specific directories.

I mean $${DATADIR}/icons/hicolor/16x16/apps/qnapi.png instead of $${DATADIR}/icons/hicolor/16x16/apps/qnapi-16.png, analogously for other sizes.

What do you think?

[Fisiu]

Yes, You're right. I'll update pull rq when get back home.

[Fisiu]

It seems files{size}.extra (in qnapi.pro) doesn't work well with multiple items. Maybe just rename files res/qnapi-{size}.png → res/icons/{sizexsize}/apps/qnapi.png?

[krzemin]

Did you try setting full target path of the file? I mean this instead of only providing directory in path:

icons32.files = res/qnapi-32.png
icons32.path = $${DATADIR}/icons/hicolor/32x32/apps/qnapi.png

[Fisiu]

Then qnapi.png dir is created, and qnapi-{size}.png inside it.

[krzemin]

Generally, it's OK to change directory structure in way you've proposed (res/icons/size/qnapi.png), but you have to be careful since current icon paths are referred to in several places (e.g. in res/resources.qrc, maybe also in some .ui files).

[Fisiu]

Pull request updated ;-)