| 12 Aug 2021 |
 das_j | (I don't know what you mean :D) | 19:10:56 |
| 13 Aug 2021 |
|  nixinator joined the room. | 01:55:50 |
| 15 Aug 2021 |
|  Rev. CornWallace III (novus ordo seclorum) changed their display name from cw (not sticky anymore) to cw (got the delta). | 00:57:52 |
| Rev. CornWallace III (novus ordo seclorum) changed their display name from cw (got the delta) to cw (just got delta). | 00:58:35 |
| 16 Aug 2021 |
 @grahamc:nixos.org | accidentally fixing bugs in hydra over here: https://github.com/NixOS/hydra/pull/998#issuecomment-899799305 | 20:36:18 |
|  Finn Behrens (he/him) changed their display name from Finn Behrens to Finn Behrens (he/him). | 20:40:48 |
| 17 Aug 2021 |
| @grahamc:nixos.org | das_j: is your hydra public? | 13:50:59 |
| das_j | In reply to @grahamc:nixos.org
das_j: is your hydra public? It's not and I don't think it ever will since it's processing customer-relevant data | 13:54:07 |
| @grahamc:nixos.org | cool | 13:54:16 |
| das_j | Well depends | 13:54:21 |
| das_j | Also I don't really trust it security-wise ;) | 13:54:29 |
| @grahamc:nixos.org | I ask because I somewhat suspect your "restart components" PR makes it possible to restart components without being logged in... not totally sure the behavior of chaining on admin | 13:54:52 |
| das_j | well I stole it from sub news_submit : Chained('admin') PathPart('news/submit') Args(0) { because I thought that would be secure | 13:55:49 |
| das_j | I haven't seen anyone who is not admin submitting news | 13:56:03 |
| @grahamc:nixos.org | ah gotcha | 13:56:17 |
| @grahamc:nixos.org | yeah probably | 13:56:45 |
| das_j | I do hope so :D | 13:56:56 |
| das_j | Btw, is the eval-triggering API endpoint protected now? | 13:57:10 |
| @grahamc:nixos.org | no | 18:51:23 |
| 18 Aug 2021 |
|  p10r left the room. | 00:38:03 |
| @grahamc:nixos.org | https://github.com/NixOS/hydra/pull/1001 | 19:46:17 |
| @grahamc:nixos.org | RFR as always ... :P | 19:51:53 |
| 19 Aug 2021 |
|  hexagonk joined the room. | 02:38:55 |
| @grahamc:nixos.org | ugh I peppered "use warnings" around and it is a lot | 20:59:00 |
| @grahamc:nixos.org | Anyone know automake? I made this change:
https://github.com/NixOS/hydra/pull/1002/files#diff-0462e381b2fb3286568215681c8983490a37ac9ae0f0c5ee304df7fa6426d4af
and after it runs perlcritic . it errors with:
/nix/store/kxj6cblcsd1qcbbxlmbswwrn89zcmgd6-bash-4.4-p23/bin/bash: line 9: ./perlcritic: No such file or directory
FAIL: perlcritic
==================
1 of 1 test failed
==================
note I don't know automake, so this was totally a shot in the dark.
| 21:07:18 |
 tomberek | grahamc (he/him): the syntax/style-checker isn't in scope? perlPackages.PerlCritic | 21:13:53 |
| @grahamc:nixos.org | it is in scope, that error message is immediately after successfully running perlcritic, but it isn't available at ./perlcritic of course | 21:15:16 |
 andi- | Perhaps since it is in the TESTS list? I haven't looked into the details but I could imagine the test scripts being "built" and then execute them | 21:16:42 |
| andi- | keep in mind that make targets usually are there for producing said target | 21:17:01 |
| @grahamc:nixos.org | right | 21:17:22 |
