[cbootimage-configs PATCH] Image dependency generator cleanups

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Stephen Warren <swarren@xxxxxxxxxx>

1) Remove whitespace from parsed filenames.
2) Make image file depend on dependency generator script.
3) Provide dummy rules for all dependencies. Without this change, if any
   dependency is renamed, make will try to build it but won't know how.
   With the dummy rules in place, make will execute an empty command to
   build any missing dependencies, which is fine if they've simply been
   renamed. Any real errors due to incorrect filenames in the current
   config files will be caught by the cbootimage invocation itself.

Signed-off-by: Stephen Warren <swarren@xxxxxxxxxx>
---
 build/gen-image-deps.sh | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/build/gen-image-deps.sh b/build/gen-image-deps.sh
index 10c4001e0050..fcbb64cdbf5e 100755
--- a/build/gen-image-deps.sh
+++ b/build/gen-image-deps.sh
@@ -18,17 +18,25 @@
 #    misrepresented as being the original software.
 # 3. This notice may not be removed or altered from any source distribution.
 
+deps_script=$0
 img_cfg_file=$1
 img_file=$2
 dep_file=$3
 
 rm -f ${dep_file}
-bct=`grep -i bctfile ${img_cfg_file} | sed -e 's/^.*=//' -e s'/[,;].*$//'`
-bootloader=`grep -i bootloader ${img_cfg_file} | sed -e 's/^.*=//' -e s'/[,;].*$//'`
+bct=`grep -i bctfile ${img_cfg_file} | sed -e 's/^.*=\s*//' -e s'/[,;].*$//'`
+bootloader=`grep -i bootloader ${img_cfg_file} | sed -e 's/^.*=\s*//' -e s'/[,;].*$//'`
 
 cat > ${dep_file} <<ENDOFHERE
 ${img_file}: \\
+	${deps_script} \\
 	${img_cfg_file} \\
 	${bct} \\
 	${bootloader}
+
+${img_cfg_file}:
+
+${bct}:
+
+${bootloader}:
 ENDOFHERE
-- 
1.8.1.5

--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux