bug#51086: closed (tlp error in tlp-func-base)

[GNU bug Tracking System]

Your message dated Fri, 08 Oct 2021 18:44:41 +0200
with message-id <87lf33ihza.fsf@nicolasgoaziou.fr>
and subject line Re: bug#51086: tlp error in tlp-func-base
has caused the debbugs.gnu.org bug report #51086,
regarding tlp error in tlp-func-base
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
51086: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=51086
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems
> --- Begin Message ---
>
>
>
> Subject: 
>
> tlp error in tlp-func-base
>
>
>
>
> Date: 
>
> Thu, 07 Oct 2021 17:53:23 +0100
>
>
>
>
> User-agent: 
>
> mu4e 1.6.6; emacs 27.2
>
>
>
>
> Hi,
> During the system boot, tlp generates an error:
> /gnu/store/68mpiffl51mcrss7zy26dnqfx3d5v2vv-tlp-1.4.0/share/tlp/tlp-func-base:
> line 528 /usr/share/tlp/bat.d/[0-9][0-9]*[a-z]: No such file or
> directory.
>
> It looks that the path to the bat.d directory is not modified during the
> package creation. I think that the following patch should fix the
> problem:
> diff --git a/gnu/packages/linux.scm b/gnu/packages/linux.scm
> index 34aba8377f..d673a4f281 100644
> --- a/gnu/packages/linux.scm
> +++ b/gnu/packages/linux.scm
> @@ -6773,6 +6773,7 @@ interface in sysfs, which can be accomplished with the 
> included udev rules.")
>                 (setenv "TLP_BIN" (string-append out "/bin"))
>                 (setenv "TLP_TLIB" (string-append out "/share/tlp"))
>                 (setenv "TLP_FLIB" (string-append out "/share/tlp/func.d"))
> +               (setenv "TLP_BATD" (string-append out "/share/tlp/bat.d"))
>                 (setenv "TLP_ULIB" (string-append out "/lib/udev"))
>                 (setenv "TLP_CONFDEF"
>                         (string-append out "/share/tlp/defaults.conf"))
>
> Regards,
> Alex
>
>
>
> --- End Message ---
> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#51086: tlp error in tlp-func-base
>
>
>
>
> Date: 
>
> Fri, 08 Oct 2021 18:44:41 +0200
>
>
>
>
> User-agent: 
>
> Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux)
>
>
>
>
> Hello,
>
> Alex Devaure <ajadevaure@gmail.com> writes:
>
> > During the system boot, tlp generates an error:
> > /gnu/store/68mpiffl51mcrss7zy26dnqfx3d5v2vv-tlp-1.4.0/share/tlp/tlp-func-base:
> > line 528 /usr/share/tlp/bat.d/[0-9][0-9]*[a-z]: No such file or
> > directory.
> >
> > It looks that the path to the bat.d directory is not modified during the
> > package creation. I think that the following patch should fix the
> > problem:
> > diff --git a/gnu/packages/linux.scm b/gnu/packages/linux.scm
> > index 34aba8377f..d673a4f281 100644
> > --- a/gnu/packages/linux.scm
> > +++ b/gnu/packages/linux.scm
> > @@ -6773,6 +6773,7 @@ interface in sysfs, which can be accomplished with the 
> > included udev rules.")
> >                 (setenv "TLP_BIN" (string-append out "/bin"))
> >                 (setenv "TLP_TLIB" (string-append out "/share/tlp"))
> >                 (setenv "TLP_FLIB" (string-append out "/share/tlp/func.d"))
> > +               (setenv "TLP_BATD" (string-append out "/share/tlp/bat.d"))
> >                 (setenv "TLP_ULIB" (string-append out "/lib/udev"))
> >                 (setenv "TLP_CONFDEF"
> >                         (string-append out "/share/tlp/defaults.conf"))
>
> Indeed! I created a patch with a proper commit message and applied it in
> your name.
>
> Thank you.
>
> Regards,
> -- 
> Nicolas Goaziou
>
>
> --- End Message ---