[Buildroot] [PATCH 2/2] lsof: add busybox tweaks

Jean-Christophe PLAGNIOL-VILLARD plagnioj at jcrosoft.com
Fri Apr 27 00:55:58 UTC 2012


On 16:45 Thu 26 Apr     , Thomas Petazzoni wrote:
> Le Thu, 26 Apr 2012 16:08:40 +0200,
> Jean-Christophe PLAGNIOL-VILLARD <plagnioj at jcrosoft.com> a écrit :
> 
> > > +if BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
> > >  source "package/lsof/Config.in"
> > put a depends on BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
> > 
> > in lsof/Config.in
> 
> No, that's not what we do for all other packages. Gustavo's patch is
> correct with regard to the current guidelines.
> 
> Now, whether we should change those guidelines to something else is a
> different story. But if we change the guideline, it should be changed
> for all packages and not just for lsof.
package/kmod/Config.in is already like this

and it's more clean

Best Regards,
J.


More information about the buildroot mailing list