Skip to content

Add waybar support for several users#1655

Open
corazon008 wants to merge 2 commits into
mylinuxforwork:mainfrom
corazon008:waybar-several-users
Open

Add waybar support for several users#1655
corazon008 wants to merge 2 commits into
mylinuxforwork:mainfrom
corazon008:waybar-several-users

Conversation

@corazon008
Copy link
Copy Markdown
Contributor

Description

Currently, to prevent waybar for running multiple time a loack file is used in /tmp. However /tmp is global for all local users which implies that only one instance of waybar can run.

Changes

So, I used another file which is appended by the username of the current USER, so that every user has it's own lock fie.

How Has This Been Tested?

  • Tested on Arch Linux/Based Distro.
  • Tested on Fedora Linux/Based Distro.
  • Tested on openSuse.

@dwilliam62
Copy link
Copy Markdown
Contributor

My thought would be don't use /tmp at all. Keep any temp files inside the users home dir. Safer. For exactly the reason you stated it's global

@corazon008
Copy link
Copy Markdown
Contributor Author

Which dir should be use for you? One advantage I found for /tmp is that it's cleared at reboot and so the file lock.

@dwilliam62
Copy link
Copy Markdown
Contributor

Which dir should be use for you? One advantage I found for /tmp is that it's cleared at reboot and so the file lock.

I'll leave that up to ML4W, part of startup could be to clear out that lock file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants