search

cdown / clipmenu

Clipboard management using dmenu
MIT License
1.14k stars 91 forks source link

README suggestion to ensure DISPLAY is set, is redundant with distribution best practices #230

Open eli-schwartz opened 3 months ago

eli-schwartz commented 3 months ago

Based on discussion at https://github.com/gentoo/gentoo/pull/38031#discussion_r1722326212, with reference to the advice offered in https://github.com/cdown/clipmenu/commit/8538e01e5c3dff35172789e946d1a60df4ed658b

Distros usually have a stock xinitrc sample which imports DISPLAY into the user environment for you. Users who write their own xinitrc or or use alternative approaches to starting the X server and running a window manager should be encouraged to load the distro-wide hook functions (often by copying the default xinitrc and customizing it rather than starting from scratch).

Perhaps say instead, "for example your xinitrc should probably load the distro hooks at /etc/X11/xinit/xinitrc.d/*, one of which most likely runs ${systemctl_command}"

cdown commented 3 months ago

(I'm in Uganda for a while with mobile only, remind me later if I don't get back :-))