| 20 Sep 2024 |
 cleverca22 | In reply to @janne.hess:helsinki-systems.de
That's valid in Perl 🤡 is it still valid in the latest version? has something changed?
what does the error mean exactly? why is it treating the secret key as a module? | 09:17:48 |
 @janne.hess:helsinki-systems.de | readFile is only in the c++ codebase | 09:49:04 |
| @janne.hess:helsinki-systems.de | for reading a file, you would use:
use File::Slurper qw(read_text);
# …
read_text($secretKeyFile);
| 09:50:00 |
| @janne.hess:helsinki-systems.de | * for reading a file in the perl codebase, you would use:
use File::Slurper qw(read_text);
# …
read_text($secretKeyFile);
| 09:50:09 |
| @janne.hess:helsinki-systems.de | * for reading a file in the perl codebase, you would use:
use File::Slurper qw(read_text);
# …
my $secretKey = read_text($secretKeyFile);
| 09:50:31 |
| @janne.hess:helsinki-systems.de | I assume something went wrong here: https://github.com/NixOS/hydra/commit/b94a7b6d5c56362af9ea85d944f8454d861ec001 | 09:52:18 |
| @janne.hess:helsinki-systems.de | But there is no reason to use the nix function rather than the perl way of life | 09:52:29 |
 Mic92 | cleverca22: https://github.com/NixOS/hydra/pull/1413 | 09:53:13 |
| cleverca22 | In reply to @joerg:thalheim.io
cleverca22: https://github.com/NixOS/hydra/pull/1413 CI tests fail, still deploying on my end | 10:04:02 |
| cleverca22 | ( STDERR ) job 69 Can't locate File/Slurp.pm in @INC (you may need to install the File::Slurp module) | 10:04:26 |
| Mic92 | cleverca22: indeed John just imagined this function... It's also present in c++ | 10:05:30 |
| Mic92 | in the nix code base | 10:05:33 |
| cleverca22 | https://github.com/NixOS/nix/blob/master/src/perl/lib/Nix/Utils.pm#L7 | 10:10:45 |
| cleverca22 | hmmm, readFile is part of Nix::Utils? | 10:11:09 |
| cleverca22 | oh, and narinfo doesnt import that now!? | 10:11:20 |
| cleverca22 | bingo, that got removed in a recent commit | 10:11:44 |
| Mic92 | cleverca22: https://github.com/NixOS/hydra/pull/1413/files#diff-99bd8eee901b9b16f07b535be184e0d630739e79cb8678b8465e82ed54f2543aR42 | 10:14:28 |
| cleverca22 | yeah, thats a more portable solution | 10:14:54 |
