Flameshot doesn't work with sway

[bblacher]

Flameshot Version

Flameshot v 0.10.1

Installation Type

User repository (AUR)

Operating System type and version

Arch Linux Rolling (Kernel 5.15.4)

Description

Flameshot starts, it does show the tray icon. Clicking it does nothing, the screenshot tool won't open.

Steps to reproduce

• Use sway
• Run Flameshot
• It won't work

Screenshots or screen recordings

Here's the log:
https://bin.snopyta.org/?4cf0e34e4783de3f#HzdpEfg4EoEwDPLVvMzqETpPszKQKhwCxGQMGRrfaPVM

System Information

OS: Arch Linux x86_64
Host: Predator PH317-52 V1.26
Kernel: 5.15.4-arch1-1
Resolution: 1920x1080
WM: sway
CPU: Intel i7-8750H (12) @ 2.200GHz
GPU: Intel CoffeeLake-H GT2 [UHD Graphics 630]
GPU: NVIDIA GeForce GTX 1050 Ti Mobile
Memory: 2039MiB / 15842MiB

Using Wayland with sway.

[mmahmoudian]

Have you already followed this:

https://github.com/flameshot-org/flameshot/blob/master/docs/Sway%20and%20wlroots%20support.md

[bblacher]

I didn't, but after doing all of the above mentioned steps it's still the same behaviour.

[RiedleroD]

works for me
I'm using current master and sway 1.6.1, no special workarounds.

[mmahmoudian]

@Benibla124 i just realized you have not provided the full version info (or your Flameshot version is some months old) try force installing the AUR (e.g paru flameshot-git) and then give us the full version info which should contain the git commit hash as well.

[bblacher]

I tried with flameshot-git, still didn't change anything.

[RiedleroD]

Works for me with stable (0.10.2-1) as well - @Benibla124 are you sure you've got the dependencies installed? They're grim xdg-desktop-portal xdg-desktop-portal-wlr.

[bblacher]

@RiedleroD Yeah, got them installed.

[mmahmoudian]

@Benibla124 do you have notification manager?

[RiedleroD]

Just freshly installed sway on my new laptop; can now reproduce. I'll send an update once I figure out which dependency I need to install.

[RiedleroD]

I GOT IT
IT DEPENDS ON GRIM WHAT THE FUCK

someone explain this to me.

[borgmanJeremy]

https://github.com/flameshot-org/flameshot/blob/master/docs/Sway%20and%20wlroots%20support.md

[RiedleroD]

wtf, didn't read that because that info is supposed to be in the package manager :/

edit: (no, it's not even in the optional deps)

[borgmanJeremy]

Sorry we don't control the packaging for upstream.

[RiedleroD]

ik, I'm sending the guy that packaged flameshot a mail rn

[borgmanJeremy]

I'm not sure how they would package it so grim is only requires on sway.

Edit: maybe just needs to be put in the arch wiki entry.

[RiedleroD]

as I said, optional dependencies. They can even add an explanation for each optdep. (and that's one reason why I think arch is better than debian ;P)

[mmahmoudian]

In the AUR I have added GRIM ages ago as optional dependency (check line 103):

https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=flameshot-git

[hosiet]

Adding optional depends similar to Arch is also always possible on debian via Recommends or Suggests fields since the very beginning. The only part missing is the conditional-depends feature, which I think won't be available in near future.

If package grim is useful to Sway users, we can update dependency fields in deb packages. The only question is that whether we should use the Recommends field or Suggests field:

• Depends: echo to users during installation, installed by default; will fail if not exist
• Recommends: echo to users during installation, installed by default; will not fail if user opt not to install
• Suggests: echo to users during installation, not installed by default; will not fail if user opt not to install

I think using Suggests or Recommends may all be reasonable.

[mmahmoudian]

@hosiet I would go with "Recommends" as it is not a massive library.

[hosiet]

Thanks, I have implemented it in Debian's flameshot (version 11.0~rc1+ds1-2). Please also see the Pull Request at #2212 .