misc/157185: [patch] r221850 break nanobsd.sh

[Henrik Brix Andersen]

>Number:         157185
>Category:       misc
>Synopsis:       [patch] r221850 break nanobsd.sh
>Confidential:   no
>Severity:       critical
>Priority:       medium
>Responsible:    freebsd-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Thu May 19 18:10:10 UTC 2011
>Closed-Date:
>Last-Modified:
>Originator:     Henrik Brix Andersen
>Release:        FreeBSD 9.0-CURRENT i386
>Organization:
>Environment:
System: FreeBSD drpepper.brixandersen.dk 9.0-CURRENT FreeBSD 9.0-CURRENT #3 r221961: Sun May 15 17:11:34 CEST 2011 root at drpepper.brixandersen.dk:/usr/obj/usr/home/brix/projects/freebsd/src/head/sys/GENERIC i386


	
>Description:

The change to populate_slice() in nanobsd.sh committed in r221850 break nanobsd.sh.

CPIO will now try to follow all symbolic links in _.w/ - e.g. _.w/sys, which points to a non-existing _.w/usr/src/sys - and fail with "cpio: Can't stat ./sys

The patch in referenced the PR referenced by commit r221850 (misc/151697) does not contain this change. It only adds -L to the copying of packages.

	
>How-To-Repeat:
	
>Fix:

	
The followin patch addresses this problem:

--- nanobsd.sh.diff begins here ---
Index: tools/tools/nanobsd/nanobsd.sh
===================================================================
--- tools/tools/nanobsd/nanobsd.sh	(revision 221878)
+++ tools/tools/nanobsd/nanobsd.sh	(working copy)
@@ -418,7 +418,7 @@
 	echo "Creating ${dev} with ${dir} (mounting on ${mnt})"
 	newfs_part $dev $mnt $lbl
 	cd ${dir}
-	find . -print | grep -Ev '/(CVS|\.svn)' | cpio -Ldumpv ${mnt}
+	find . -print | grep -Ev '/(CVS|\.svn)' | cpio -dumpv ${mnt}
 	df -i ${mnt}
 	umount ${mnt}
 )
--- nanobsd.sh.diff ends here ---


>Release-Note:
>Audit-Trail:
>Unformatted: