[Buildroot] [PATCH] iftop: new package
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Mon Jan 21 17:48:53 UTC 2013
Dear Bogdan Radulescu,
On Mon, 21 Jan 2013 19:24:48 +0000, Bogdan Radulescu wrote:
> I adjusted my patch and yes, the correct way is very nice :)
> I was at the mountains and I didn't have my computer to send the
> adjustment earlier. Thanks for you advice!
You shouldn't put such a message here in the commit log: it will be
preserved forever in the Git history. If you want to send such
informations, either use an introduction e-mail (separate from the
e-mail carrying the patch itself), or for a single patch, put your
"personal" comments...
>
>
> Signed-off-by: Bogdan Radulescu <bogdan at nimblex.net>
> ---
... here. I.e, after the --- line.
> +config BR2_PACKAGE_IFTOP
> + bool "iftop"
> + select BR2_PACKAGE_NCURSES
> + select BR2_PACKAGE_LIBPCAP
> + select BR2_INET_IPV6
No. Please look at how other packages do it for BR2_INET_IPV6. You're
not allowed to select it. You should depend on it, and show a comment
when BR2_INET_IPV6 is not available.
> +
> + help
No empty line between the bool/select/depends on line and the help line.
Thanks!
Thomas
--
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
More information about the buildroot
mailing list