[Buildroot] [PATCH 3/3] support/script/scancpan: add option test

François Perrad francois.perrad at gadz.org
Fri Oct 3 12:57:21 UTC 2014


2014-10-01 21:34 GMT+02:00 Thomas Petazzoni
<thomas.petazzoni at free-electrons.com>:
> Dear Francois Perrad,
>
> On Wed,  1 Oct 2014 20:36:59 +0200, Francois Perrad wrote:
>> test modules are usually useless on BR
>>
>> Signed-off-by: Francois Perrad <francois.perrad at gadz.org>
>
> Sorry for asking only now, but can we have a real commit message that
> explains what's going on?
>
> Something like:
>
> """
> This commit adds a new -test option to the scancpan script, which
> allows to ...
> """
>
> With your current message "test modules are usually useless on BR", I
> don't understand what this option does: does it disables test modules?
> does it enables it? what does it change compared to the previous
> situation?
>
> Best regards,
>

find below an improved message for this patch

"""
Between Perl modules, there are 3 kinds of dependency:
 - configure/build time dependency which becomes host dependency
 - runtime dependency which becomes target dependency
 - test time dependency which is useless in a cross-compiling context like BR

Before this patch, test time dependencies are handled like runtime dependencies.

After this patch, test time dependencies are ignored by default.
The new option -test allows to add them anyway.
"""

François.


> Thomas
> --
> Thomas Petazzoni, CTO, Free Electrons
> Embedded Linux, Kernel and Android engineering
> http://free-electrons.com
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot


More information about the buildroot mailing list