<cole-h>
eval-7 has been rebooting for a hot minute...
<gchristensen>
yeah ... not sure what is up with that
<gchristensen>
maybe something really simple, like touching a file on the machine if it has been rebooted and amking it do a deploy with a --reboot flag if the file is absent
<andi->
Any objections against adding `clap` to ofborg for CLI argument handling? I am adding a new command and I'd like to reduce the amount of hardcoded variables and avoid reimplementing the argument parsing "wheel"
<gchristensen>
by all means :)
<gchristensen>
basically any PR making it nicer like that is going to be wanted
<gchristensen>
related, I think it still uses cargo2nix and naersk would be good too
<andi->
Didn't I switch it to crate2nix some time ago?
<gchristensen>
I don't remember :)
<andi->
Yeah it is crate2nix but I can have a look if that is doable. My currently favorite flavor of rust tooling is import-cargo based but these things change almost every quarter..
<andi->
I am trying to build things through ofborg locally without requiring rabbitmq etc..
<andi->
to allow me to test your PR locally without any kinds of setup
<andi->
yay, a few more changes down the road it indeed seems to work as intended :D
<andi->
> May 16 16:45:14.810 INFO job{pr=123170}: ofborg::tasks::build: ok built (HashMismatch), building
<{^_^}>
error: syntax error, unexpected ':', expecting ')', at (string):494:10
<{^_^}>
[ofborg] @andir opened pull request #561 → Remove the origin prefix from the fallback branch name in BuildWorker → https://github.com/NixOS/ofborg/pull/561
<gchristensen>
yay :)
<andi->
Hopefully not the last PR of the day but at least some work that I can already PR ;-)
<gchristensen>
:)
<gchristensen>
andi-: how did it work before ....?
<gchristensen>
seems weird
<andi->
It is never None
<andi->
except for my new code
<gchristensen>
ah
<{^_^}>
[ofborg] @andir opened pull request #562 → WIP: Introduce an one-off-build command to reproduce ofBorg locally → https://github.com/NixOS/ofborg/pull/562
<andi->
gchristensen: If you want naersk you'll have to add that to the hydra job as an input
<gchristensen>
andi-: ehh... I can do that if you'd like but I don't feel like I need to migrate
<gchristensen>
unless you think we should
<gchristensen>
also, there is a minor update in the pedantry check, mind doing that?