TLP 1.8 package misses perl

  • Done
  • quality assurance status badge
Details
2 participants
  • glab
  • Nicolas Goaziou
Owner
unassigned
Submitted by
glab
Severity
normal

Debbugs page

glab wrote 6 days ago
(address . bug-guix@gnu.org)
fcc0506edc841c69838776d0976f4701@disroot.org
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
Nicolas Goaziou wrote 5 days ago
(name . glab via Bug reports for GNU Guix)(address . bug-guix@gnu.org)(name . glab)(address . glab@disroot.org)(address . 76848@debbugs.gnu.org)
874j03ibz5.fsf@nicolasgoaziou.fr
Hello,

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

Toggle quote (11 lines)
> 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
glab wrote 2 days ago
(name . Nicolas Goaziou)(address . mail@nicolasgoaziou.fr)(address . 76848@debbugs.gnu.org)
c34c6ee15226612121c44c434d1b2f60@disroot.org
Toggle quote (16 lines)
>> 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
Nicolas Goaziou wrote 2 days ago
(name . glab via Bug reports for GNU Guix)(address . bug-guix@gnu.org)(name . glab)(address . glab@disroot.org)(address . 76848-done@debbugs.gnu.org)
87h63z5zlm.fsf@nicolasgoaziou.fr
Hello,

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

Toggle quote (17 lines)
>>> 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
?
Your comment

Commenting via the web interface is currently disabled.

To comment on this conversation send an email to 76848@debbugs.gnu.org

To respond to this issue using the mumi CLI, first switch to it
mumi current 76848
Then, you may apply the latest patchset in this issue (with sign off)
mumi am -- -s
Or, compose a reply to this issue
mumi compose
Or, send patches to this issue
mumi send-email *.patch
You may also tag this issue. See list of standard tags. For example, to set the confirmed and easy tags
mumi command -t +confirmed -t +easy
Or, remove the moreinfo tag and set the help tag
mumi command -t -moreinfo -t +help