[PATCH 08/20] - quoted log messages properly

[Steffen Pankratz]

On Thu, 16 Jul 2015 19:27:49 +0200
Tim Niemeyer <tim.niemeyer at mastersword.de> wrote:

Hi Tim
> Wenn du das Escapezeichen noch fixt:
> Reviewed-by: Tim Niemeyer <tim.niemeyer at mastersword.de>
> 
> Kann ich sonst auch gern beim applyen fixen.

Wenn du es nicht vergisst, kannst du es gerne machen :)

> > ---
> >  buildscript | 18 +++++++++---------
> >  1 file changed, 9 insertions(+), 9 deletions(-)
> > 
> > diff --git a/buildscript b/buildscript
> > index 6c79269..4ba47b0 100755
> > --- a/buildscript
> > +++ b/buildscript
> > @@ -39,23 +39,23 @@ checkout_git(){
> >  	local COMMITID=$3
> >  
> >  	local MYGIT="git -C $DIRECTORY"
> > -	echo checking out $REPO_URL to $DIRECTORY in version $COMMITID
> > +	echo "checking out $REPO_URL to $DIRECTORY in version $COMMITID"
> >  	if [ -d $DIRECTORY ]; then
> >  		if $MYGIT remote -v | grep $REPO_URL ; then
> > -			echo Right remote detected
> > +			echo "Right remote detected"
> >  			if ! $MYGIT checkout $COMMITID ; then
> > -				echo commitid not found trying to fetch new commits
> > +				echo "commitid not found trying to fetch new commits"
> >  				$MYGIT	pull && $MYGIT checkout $COMMITID
> >  			fi
> >  		else
> > -			echo wrong remote or not an git repo at all -\> deleting whole directory
> > +			echo "wrong remote or not an git repo at all -\> deleting whole directory"
> Hier müsste das Escapezeichen jetzt weg.
> 
> >  			/bin/rm -rf $DIRECTORY
> >  			#needs to be without -C!!!
> >  			git clone $REPO_URL $DIRECTORY
> >  			$MYGIT checkout $COMMITID
> >  		fi
> >  	else
> > -		echo We need to do a fresh checkout
> > +		echo "We need to do a fresh checkout"
> >  		#needs to be without -C!!!
> >  		git clone $REPO_URL $DIRECTORY
> >  		$MYGIT checkout $COMMITID
> > @@ -85,7 +85,7 @@ get_source() {
> >  		# Patches for feeds could be stored in known directories like build_patches/$NAME/
> >  		# That way multiple patches for one feed could be supported
> >  		if [ ! -z "$PATCH" ] ; then
> > -			echo Patching $PATCH
> > +			echo "Patching $PATCH"
> >  			git -C $NAME am --whitespace=nowarn $PATCH
> >  		fi
> >  	done
> > @@ -109,11 +109,11 @@ prepare() {
> >  	for FEEDVAR in ${FEEDS[@]} ; do
> >  		FEED=$FEEDVAR[@] FEED=( ${!FEED} )
> >  		local NAME=${FEED[0]}
> > -		echo adding $NAME to package feeds
> > +		echo "adding $NAME to package feeds"
> >  		echo src-link $NAME $PACKAGEBASE/$NAME >> $target/feeds.conf
> >  	done
> >  
> > -	echo cleaning feeds
> > +	echo "cleaning feeds"
> >  	/bin/rm -rf $target/feeds
> >  
> >  	$target/scripts/feeds update
> > @@ -124,7 +124,7 @@ prepare() {
> >  		local PACKAGESVAR=${FEEDVAR}_PKGS
> >  
> >  		if [[ -n "${!PACKAGESVAR}" ]] ; then
> > -			echo adding ${!PACKAGESVAR} from feed $NAME to available packages
> > +			echo "adding ${!PACKAGESVAR} from feed $NAME to available packages"
> Das hat jetzt hier mit diesem Patch nichts zu tun, aber es fällt einfach
> auf. Ui, wtf.. Wie ist das denn da rein gekommen. Da werd ich n Patch
> schicken.

Das habe ich in Patch 17 behoben, wenn wir Beide das Selbe meinen.


Gruss
-Steffen

-- 
Hermes powered by Manjaro Linux (Linux 4.1.2)

Best regards, Steffen Pankratz.
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : nicht verfügbar
Dateityp    : application/pgp-signature
Dateigröße  : 181 bytes
Beschreibung: OpenPGP digital signature
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20150716/152c45e1/attachment-0002.sig>