[Buildroot] [PATCH 4/4] Package netsnmp: Added OK/FAIL output

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Fri Jul 1 13:04:15 UTC 2016


Hello,

On Fri, 1 Jul 2016 14:36:44 +0200, Yann E. MORIN wrote:

> > @@ -92,6 +98,7 @@ case "$1" in
> >      reload|force-reload)
> >          reload
> >          ;;
> > +  
> 
> Spurious empty line.

It's indeed not strictly related, but in all other cases of this
switch, there is an empty line separating cases. So I think it's good
to also have this one for consistency. It could have been in a separate
commit, but oh well, that was good enough.

Thanks for the review!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com



More information about the buildroot mailing list