#76848 TLP 1.8 package misses perl

Message #5 received at submit@debbugs.gnu.org (full text, mbox, reply):

From: glab <glab@disroot.org>

To: bug-guix@gnu.org

Subject: TLP 1.8 package misses perl

Date: Fri, 07 Mar 2025 23:57:05 +0000

I get an error when executing the command

  $ sudo tlp setcharge 40 60
  
/gnu/store/ypdi8gxswgnshhmxxdj66cb51krx1d63-tlp-1.8.0/share/tlp/bat.d/05-thinkpad: 
line 651: perl: command not found
  
/gnu/store/ypdi8gxswgnshhmxxdj66cb51krx1d63-tlp-1.8.0/share/tlp/bat.d/05-thinkpad: 
line 1231: : : integer expression expected

It seems the TLP shell script is employing the wrong perl PATH, and if 
perl is not installed in the current prefix, it fails.

-- 
Gustavo

Message #8 received at submit@debbugs.gnu.org (full text, mbox, reply):

From: Nicolas Goaziou <mail@nicolasgoaziou.fr>

To: glab via Bug reports for GNU Guix <bug-guix@gnu.org>

Cc: glab <glab@disroot.org>, 76848@debbugs.gnu.org

Subject: Re: bug#76848: TLP 1.8 package misses perl

Date: Sat, 08 Mar 2025 20:00:46 +0100

Hello,

glab via Bug reports for GNU Guix <bug-guix@gnu.org> writes:

> I get an error when executing the command
>
>   $ sudo tlp setcharge 40 60
>   /gnu/store/ypdi8gxswgnshhmxxdj66cb51krx1d63-tlp-1.8.0/share/tlp/bat.d/05-thinkpad:
>   line 651: perl: command not found
>   /gnu/store/ypdi8gxswgnshhmxxdj66cb51krx1d63-tlp-1.8.0/share/tlp/bat.d/05-thinkpad:
>   line 1231: : : integer expression expected
>
> It seems the TLP shell script is employing the wrong perl PATH, and if
> perl is not installed in the current prefix, it fails.

Thank you for the report. I committed a fix on master branch. Could you
confirm it solves your issue?

Regards,
-- 
Nicolas Goaziou

Message #14 received at 76848@debbugs.gnu.org (full text, mbox, reply):

From: glab <glab@disroot.org>

To: Nicolas Goaziou <mail@nicolasgoaziou.fr>

Cc: 76848@debbugs.gnu.org

Subject: Re: bug#76848: TLP 1.8 package misses perl

Date: Tue, 11 Mar 2025 13:05:58 +0000

>> I get an error when executing the command
>> 
>>   $ sudo tlp setcharge 40 60
>>   
>> /gnu/store/ypdi8gxswgnshhmxxdj66cb51krx1d63-tlp-1.8.0/share/tlp/bat.d/05-thinkpad:
>>   line 651: perl: command not found
>>   
>> /gnu/store/ypdi8gxswgnshhmxxdj66cb51krx1d63-tlp-1.8.0/share/tlp/bat.d/05-thinkpad:
>>   line 1231: : : integer expression expected
>> 
>> It seems the TLP shell script is employing the wrong perl PATH, and if
>> perl is not installed in the current prefix, it fails.
> 
> Thank you for the report. I committed a fix on master branch. Could you
> confirm it solves your issue?

Thank you. I can confirm you solved this issue.
I can now happily use the new TLP 1.8 without problems.
Have a good day.
-- 
Gustavo

Message #17 received at submit@debbugs.gnu.org (full text, mbox, reply):

From: Nicolas Goaziou <mail@nicolasgoaziou.fr>

To: glab via Bug reports for GNU Guix <bug-guix@gnu.org>

Cc: glab <glab@disroot.org>, 76848-done@debbugs.gnu.org

Subject: Re: bug#76848: TLP 1.8 package misses perl

Date: Tue, 11 Mar 2025 16:57:57 +0100

Hello,

glab via Bug reports for GNU Guix <bug-guix@gnu.org> writes:

>>> I get an error when executing the command
>>>   $ sudo tlp setcharge 40 60
>>>   /gnu/store/ypdi8gxswgnshhmxxdj66cb51krx1d63-tlp-1.8.0/share/tlp/bat.d/05-thinkpad:
>>>   line 651: perl: command not found
>>>   /gnu/store/ypdi8gxswgnshhmxxdj66cb51krx1d63-tlp-1.8.0/share/tlp/bat.d/05-thinkpad:
>>>   line 1231: : : integer expression expected
>>> It seems the TLP shell script is employing the wrong perl PATH, and
>>> if
>>> perl is not installed in the current prefix, it fails.
>> Thank you for the report. I committed a fix on master branch. Could
>> you
>> confirm it solves your issue?
>
> Thank you. I can confirm you solved this issue.
> I can now happily use the new TLP 1.8 without problems.
> Have a good day.

Thanks for the feedback. I’m closing this issue.

Regards,
-- 
Nicolas Goaziou

Send a report that this bug log contains spam.