Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bug: safeeyes shows an error message instead of a tray icon #5728

Closed
2 tasks done
DamienCassou opened this issue Aug 13, 2024 · 8 comments
Closed
2 tasks done

bug: safeeyes shows an error message instead of a tray icon #5728

DamienCassou opened this issue Aug 13, 2024 · 8 comments
Assignees
Labels
bug triage Issues or feature request that have not been triaged yet

Comments

@DamienCassou
Copy link
Contributor

Are you following the right branch?

  • My Nixpkgs and Home Manager versions are in sync

Is there an existing issue for this?

  • I have searched the existing issues

Issue description

Starting SafeEyes shows an error message

The required plugin 'Tray Icon' is missing dependencies!

followed by a link Click here for more information.

image

The documentation says we should start snixembed.

How to reproduce:

{
  services.safeeyes.enable = true;
}

or just install safeeyes and execute it.

Maintainer CC

@Rosuavio

System information

- system: `"x86_64-linux"`
 - host os: `Linux 6.10.3-200.fc40.x86_64, Fedora Linux, 40 (Workstation Edition), nobuild`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.5`
 - channels(root): `"nixpkgs"`
 - channels(temp): `"home-manager, nixpxkgs"`
 - channels(cassou): `"home-manager-22.11.tar.gz, nixpkgs"`
 - nixpkgs: `/home/cassou/Documents/projects/nix-system/nixpkgs`
@archisman-panigrahi
Copy link

archisman-panigrahi commented Aug 15, 2024

Which desktop environment are you using?

You only need snixembed in LXDE/Openbox/MATE(outside Ubuntu MATE)

@DamienCassou
Copy link
Contributor Author

I'm on Fedora Linux with i3wm.

@archisman-panigrahi
Copy link

Can you try the snixembed solution described in the wiki? Also, can you try the native package using pip? People have reported that it works in i3 slgobinath/SafeEyes#617 (comment)

@DamienCassou
Copy link
Contributor Author

sorry for the delay @archisman-panigrahi

Can you try the snixembed solution described in the wiki?

starting snixembed before SafeEyes make SafeEyes appear correctly. I thought this was the only solution so I opened this issue in home-manager rather than in SafeEyes. Do you agree?

can you try the native package using pip?

I have tried but installing with pip complains while compiling cairo. I could investigate further but I don't much of the python ecosystem.

@archisman-panigrahi
Copy link

starting snixembed before SafeEyes make SafeEyes appear correctly. I thought this was the only solution so I opened this issue in home-manager rather than in SafeEyes. Do you agree?

This is normal and expected behavior in the latest version of safeeyes. i3 does not have the necessary backend for the trayicon technology we are using. Snixembed acts like a bridge in this case.

Full fledged DEs like KDE or XFCE don't need snixembed.

@archisman-panigrahi
Copy link

archisman-panigrahi commented Sep 9, 2024

I think there is no bug here. Just add snixembed --fork to startup applications for the safeeyes tray icon to work.

@DamienCassou
Copy link
Contributor Author

@archisman-panigrahi: that was my impression as well, thank you for confirming!

@rycee: we probably need a service for snixembed. The question I have is what kind of API we want given that:

  1. not everyone will need snixembed
  2. the others will want it to be started before SafeEyes

Should there be an option in SafeEyes module or are the two modules completely separate?

@archisman-panigrahi
Copy link

In my opinion, it should be left to the user to install snixembed if they want (safeeyes presents them a link which has detailed instruction of how to install). snixembed is really not a dependency of safeeyes. It is rather required because i3/lxde/mate(without Ubuntu's customizations) etc. do not have out-of-the-box support for kstatusnotifier.

DamienCassou added a commit to DamienCassou/home-manager that referenced this issue Oct 9, 2024
https://git.sr.ht/~steef/snixembed

This is used by SafeEyes (another home-manager) module to show a
systemtray icon.

This fixes issue nix-community#5728.
@DamienCassou DamienCassou mentioned this issue Oct 9, 2024
6 tasks
DamienCassou added a commit to DamienCassou/home-manager that referenced this issue Oct 9, 2024
https://git.sr.ht/~steef/snixembed

This is used by SafeEyes (another home-manager) module to show a
systemtray icon.

This fixes issue nix-community#5728.
@rycee rycee closed this as completed in 03f8e0b Oct 10, 2024
Mikilio pushed a commit to Mikilio/home-manager that referenced this issue Oct 11, 2024
This is used by SafeEyes (another home-manager) module to show a
systemtray icon. See https://git.sr.ht/~steef/snixembed.

Fixes nix-community#5728
colonelpanic8 pushed a commit to colonelpanic8/home-manager that referenced this issue Dec 30, 2024
This is used by SafeEyes (another home-manager) module to show a
systemtray icon. See https://git.sr.ht/~steef/snixembed.

Fixes nix-community#5728
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug triage Issues or feature request that have not been triaged yet
Projects
None yet
Development

No branches or pull requests

5 participants