Skip to contentSkip to conversation

echo to stderr in mg-debian-installer-update

PATCH New Dario Faggioli 1 participant 1 message Jun 13, 2014

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 echo to stderr in mg-debian-installer-update

Otherwise the script would 'return' an array of strings which results, when called from standalone-reset, in something like this:

+ ln -s http://ftp.debian.org/debian//dists/wheezy/main/installer-amd64/current/images/netboot//debian-installer/amd64/initrd.gz '=>' initrd.gz.new http://ftp.debian.org/debian//dists/wheezy/main/installer-amd64/current/images/netboot//debian-installer/amd64/linux '=>' linux.new http://ftp.debian.org/debian//dists/wheezy/main/installer-amd64/current/images/netboot//xen/vmlinuz '=>' vmlinuz-xen.new http://ftp.debian.org/debian//dists/wheezy/main/installer-amd64/current/images/netboot//xen/initrd.gz '=>' initrd.gz-xen.new s=debian-installer/amd64/initrd.gz d=initrd.gz s=debian-installer/amd64/linux d=linux s=xen/vmlinuz d=vmlinuz-xen s=xen/initrd.gz d=initrd.gz-xen 2014-06-13-wheezy /usr/groups/netboot//dariof/osstest/debian-installer/amd64/current-wheezy
ln: target `/usr/groups/netboot//dariof/osstest/debian-installer/amd64/current-wheezy' is not a directory

This was introduced in commit 20e98de815870b879157bb79559a70a0c058259a (mg-debian-installer-update: grab Xen PV domU capable images too).

---

mg-debian-installer-update +1 -1
index d6b688e..b964f95 100755
@@ -62,7 +62,7 @@ cd $dst
 for f in $files; do
         echo >&2 "fetching $f"
         s=${f/:*} ; d=${f/*:}
-        echo "$src/$s => $d.new"
+        echo >&2 "$src/$s => $d.new"
         curl -s $src/$s >$d.new
 done
 
Signature · 4 lines
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)