search

tvrzna / emptty

Dead simple CLI Display Manager on TTY
MIT License
696 stars 25 forks source link

Fix typo in SAMPLES.md #120

Closed tonymanou closed 3 months ago

tonymanou commented 3 months ago

Fix user-specific custom-sessions directory path, to match the layout from system-wide's config.
I also took the liberty to also remove the trailing slash... this can be reverted if you wish.

tvrzna commented 3 months ago

Hello, thank you for the PR.

I agree, this change would be better. However it's not possible to have ~/.config/emptty file and ~/.config/emptty/ folder. It would require some kind of backward compatibility to not make it a breaker. Also current ~/.config/emptty file would require new name/path convention (e.g. script or start in ~/.config/emptty/). And even though I change it and make it backward compatible, I'm not sure, if it will be used since the backward compatibility would be still there.

tonymanou commented 3 months ago

Oh, did not notice the conflict between file and directory names, bummer...
This is unrelated to this PR but maybe you could circumvent this conflict by adding a .d suffix to the directory name, like some programs that support both a config file and a directory in /etc do: grub.d, modules.d, mkinitcpio.d... up to you!

I guess we can close this PR.