Create icons folder if it doesn't exist#367
Create icons folder if it doesn't exist#367Et0h merged 1 commit intoSyncplay:masterfrom Lctrs:patch-1
Conversation
If it doesn't exist first, `hicolor` will be stripped from the path by the `cp` command
|
I'm expecting this is a non-controversial change, but as I use Windows it would be useful if someone with makefile experience can confirm that this is the correct approach. |
|
@Lctrs What OS are you using which you believe exhibits this behaviour (of cp not creating the hicolor folder when the icon folder does not exist) and have you confirmed it? |
|
When trying to create a flatpak for the client. The icons folder doesn't exist in this case beforehand, and the |
daniel-123
left a comment
There was a problem hiding this comment.
I've not seen this crop up up until now mostly because the assumption behind this installer script was that it will typically run on a "normal" operating system. Which pretty much always already has some software with icons, so I didn't think about situation where it doesn't exist.
It think that overall this is completely reasonable change. Though we don't have an official flatpak build process, so there is some risk of your package breaking unintentionally with some other changes in future.
If it doesn't exist first,
hicolorwill be stripped from the path by thecpcommand