Revision: 6490
          http://ipcop.svn.sourceforge.net/ipcop/?rev=6490&view=rev
Author:   gespinasse
Date:     2012-03-18 11:14:38 +0000 (Sun, 18 Mar 2012)
Log Message:
-----------
Explain a little bit more where implicit rules come

While there, prefer some long lines and simplier than if else construction
Nothing should change

Modified Paths:
--------------
    ipcop/trunk/make.sh

Modified: ipcop/trunk/make.sh
===================================================================
--- ipcop/trunk/make.sh 2012-03-18 08:54:20 UTC (rev 6489)
+++ ipcop/trunk/make.sh 2012-03-18 11:14:38 UTC (rev 6490)
@@ -1398,11 +1398,11 @@
        # so we catch for sure error in PREPLOGFILE
        unset LOGFILE
 
-       # Warn if somehow an accident happen
+       # Warn if somehow an accident happen inside our own source tree
        [ -f ${BASEDIR}/lfs/${1} ] || exiterror "Script: ${BASEDIR}/lfs/${1} 
missing"
        [ -r ${BASEDIR}/lfs/${1} ] || exiterror "Script: ${BASEDIR}/lfs/${1} 
not readable"
-       # If the file is found empty, you should check if an implicit rules did 
not broke the lfs file.
-       # Look for make -p | grep SUFFIXES, if one the suffix added to lfs name 
did not match DL_FILE or another object
+       # If the file is found empty, you should check if an implicit make rule 
did not broke the lfs file.
+       # Look for make -p | grep SUFFIXES, if one of the suffix added to lfs 
name did not match DL_FILE or another object
        [ -s ${BASEDIR}/lfs/${1} ] || exiterror "Script: ${BASEDIR}/lfs/${1} 
empty"
        # If we replace by mistake lfs by config/rootfiles, avoid silently 
doing nothing
        cd ${BASEDIR}/lfs && (grep -q '^check' ${1} && grep -q '^download' 
${1}) || exiterror "Script: ${BASEDIR}/lfs/${1} doesn't look our Makefile"
@@ -1417,18 +1417,12 @@
                # make: Nothing to be done for `install'
                echo -e "`date -u '+%b %e %T'`: Building $*" >> ${PREPLOGFILE}
 
-               # -r as implicit rules are never used for download and md5, 
that should be faster, -s is less verbose for rebuild
-               cd ${BASEDIR}/lfs && make -srf $* MACHINE=${MACHINE} 
LFS_BASEDIR=${BASEDIR} MESSAGE="${1}\t " \
-                       download  >> ${PREPLOGFILE} 2>&1
-               if [ ${?} -ne 0 ]; then
-                       exiterror "Download error in ${1}"
-               fi
+               # -r as implicit make rules are never used for download and 
md5, that should be faster, -s is less verbose for rebuild
+               cd ${BASEDIR}/lfs && make -srf $* MACHINE=${MACHINE} 
LFS_BASEDIR=${BASEDIR} MESSAGE="${1}\t " download >> ${PREPLOGFILE} 2>&1
+               [ ${?} -ne 0 ] && exiterror "Download error in ${1}"
 
-               cd ${BASEDIR}/lfs && make -srf $* MACHINE=${MACHINE} 
LFS_BASEDIR=${BASEDIR} MESSAGE="${1}\t md5sum" \
-                       md5  >> ${PREPLOGFILE} 2>&1
-               if [ ${?} -ne 0 ]; then
-                       exiterror "md5sum error in ${1}, check file in cache or 
signature"
-               fi
+               cd ${BASEDIR}/lfs && make -srf $* MACHINE=${MACHINE} 
LFS_BASEDIR=${BASEDIR} MESSAGE="${1}\t md5sum" md5 >> ${PREPLOGFILE} 2>&1
+               [ ${?} -ne 0 ] && exiterror "md5sum error in ${1}, check file 
in cache or signature"
 
                # Split log per package
                
LOGFILE="${BASEDIR}/log_${MACHINE}/${STAGE_ORDER}_${STAGE}/${1}-${PKG_VER}"

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.


------------------------------------------------------------------------------
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
_______________________________________________
Ipcop-svn mailing list
Ipcop-svn@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ipcop-svn

Reply via email to