Re: [PATCH 2/2] gnu: Add nyx.

[Ludovic Courtès]

Tobias Geerinckx-Rice <address@hidden> skribis:

> On 30/01/17 07:13, ng0 wrote:
>> Tobias Geerinckx-Rice <address@hidden> writes:
>>> This adds Nyx (no relation), a handy status monitor for Tor relays.
>> 
>> Cool!
>> So it seems you have evaded the problems with the custom setup
>> phase they have, which I ran into. Strange. But nice.
>> 
>> I think it's okay, take a look at this search and the reactions I
>> got about it.
>> https://lists.gnu.org/archive/cgi-bin/namazu.cgi?query=nyx&submit=Search&idxname=guix-devel
>
> Oy, that's... so recent I completely missed it? I'm still trudging
> through the post-exam backlog & didn't know you'd submitted this too!
>
>> For the git commit you could mention that this used to be called
>> "arm" and since then has been replaced by "nyx", see the comment
>> Ludovic gave.
>
> Even better. I'll add it to the description field as well.

Yeah if you can make sure to integrate any missing bits of ng0’s
original patch and to address what was previously raised during review,
I think you can go ahead and push.

Thanks!

Ludo’.