artemsen / swayimg

Image viewer for Wayland
MIT License
380 stars 31 forks source link

SWAYSOCK variable is not defined #105

Closed AERDU closed 11 months ago

AERDU commented 11 months ago

sway = no got removed with dbf6121ff4a0ff748cf4eab79cd1e1b7cc1e97c3 as mentioned in the commit, you can now emulate the same setting by unsetting SWAYSOCK. i'm sure everything works as it should internally, but these lines might need to be rewritten https://github.com/artemsen/swayimg/blob/d026cb9d182ca742d3adab67b3fa7ba7db1d3bd6/src/sway.c#L256 would it be ok to just remove the error now since unsetting SWAYSOCK is default behavior on everything except sway? Sway users without SWAYSOCK set might not be aware that they essentially have set sway = no though, which could be an issue. reverting some of dbf6121ff4a0ff748cf4eab79cd1e1b7cc1e97c3 or adding --silent might be preferable.