Re: [hackers] [slstatus][PATCH] backlight module

From: Ryan <alrayyes_AT_gmail.com>
Date: Sat, 30 Mar 2019 16:03:46 +0100

Well I can confirm it works for following workflow:

- git clone https://git.suckless.org/slstatus
- cd slstatus
- git am backlight-20190305-85a4a18.diff ( or patch -p1 <
backlight-20190305-85a4a18.diff if you want)
- ./slstatus

make and running slstatus works for me.

On Sat, Mar 30, 2019 at 3:59 PM Abdullah <abdullah_AT_abdullah.today> wrote:

> I did check it yesterday. and yeah you should add the backlight.c in
> components directory.
>
> On 30/03, Ryan wrote:
> > I just tested the patch again against master, works for me. You sure you
> > tested it against slstatus master?
> >
> > On Fri, Mar 29, 2019 at 10:34 PM Abdullah <abdullah_AT_abdullah.today>
> wrote:
> >
> > > No such file or directory, backlight.o
> > > This error comes after applying this patch.
> > >
> > > On 29/03, Ryan Kes wrote:
> > > > ---
> > > > Makefile | 1 +
> > > > components/backlight.c | 36 ++++++++++++++++++++++++++++++++++++
> > > > config.def.h | 1 +
> > > > slstatus.h | 3 +++
> > > > 4 files changed, 41 insertions(+)
> > > > create mode 100644 components/backlight.c
> > > >
> > > > diff --git a/Makefile b/Makefile
> > > > index 945b5e3..28a205f 100644
> > > > --- a/Makefile
> > > > +++ b/Makefile
> > > > _AT__AT_ -6,6 +6,7 @@ include config.mk
> > > >
> > > > REQ = util
> > > > COM =\
> > > > + components/backlight\
> > > > components/battery\
> > > > components/cpu\
> > > > components/datetime\
> > > > diff --git a/components/backlight.c b/components/backlight.c
> > > > new file mode 100644
> > > > index 0000000..fcc062a
> > > > --- /dev/null
> > > > +++ b/components/backlight.c
> > > > _AT__AT_ -0,0 +1,36 @@
> > > > +/* See LICENSE file for copyright and license details. */
> > > > +#include <stdio.h>
> > > > +#include <string.h>
> > > > +
> > > > +#include "../util.h"
> > > > +
> > > > +#if defined(__linux__)
> > > > + #include <limits.h>
> > > > + #include <stdint.h>
> > > > + #include <unistd.h>
> > > > +
> > > > + const char *
> > > > + backlight_perc(const char *backlight)
> > > > + {
> > > > + float value;
> > > > + char path[PATH_MAX];
> > > > +
> > > > + if (esnprintf(path, sizeof(path),
> > > > +
> > > "/sys/class/backlight/%s/actual_brightness", backlight) < 0) {
> > > > + return NULL;
> > > > + }
> > > > + if (pscanf(path, "%f", &value) != 1) {
> > > > + return NULL;
> > > > + }
> > > > +
> > > > + return bprintf("%d", (int)(value / 852 * 100));
> > > > + }
> > > > +
> > > > +#else
> > > > + const char *
> > > > + backlight_perc(const char *backlight)
> > > > + {
> > > > + return NULL;
> > > > + }
> > > > +
> > > > +#endif
> > > > diff --git a/config.def.h b/config.def.h
> > > > index e06be66..d0996ef 100644
> > > > --- a/config.def.h
> > > > +++ b/config.def.h
> > > > _AT__AT_ -12,6 +12,7 @@ static const char unknown_str[] = "n/a";
> > > > /*
> > > > * function description argument
> > > (example)
> > > > *
> > > > + * backlight_perc backlight percentage backlight
> name
> > > (intel_backlight)
> > > > * battery_perc battery percentage battery name
> > > (BAT0)
> > > > * NULL on
> > > OpenBSD/FreeBSD
> > > > * battery_state battery charging state battery name
> > > (BAT0)
> > > > diff --git a/slstatus.h b/slstatus.h
> > > > index 08f610a..d9c0b5f 100644
> > > > --- a/slstatus.h
> > > > +++ b/slstatus.h
> > > > _AT__AT_ -1,5 +1,8 @@
> > > > /* See LICENSE file for copyright and license details. */
> > > >
> > > > +/* backlight */
> > > > +const char *backlight_perc(const char *backlight);
> > > > +
> > > > /* battery */
> > > > const char *battery_perc(const char *);
> > > > const char *battery_state(const char *);
> > > > --
> > > > 2.21.0
> > > >
> > > >
> > >
> > >
> > > A K
> > > https://abdullah.today
> > > 69C8 7720 E507 86DF 1D85
> > > EA5E FC59 7891 5CBC 7BC7
> > >
> > >
> > >
>
>
> A K
> https://abdullah.today
> 69C8 7720 E507 86DF 1D85
> EA5E FC59 7891 5CBC 7BC7
>
>
>
Received on Sat Mar 30 2019 - 16:03:46 CET

This archive was generated by hypermail 2.3.0 : Sat Mar 30 2019 - 16:12:21 CET