Skip to content

Update environment import command in niri-session#3495

Open
Zamanhuseyinli wants to merge 1 commit intoniri-wm:mainfrom
Zamanhuseyinli:patch-1
Open

Update environment import command in niri-session#3495
Zamanhuseyinli wants to merge 1 commit intoniri-wm:mainfrom
Zamanhuseyinli:patch-1

Conversation

@Zamanhuseyinli
Copy link

Since systemctl's import-environment support is now marked as deprecated and therefore gives a warning, I wanted to make such a correction so that the warning screen disappears, but since systemctl does not support import-environment directly anymore, I think it makes more sense to use only dbus or use systemctl in accordance with its new format.

Since systemctl's import-environment support is now marked as deprecated and therefore gives a warning, I wanted to make such a correction so that the warning screen disappears, but since systemctl does not support import-environment directly anymore, I think it makes more sense to use only dbus or use systemctl in accordance with its new format.
@YaLTeR
Copy link
Member

YaLTeR commented Feb 23, 2026

No we are not doing these hacks just to "hide a warning"

@Zamanhuseyinli
Copy link
Author

This isn't a hack or anything; it's a technical method for passing necessary variables using xargs.

@Zamanhuseyinli
Copy link
Author

I added this for compatibility reasons, not just as a warning.

@noahpolimon
Copy link

@YaLTeR i think denying it is just waiting for it to bite you later on when it breaks something. this is no longer a supported method by systemd.

@HigherOrderLogic
Copy link
Contributor

But it still works, no? Also I prefer it over some hacky bash scripts chaining this PR is proposing.

@Zamanhuseyinli
Copy link
Author

Yes, it works, and yes, it's more technical than hacky methods in my opinion, that's why I wanted to add that.Instead of unnecessary scripting clutter, you can use simpler methods like these.

@YaLTeR
Copy link
Member

YaLTeR commented Feb 26, 2026

i'm happy to wait until it does bite me later

@Sempyos Sempyos added the area:session Screen locking, idle inhibit, suspend/resume label Mar 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:session Screen locking, idle inhibit, suspend/resume

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants