phillipberndt / autorandr

Auto-detect the connected display hardware and load the appropriate X11 setup using xrandr
2.51k stars 124 forks source link

[autorandr-launcher] Add makefile & create macro to enable local path of autorandr #350

Closed horstmannmat closed 1 year ago

horstmannmat commented 1 year ago

Hi, this PR adds some improvement to the code;

The internal makefile can be a readme explaining the process, it is up to the owner to define the best

[UPDATE] In fact, since it was assuming that the path would be always the "/usr/bin", but if for any reason it was different. Let's suppose that DESTDIR=~/ PREFIX=.local/ make install TARGETS='autorandr launcher' So I added also in the general makefile to use the user inputs.

The launcher is an alternative to the "udev/systemd setup" that is more stable for some users

The assumption of the PATH, combined with no error checking when executed could be the root cause of why udev/systemd is more stable for some user

Closes #209

horstmannmat commented 1 year ago

cool thing is that this resolves #209

phillipberndt commented 1 year ago

lgtm, thanks!