koiuo / randrctl

Profile based screen manager for X
GNU General Public License v3.0
47 stars 12 forks source link

Question: Removal of --system #20

Closed mmellison closed 4 years ago

mmellison commented 5 years ago

Thank you for this awesome utility - I've been using it for some time.

I recently tried upgrading to the latest version and noticed that the --system flag was removed. Was this intentional? I relied on storing profiles in /etc so that I could call randrctl from my login manager to setup the screens on startup (e.g. LightDM's display-setup-script).

koiuo commented 5 years ago

I was not aware anyone is using it. So decided to remove it and simplify code a little bit. Should not be hard to get it back though.

I'll try to take a look into that as soon as i have time (hopefully this weekend).

koiuo commented 5 years ago

@seglberg , could you please elaborate, how exactly randrctl --system is used in your case?

Do you only use /etc to make profiles available for the root user (or whoever owns lightdm process)?Would it work for you if I added an option to set home directory for randrctl?

I would envision something like

# randrctl --dir /home/seglberg/.config/randrctl

or if you want to make your profiles readable not only for root

# chown a+r -R /home/seglberg/.config/randrctl
# ln -s  /home/seglberg/.config/randrctl /etc/randrctl
# randrctl --dir /etc/randrctl
mmellison commented 5 years ago

A dir option would be just fine, as you mentioned. Honestly, makes it a bit more flexible.

I'm just looking to store my profiles outside of my home directory for different processes to use (which aren't always running as my own user). Thus your symlink example would work great.

On Wed, Nov 28, 2018, 11:57 AM Dmytro Kostiuchenko <notifications@github.com wrote:

@seglberg https://github.com/seglberg , could you please elaborate, how exactly randrctl --system is used in your case?

Do you only use /etc to make profiles available for the root user (or whoever owns lightdm process)?Would it work for you if I added an option to set home directory for randrctl?

I would envision something like

randrctl --dir /home/seglberg/.config/randrctl

or if you want to make your profiles readable not only for root

chown a+r -R /home/seglberg/.config/randrctl

ln -s /home/seglberg/.config/randrctl /etc/randrctl

randrctl --dir /etc/randrctl

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/edio/randrctl/issues/20#issuecomment-442521984, or mute the thread https://github.com/notifications/unsubscribe-auth/AAwz7_mpJ-3_AucMZslfgE16WDI6aOFbks5uzsBygaJpZM4Y3oS6 .

koiuo commented 5 years ago

I don't have time to work on this atm. Likely it will only happen after the New Year. Any help with this would be highly appreciated.