diff mbox series

utils/test-pkg: always run a global legal-info

Message ID 20180107225005.29898-1-yann.morin.1998@free.fr
State Accepted
Headers show
Series utils/test-pkg: always run a global legal-info | expand

Commit Message

Yann E. MORIN Jan. 7, 2018, 10:50 p.m. UTC
Instead of limiting it to the package under test, we run it globally.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Matthew Weber <matthew.weber@rockwellcollins.com>
---
 utils/test-pkg | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

Comments

Thomas Petazzoni Jan. 8, 2018, 8:17 a.m. UTC | #1
Hello,

On Sun,  7 Jan 2018 23:50:05 +0100, Yann E. MORIN wrote:
> Instead of limiting it to the package under test, we run it globally.
> 
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> Cc: Matthew Weber <matthew.weber@rockwellcollins.com>
> ---
>  utils/test-pkg | 6 ++----
>  1 file changed, 2 insertions(+), 4 deletions(-)

Applied to master, thanks.

Thomas
diff mbox series

Patch

diff --git a/utils/test-pkg b/utils/test-pkg
index 1b7046eac4..cd7b9dad7b 100755
--- a/utils/test-pkg
+++ b/utils/test-pkg
@@ -127,10 +127,8 @@  build_one() {
 
     # legal-info done systematically, because some packages have different
     # sources depending on the configuration (e.g. lua-5.2 vs. lua-5.3)
-    if [ -n "${pkg}" ]; then
-        if ! make O="${dir}" "${pkg}-legal-info" >> "${dir}/logfile" 2>&1; then
-            return 3
-        fi
+    if ! make O="${dir}" legal-info >> "${dir}/logfile" 2>&1; then
+        return 3
     fi
 }