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

[REQUEST] pass environment variables to drun-ed apps #2059

Closed
2 tasks done
ildar opened this issue Dec 20, 2024 · 8 comments
Closed
2 tasks done

[REQUEST] pass environment variables to drun-ed apps #2059

ildar opened this issue Dec 20, 2024 · 8 comments

Comments

@ildar
Copy link

ildar commented Dec 20, 2024

Before opening a feature request

  • I checked the next branch to see if the feature has already been implemented
  • I searched existing reports to see if it is already requested.

What is the user problem or growth opportunity you want to see solved?

as rofi is an application launcher it must have ability to pass environment variables to drun-ed apps. This is natural

How do you know that this problem exists today? Why is this important?

this is a useful feature that's why many/most application launchers can do it. E.g. shells and (some) DEs.

Who will benefit from it?

advanced users

Rofi version (rofi -v)

1.7.2

Configuration

N/A

Additional information

No response

@DaveDavenport
Copy link
Collaborator

I am very confused, rofi passes all environment to the program it launches..

I've validated this using a desktop file that does:

[Desktop Entry]
Encoding=UTF-8
Version=1.0
Type=Application
Terminal=false
Exec=env
Name=Env test
Icon=Env test

and launching rofi from the terminal with a set of environment variables and checking if those are printed out.

@ildar
Copy link
Author

ildar commented Dec 20, 2024 via email

@DaveDavenport
Copy link
Collaborator

I don't understand what you want.

This works fine:

MONK=a rofi -show drun | grep -i MONK
MONK=a

@DaveDavenport
Copy link
Collaborator

You claim most launchers can do this? can you show a screenshot of how you do it? that might clear things up?

@DaveDavenport
Copy link
Collaborator

There is nothing in the free desktop standard about this.
https://specifications.freedesktop.org/desktop-entry-spec/latest/

@ildar
Copy link
Author

ildar commented Dec 21, 2024 via email

@DaveDavenport
Copy link
Collaborator

DaveDavenport commented Dec 21, 2024

Ok, this feels dishonest:

Sorry, I didn't claim most DEs but some DEs do this. E.g.

While I said.

You claim most launchers can do this?

and you said in the opening post:

this is a useful feature that's why many/most application launchers can do it.

@DaveDavenport
Copy link
Collaborator

I do not want to continue with this anymore.

@DaveDavenport DaveDavenport closed this as not planned Won't fix, can't repro, duplicate, stale Dec 21, 2024
@davatorium davatorium locked and limited conversation to collaborators Dec 21, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants