[Buildroot] [PATCHv3] support/tests: allow properly indented config fragment

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Wed Jul 19 12:13:42 UTC 2017


Hello,

On Wed, 19 Jul 2017 14:00:40 +0200, Arnout Vandecappelle wrote:
> On 18-07-17 22:38, Yann E. MORIN wrote:
> > Currently, defining a config fragment in the runtime test infra requires
> > that the fragment not to be indented. This is beark, and causes grievance
> > when looking at the code (e.g. to fix it).
> > 
> > Just strip out all leading spaces/tabs when writing the configuration
> > lines into the config file, allowing in-line indented config fragments,
> > like so:
> > 
> >     class TestFoo(bla):
> >         config = bla.config + \
> >             """
> >             FOO=y
> >             # BAR is not set
> >             """
> > 
> > Signed-off-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>
> > Cc: Thomas Petazzoni <thomas.petazzoni at free-electrons.com>
> > Cc: Arnout Vandecappelle <arnout at mind.be>  
> 
> Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout at mind.be>

I already applied yesterday evening:
https://git.buildroot.org/buildroot/commit/?id=cf3cd4388a652c9af27ef1c35622e2d0a55b99a9.

Best regards,

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


More information about the buildroot mailing list