search

koiuo / intelpwm-udev

Script and udev rule to set backlight PWM frequency on laptops with Intel i915 GPU
https://127001.me/post/eliminate-backlight-flicker-with-i915/
17 stars 6 forks source link

Added ansible installer #1

Closed sergioperez closed 4 years ago

sergioperez commented 4 years ago

I created a small installer using ansible, so this could be installed with just one line and without adding a big complexity when maintaining the installer

koiuo commented 4 years ago

I approved the PR, feel free to merge as is or to experiment with the directory structure if you feel like it.

sergioperez commented 4 years ago

This is great, thanks a lot!

I was thinking about adding a PKGBUILD for AUR, and a nix derivation (once I figure that one out) to this repo. Your contribution fits great here.

What do you think about dir structure like this to manage installation files?

.
└── install
    ├── ansible
    │   └── install.yaml
    ├── aur
    │   └── PKGBUILD
    └── nix
        └── intelpwm-udev.nix

Is this reasonable? Would it be possible (if required at all) to adjust install.yaml somehow to work from install/ansible/ directory?

In any case this already very helpful.

First of all, thanks to you! Your script made my x230 usable after I swapped my TN screen with an IPS.

About that organization, yeah! I think this is a great idea, more than reasonable :)

The Ansible installer would require just a minor modification in the paths, I'm adding it to the PR!