Re: [PATCH] Add bwm-ng.

2016-08-28 Thread Ricardo Wurmus
Arun Isaac writes: >> thanks for the patch. I’ve made a couple of cosmetic changes and >> removed the list of supported operating systems from the description. I >> also moved it from admin.scm to networking.scm where we already have a >> couple of network monitoring packages. > > Sure. > >> I

Re: [PATCH] Add bwm-ng.

2016-08-27 Thread Arun Isaac
> thanks for the patch. I’ve made a couple of cosmetic changes and > removed the list of supported operating systems from the description. I > also moved it from admin.scm to networking.scm where we already have a > couple of network monitoring packages. Sure. > I’d like to add a copyright lin

Re: [PATCH] Add bwm-ng.

2016-08-27 Thread Ricardo Wurmus
Hi Arun, > Please find attached patch to add new package 'bwm-ng' to (gnu packages > admin). thanks for the patch. I’ve made a couple of cosmetic changes and removed the list of supported operating systems from the description. I also moved it from admin.scm to networking.scm where we already

Re: [PATCH] Add bwm-ng.

2016-08-27 Thread Arun Isaac
> For a first(?) package this is very good. > Other than these changes, which the person commiting your patch can > apply on your behalf, it looks good to me! Yes, this is my first package. Thank you! :-) signature.asc Description: PGP signature

Re: [PATCH] Add bwm-ng.

2016-08-27 Thread ng0
Hi Arun, thanks for your contribution. Arun writes: > [ Unknown signature status ] > > Please find attached patch to add new package 'bwm-ng' to (gnu packages > admin). > > Regards, > Arun Isaac > > From 2a88508183f9b8005c2a0694992f1b3ae4208049 Mon Sep 17 00:00:00 2001 > From: Arun Isaac > Dat