Skip to contentSkip to conversation

autbuild-run: fix legal-info

PATCH New Yann E. MORIN 2 participants 2 messages Apr 6, 2015

lore
Share & tools
lore

Yann E. MORIN

author
#0 lore raw

PATCH 1/1 autbuild-run: fix legal-info

Currently, running legal-info is broken, as it only runs a plain 'make', which does not do much...

Fix that by explicitly pass 'legal-info' as the make target. Also, we need to specify the download dir, to avoid using the default one (which does not exist) or a user-defined one (in the environment).

Signed-off-by "Yann E. MORIN" <[email protected]>
Cc Thomas Petazzoni <[email protected]>
---
 scripts/autobuild-run | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
scripts/autobuild-run +4 -1
index 409570d..0e12080 100755
@@ -560,7 +560,10 @@ def do_build(**kwargs):
     if ret != 0:
         log_write(log, "INFO: build failed")
         return -1
-    ret = subprocess.call(["make", "O=%s" % outputdir, "-C", srcdir], stdout=f, stderr=f)
+    cmd = ["make", "O=%s" % outputdir, "-C", srcdir,
+            "BR2_DL_DIR=%s" % dldir, "legal-info"] \
+          + kwargs['make_opts'].split()
+    ret = subprocess.call(cmd, stdout=f, stderr=f)
     if ret != 0:
         log_write(log, "INFO: build failed during legal-info")
         return -1
Signature · 1 line
1.9.1
Applied
#1 lore raw

PATCH 1/1 autbuild-run: fix legal-info

Dear Yann E. MORIN,

Quoted 9 lines · Yann E. MORIN

Currently, running legal-info is broken, as it only runs a plain 'make', which does not do much...

Fix that by explicitly pass 'legal-info' as the make target. Also, we need to specify the download dir, to avoid using the default one (which does not exist) or a user-defined one (in the environment).

Signed-off-by "Yann E. MORIN" <[email protected]>
Cc Thomas Petazzoni <[email protected]>

Applied, thanks!

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com