[Buildroot] [PATCH] fs/ubifs: factorize all the "depends on" into "if...endif" blocks
Mark Jackson
mpfj-list at newflow.co.uk
Thu Apr 25 07:49:52 UTC 2013
On 24/04/13 22:26, Yann E. MORIN wrote:
> Mark, All,
>
> On Wed, Apr 24, 2013 at 01:30:25PM +0100, Mark Jackson wrote:
>> All the UBIFS options use "depends on BR2_TARGET_ROOTFS_UBIFS" but
>> we can simplify the config file by enclosing them in an "if..endif"
>> block.
>>
>> We also do the same for the "depends on BR2_TARGET_ROOTFS_UBI".
>
> A few comments below (mostly nitpicking, but hey! ;-) )
>
>> Signed-off-by: Mark Jackson <mpfj at newflow.co.uk>
>> ---
>> fs/ubifs/Config.in | 18 ++++++++----------
>> 1 file changed, 8 insertions(+), 10 deletions(-)
>>
>> diff --git a/fs/ubifs/Config.in b/fs/ubifs/Config.in
>> index f17c7dc..c1d2ce7 100644
>> --- a/fs/ubifs/Config.in
>> +++ b/fs/ubifs/Config.in
>> @@ -3,21 +3,20 @@ config BR2_TARGET_ROOTFS_UBIFS
>> help
>> Build a ubifs root filesystem
>>
>> +if BR2_TARGET_ROOTFS_UBIFS
> [--SNIP--]
>> +if BR2_TARGET_ROOTFS_UBI
> [--SNIP--]
>> +endif
>> +
>> +endif
>
> Comment the endif-s so we know at a glance to which if they belong to:
>
> +endif # BR2_TARGET_ROOTFS_UBI
> +
> +endif # BR2_TARGET_ROOTFS_UBIFS
I *very* nearly put those in anyway, but checked with a few other Config.in files
and none of them had such closing comments.
> With that fixed:
> Acked-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>
I'll post a v2.
Cheers
Mark J.
More information about the buildroot
mailing list