Re: [PATCH v2] busybox: Upgrade to upstream 1.20.2

[Phil Blundell <philb@gnu.org>]

On Mon, 2012-08-20 at 09:25 +0100, Paul Eggleton wrote:
> On Monday 20 August 2012 09:18:17 Phil Blundell wrote:
> > On Mon, 2012-08-20 at 10:46 +0300, Radu Moisan wrote:
> > > @@ -97,6 +97,8 @@ do_prepare_config () {
> > > 
> > >  		< ${WORKDIR}/defconfig > ${S}/.config
> > >  	
> > >  	sed -i -e '/CONFIG_STATIC/d' .config
> > >  	echo "# CONFIG_STATIC is not set" >> .config
> > > 
> > > +	sed -i -e '/CONFIG_LSOF/d' .config
> > > +	echo "# CONFIG_LSOF is not set" >> .config
> > > 
> > >  	for i in 'CROSS' 'DISTRO FEATURES'; do echo "### $i"; done >> \
> > >  	
> > >  		${S}/.config
> > >  	
> > >  	sed -i -e '${configmangle}' ${S}/.config
> > 
> > Why is this desirable?
> 
> Well, the more appropriate question would be: could we not just update the 
> defconfig and then make that change to it directly?
> 
> (In case it isn't clear - lsof is being disabled because it was not enabled in 
> the previous 1.19.4 version.)

Ah, I see.  In that case, yes, just updating the defconfig is the right
thing to do.  The patch above will, if I am understanding it correctly,
make it impossible (or at least quite hard) for any other layer to turn
CONFIG_LSOF back on.

p.