flameshot-org / flameshot

Powerful yet simple to use screenshot software :desktop_computer: :camera_flash:
https://flameshot.org
GNU General Public License v3.0
24.98k stars 1.6k forks source link

Flameshot does not restart if toggled by a keybinding(only for cutouts) #2661

Open txtyash opened 2 years ago

txtyash commented 2 years ago

Flameshot Version

flameshot -v ouput:

Flameshot v11.0.0 (-)
Compiled with Qt 5.15.3

After using flameshot once I have to kill the flameshot process with killall flameshot to be able to use flameshot again.

Installation Type

User repository (AUR)

Operating System type and version

Linux arch 5.18.1-zen1-1-zen #1 ZEN SMP PREEMPT_DYNAMIC Mon, 30 May 2022 17:53:16 +0000 x86_64 GNU/Linux

Description

After using flameshot once I have to kill the flameshot process with killall flameshot to be able to use flameshot again. Now this only happens when I use flameshot with a keybinding or application launcher. From the command line I'm able to kill(have to kill) flameshot with ctrl-c in the terminal since running flameshot gui, taking an "area select" screenshot & saving it doesn't kill the process.

Steps to reproduce

Without CLI:

  1. run flameshot gui(with an OS binding)
  2. Take a "area select" screenshot
  3. Save
  4. Repeat step 1 and nothing happens

With CLI:

  1. run flameshot gui in one terminal
  2. Use "area select" screenshot and save
  3. The process inside the terminal doens't end
  4. Do not end the process
  5. Repeat step 1 in another terminal and it doesn't work

Screenshots or screen recordings

No response

System Information

uname -a output:

Linux arch 5.18.1-zen1-1-zen #1 ZEN SMP PREEMPT_DYNAMIC Mon, 30 May 2022 17:53:16 +0000 x86_64 GNU/Linux

Neofetch output:

OS: Arch Linux x86_64
Host: VivoBook_ASUSLaptop X412DA_X412DA 1.0
Kernel: 5.18.1-zen1-1-zen
Uptime: 2 days, 8 hours, 6 mins
Packages: 1210 (pacman)
Shell: fish 3.4.1
Resolution: 1920x1080
WM: xmonad
Theme: Adwaita [GTK2/3]
Icons: Adwaita [GTK2/3]
Terminal: kitty
CPU: AMD Ryzen 5 3500U with Radeon Vega Mobile Gfx (8) @ 2.100GHz
GPU: AMD ATI Radeon Vega Series / Radeon Vega Mobile Series
Memory: 3277MiB / 5939MiB
mmahmoudian commented 2 years ago

@zim0369 if you install from AUR, it is impossible to have that version number. Please double check your version number and your installation method. Also, is it safe to assume that you are on X11? Xmonad so far does not support wayland, but there are forks that do.

txtyash commented 2 years ago

@mmahmoudian

❯ : paru -S flameshot
resolving dependencies...
looking for conflicting packages...
.
.
.
❯ : flameshot -v
Flameshot v11.0.0 (-)
Compiled with Qt 5.15.3

I'm sure I'm using X11

mmahmoudian commented 2 years ago

Paru is not only an AUR helper, it is also a pacman wrapper.

If you use pacman (which does not support AUR) to search for flameshot, you will see that there is a package in the community repo that has a package named flameshot

❯  pacman --sync --search flameshot
community/flameshot 11.0.0-5
    Powerful yet simple to use screenshot software

You can also get more info about that from pacman:

❯  pacman --sync --info flameshot
Repository      : community
Name            : flameshot
Version         : 11.0.0-5
Description     : Powerful yet simple to use screenshot software
Architecture    : x86_64
URL             : https://github.com/flameshot-org/flameshot
Licenses        : GPL
Groups          : None
Provides        : None
Depends On      : qt5-svg  hicolor-icon-theme  kguiaddons
Optional Deps   : gnome-shell-extension-appindicator: for system tray icon if
                  you are using Gnome
                  grim: for wlroots wayland support
                  xdg-desktop-portal: for wayland support, you will need the
                  implementation for your wayland desktop environment
Conflicts With  : None
Replaces        : None
Download Size   : 507.50 KiB
Installed Size  : 2079.67 KiB
Packager        : Antonio Rojas <arojas@archlinux.org>
Build Date      : Sat 23 Apr 2022 04:26:49 PM EEST
Validated By    : MD5 Sum  SHA-256 Sum  Signature

the package flameshot can also be found on the website:

The AUR package on the other hand has the name flameshot-git as clearly explained on our website and can be found on AUR website as well:

https://aur.archlinux.org/packages/flameshot-git/

Therefore, you have installed the Flameshot from Arch's community repo and not AUR. 😉

Please:

  1. Edit your issue description accordingly
  2. try installing via the AUR and see if you still are experiencing the same issue.

Thanks

txtyash commented 2 years ago

@mmahmoudian I installed the AUR version and I still experience the same problem. The flameshot process keeps running once started, doesn't matter if I take a screenshot or I don't.

BTW, I have been able to fix it by prepending my binding commands with killall flameshot

From my config:

++ [ ((0, xK_Print), spawn "killall flameshot; flameshot full -p ~/screenshots_full -c"),
((shiftMask, xK_Print), spawn "killall flameshot; flameshot gui -s -p ~/screenshots_cutout -c"),
mmahmoudian commented 2 years ago

have you tried running the daemon first (e.g when your system boots), and then try to use the flameshot gui on two separate terminal to see if you can reproduce the issue? If not, please let me know the result:

  1. killall flameshot
  2. open 3 terminals
  3. in terminal 1: flameshot
  4. in terminal 2: flameshot gui and do whatever you want
  5. in terminal 3: flameshot gui and let me know if it works
txtyash commented 2 years ago
  1. done
  2. done
  3. stays running in the terminal(no output)
  4. worked.
  5. didn't work. stuck command for a while then both the flameshot gui commands free the prompt after awhile except flameshot command in terminal 1.
borgmanJeremy commented 2 years ago

This might be xmonad specific. I cannot replicated on KDE + Arch.