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

The Waybar launch.sh to source the .bashrc #483

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

zzvara
Copy link

@zzvara zzvara commented Nov 25, 2024

Supports use-cases for custom Python venvs, for example, for plugins like nextmeeting.

Supports use-cases for custom Python `venvs`, for example, for plugins like `nextmeeting`.
@mylinuxforwork
Copy link
Owner

Hi. Could this cause any side effects? I will test.

@mylinuxforwork
Copy link
Owner

@zzvara What happens if someone switched to zsh? Is it the required to source the .zshrc?

@feikedonia
Copy link

I don't understand what is the benefit of this?

@zzvara
Copy link
Author

zzvara commented Dec 3, 2024

@mylinuxforwork I indeed use .zshrc, but I had to replicate my .zshrc setup to .bashrc, because some programs that interface with bash use .bashrc, not .zshrc. I define Python, sdkman and other development environment stuff to .zshrc and .bashrc and by default these programs add themselves to these rc files.

I realized, that Waybar could not call nextmeeting Python program that uses gcalcli, because nextmeeting lives in my Python venv. While I customize Waybar, I add custom modules that make calls to programs that work correctly after .rc files are loaded.

Maybe sourcing these .rc files in the Waybar launch.sh is not the best way, but I could not find a better place globally, and it worked.

@zzvara
Copy link
Author

zzvara commented Dec 22, 2024

@mylinuxforwork I now have a python env, scala, java, sdkman and soon other stuff which requires that .rc files to be loaded for Waybar. I love how customizable it is.

I now have Teleport VPN login panels, meeting monitors, indoor and outdoor air quality monitors with notifications for me to open the window to get some fresh air inside, because of high CO2.

image

These all need .rc files to be loaded, because the runtimes required for these also add themselves to the .rc file by default, during their installation.

@JumpIn-Git
Copy link

I would just add a option to the ml4w app

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.

4 participants