-
Notifications
You must be signed in to change notification settings - Fork 24
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
update for kernels >= 4.14 #5
Conversation
remove include that is not needed anymore Signed-off-by: Henning Schild <[email protected]>
is this repo unmaintained? |
hey, sorry. |
Thanks, because i am working on another patch that i would like to get merged. (here?) That is also coming from tpacpi-bat. |
hey look, it only took me 1 full year to actually test this! ...sorry :( EDIT: ok not QUITE a fully year, but we can take on the additional full year it took me to notice the pull req, and call it 2 full years |
oh progress ;) thx i do not even remember what that other patch was, interesting to see that it took you so long to adopt >= 4.15 what is "upstream" if this is potentially not? |
unfortunately this is as upstream as it gets, afaik. (EDIT: ha! "maintain"...) |
not sure about other distros but gentoo has both ... not sure how many users there are https://github.com/gentoo/gentoo/tree/master/app-laptop/tpacpi-bat maybe that helps in terms of motivation, there are people using it! |
yea i suppose i should maintain these better. please feel free to pull-req any patches you like. i will do at least basic code review and testing on at least one thinkpad. |
Btw. The NixOS community also maintains a fork now. Maybe we can consolidate the effort somehow: mkottman#89 |
remove include that is not needed anymore
Signed-off-by: Henning Schild [email protected]