From owner-freebsd-current@FreeBSD.ORG Tue Nov 16 18:05:03 2010 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 43D49106566B; Tue, 16 Nov 2010 18:05:03 +0000 (UTC) (envelope-from yanegomi@gmail.com) Received: from mail-ew0-f54.google.com (mail-ew0-f54.google.com [209.85.215.54]) by mx1.freebsd.org (Postfix) with ESMTP id 42EF98FC24; Tue, 16 Nov 2010 18:05:01 +0000 (UTC) Received: by ewy3 with SMTP id 3so488360ewy.13 for ; Tue, 16 Nov 2010 10:05:00 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:received:sender:received :in-reply-to:references:date:x-google-sender-auth:message-id:subject :from:to:cc:content-type; bh=dYKXwu4MmTrYL/HeXhk6vnq9wIoCZ8f2jWNfH4VwGEQ=; b=hdM4h+SWGO+gu3aeT3cr56rSbq4qHfkbLBqCciZvvmj3AcW0wBp49+a3EQpVjbKHrz FJbA/D1HK2+wGgxI2fa7aLKlA+PqQfi4SK5P9Xkfmbx6viWokhE/pAo2t6kvm1d7Wt3k jPX0h42revKJ7P1JXpWWXXGkSCWrWeSE3e3qY= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:cc:content-type; b=BID756Oswrvhp5wuyKMimT1U2Vis7NBAZtn1FG5OTevfCwRdwKep43uIZ8SZVUGxgr zSeuMjyqWUPba33HB9T8DJ/8RzN4mnOT2XLJurupTc7WWH6c/dzjXvfB5GMLIHFIbNtz ZpAS6FmkiI43u/FMcLAiQzZVfXT3bomz0g8hk= MIME-Version: 1.0 Received: by 10.216.46.200 with SMTP id r50mr650331web.45.1289930698964; Tue, 16 Nov 2010 10:04:58 -0800 (PST) Sender: yanegomi@gmail.com Received: by 10.216.198.27 with HTTP; Tue, 16 Nov 2010 10:04:58 -0800 (PST) In-Reply-To: References: <201011160829.13511.jhb@freebsd.org> <4CE28AE4.70101@yandex.ru> <201011160912.27693.jhb@freebsd.org> Date: Tue, 16 Nov 2010 10:04:58 -0800 X-Google-Sender-Auth: vzdiBGHsrwYxIV4RYGtDyb46xuU Message-ID: From: Garrett Cooper To: John Baldwin Content-Type: multipart/mixed; boundary=0016364c7a13191b8204952f65dc Cc: Bruce Cran , freebsd-hackers@freebsd.org, Adrian Chadd , "Andrey V. Elsukov" , freebsd-current@freebsd.org Subject: Re: breaking the crunchgen logic into a share/mk file X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 16 Nov 2010 18:05:03 -0000 --0016364c7a13191b8204952f65dc Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable On Tue, Nov 16, 2010 at 9:52 AM, Garrett Cooper wrote= : > On Tue, Nov 16, 2010 at 6:12 AM, John Baldwin wrote: >> On Tuesday, November 16, 2010 8:45:08 am Andrey V. Elsukov wrote: >>> On 16.11.2010 16:29, John Baldwin wrote: >>> > Err, are there no longer hard links to all of the frontends for a giv= en >>> > crunch? =A0If so, that is a problem as it will make rescue much harde= r to use. >>> >>> Yes, probably this patch is not needed and it should be fixed somewhere= in >>> makefiles. But currently rescue does not have any hardlinks: >>> http://pub.allbsd.org/FreeBSD-snapshots/i386-i386/9.0-HEAD-20101116-JPS= NAP/cdrom/livefs/rescue/ >>> >>> And what is was before: >>> http://pub.allbsd.org/FreeBSD-snapshots/i386-i386/9.0-HEAD-20101112-JPS= NAP/cdrom/livefs/rescue/ >> >> That definitely needs to be fixed. > > =A0 =A0The .mk file wasn't being installed. Could someone please test > this patch for validity and commit it? > =A0 =A0More importantly, why didn't the above build process, or tinderbox > fail properly with this change? Ouch. I can definitely still see the other problem: # make install install -s -o root -g wheel -m 555 rescue /rescue install -o root -g wheel -m 555 nextboot_FIXED /rescue/nextboot install -o root -g wheel -m 555 dhclient_FIXED /rescue/dhclient-script The second attached patch (not a superset of the first) what unbreaks it: # make -C rescue/rescue/ install install -s -o root -g wheel -m 555 rescue /rescue install -o root -g wheel -m 555 nextboot_FIXED /rescue/nextboot install -o root -g wheel -m 555 dhclient_FIXED /rescue/dhclient-script /rescue/cat -> /rescue/rescue /rescue/chflags -> /rescue/rescue /rescue/chio -> /rescue/rescue /rescue/chmod -> /rescue/rescue /rescue/cp -> /rescue/rescue /rescue/date -> /rescue/rescue /rescue/dd -> /rescue/rescue /rescue/df -> /rescue/rescue /rescue/echo -> /rescue/rescue /rescue/ed -> /rescue/rescue /rescue/red -> /rescue/rescue /rescue/expr -> /rescue/rescue /rescue/getfacl -> /rescue/rescue /rescue/hostname -> /rescue/rescue /rescue/kenv -> /rescue/rescue /rescue/kill -> /rescue/rescue /rescue/ln -> /rescue/rescue /rescue/link -> /rescue/rescue /rescue/ls -> /rescue/rescue /rescue/mkdir -> /rescue/rescue /rescue/mv -> /rescue/rescue /rescue/pkill -> /rescue/rescue /rescue/pgrep -> /rescue/rescue /rescue/ps -> /rescue/rescue /rescue/pwd -> /rescue/rescue /rescue/realpath -> /rescue/rescue /rescue/rm -> /rescue/rescue /rescue/unlink -> /rescue/rescue /rescue/rmdir -> /rescue/rescue /rescue/setfacl -> /rescue/rescue /rescue/sh -> /rescue/rescue /rescue/stty -> /rescue/rescue /rescue/sync -> /rescue/rescue /rescue/test -> /rescue/rescue /rescue/[ -> /rescue/rescue /rescue/csh -> /rescue/rescue /rescue/tcsh -> /rescue/rescue /rescue/atacontrol -> /rescue/rescue /rescue/badsect -> /rescue/rescue /rescue/camcontrol -> /rescue/rescue /rescue/ccdconfig -> /rescue/rescue /rescue/clri -> /rescue/rescue /rescue/devfs -> /rescue/rescue /rescue/dmesg -> /rescue/rescue /rescue/dump -> /rescue/rescue /rescue/rdump -> /rescue/rescue /rescue/dumpfs -> /rescue/rescue /rescue/dumpon -> /rescue/rescue /rescue/fsck -> /rescue/rescue /rescue/fsck_ffs -> /rescue/rescue /rescue/fsck_4.2bsd -> /rescue/rescue /rescue/fsck_ufs -> /rescue/rescue /rescue/fsck_msdosfs -> /rescue/rescue /rescue/fsdb -> /rescue/rescue /rescue/fsirand -> /rescue/rescue /rescue/gbde -> /rescue/rescue /rescue/geom -> /rescue/rescue /rescue/glabel -> /rescue/rescue /rescue/gpart -> /rescue/rescue /rescue/ifconfig -> /rescue/rescue /rescue/init -> /rescue/rescue /rescue/kldconfig -> /rescue/rescue /rescue/kldload -> /rescue/rescue /rescue/kldstat -> /rescue/rescue /rescue/kldunload -> /rescue/rescue /rescue/ldconfig -> /rescue/rescue /rescue/md5 -> /rescue/rescue /rescue/mdconfig -> /rescue/rescue /rescue/mdmfs -> /rescue/rescue /rescue/mknod -> /rescue/rescue /rescue/mount -> /rescue/rescue /rescue/mount_cd9660 -> /rescue/rescue /rescue/mount_msdosfs -> /rescue/rescue /rescue/mount_nfs -> /rescue/rescue /rescue/mount_ntfs -> /rescue/rescue /rescue/mount_nullfs -> /rescue/rescue /rescue/mount_udf -> /rescue/rescue /rescue/mount_unionfs -> /rescue/rescue /rescue/newfs -> /rescue/rescue /rescue/newfs_msdos -> /rescue/rescue /rescue/nos-tun -> /rescue/rescue /rescue/ping -> /rescue/rescue /rescue/reboot -> /rescue/rescue /rescue/fastboot -> /rescue/rescue /rescue/halt -> /rescue/rescue /rescue/fasthalt -> /rescue/rescue /rescue/restore -> /rescue/rescue /rescue/rrestore -> /rescue/rescue /rescue/rcorder -> /rescue/rescue /rescue/route -> /rescue/rescue /rescue/routed -> /rescue/rescue /rescue/rtquery -> /rescue/rescue /rescue/rtsol -> /rescue/rescue /rescue/savecore -> /rescue/rescue /rescue/spppcontrol -> /rescue/rescue /rescue/swapon -> /rescue/rescue /rescue/sysctl -> /rescue/rescue /rescue/tunefs -> /rescue/rescue /rescue/umount -> /rescue/rescue /rescue/bsdlabel -> /rescue/rescue /rescue/disklabel -> /rescue/rescue /rescue/fdisk -> /rescue/rescue /rescue/dhclient -> /rescue/rescue /rescue/head -> /rescue/rescue /rescue/mt -> /rescue/rescue /rescue/sed -> /rescue/rescue /rescue/tail -> /rescue/rescue /rescue/tee -> /rescue/rescue /rescue/gzip -> /rescue/rescue /rescue/gunzip -> /rescue/rescue /rescue/gzcat -> /rescue/rescue /rescue/zcat -> /rescue/rescue /rescue/bzip2 -> /rescue/rescue /rescue/bunzip2 -> /rescue/rescue /rescue/bzcat -> /rescue/rescue /rescue/xz -> /rescue/rescue /rescue/unxz -> /rescue/rescue /rescue/lzma -> /rescue/rescue /rescue/unlzma -> /rescue/rescue /rescue/xzcat -> /rescue/rescue /rescue/lzcat -> /rescue/rescue /rescue/tar -> /rescue/rescue /rescue/vi -> /rescue/rescue /rescue/ex -> /rescue/rescue /rescue/id -> /rescue/rescue /rescue/groups -> /rescue/rescue /rescue/whoami -> /rescue/rescue /rescue/chroot -> /rescue/rescue /rescue/chown -> /rescue/rescue /rescue/chgrp -> /rescue/rescue So it looks like something tunable needs to be added to the .mk file to deal with hardlinks as this basically reenables functionality that Adrian disabled. Thanks, -Garrett --0016364c7a13191b8204952f65dc Content-Type: text/x-patch; charset=US-ASCII; name="fix-crunchgen-hardlinking.patch" Content-Disposition: attachment; filename="fix-crunchgen-hardlinking.patch" Content-Transfer-Encoding: base64 X-Attachment-Id: f_ggl3c4zt1 SW5kZXg6IHNoYXJlL21rL2JzZC5jcnVuY2hnZW4ubWsKPT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gc2hhcmUvbWsv YnNkLmNydW5jaGdlbi5tawkocmV2aXNpb24gMjE1MzMzKQorKysgc2hhcmUvbWsvYnNkLmNydW5j aGdlbi5tawkod29ya2luZyBjb3B5KQpAQCAtNTEsMTkgKzUxLDE2IEBACiAuZWxzZQogJChPVVRQ VVRTKTogJCguQ1VSRElSKS8uLi8uLi8kKEQpLyQoUCkvTWFrZWZpbGUKIC5lbmRpZgotIyBEaXNh YmxlIGJ1aWxkaW5nIGxpbmtzIGZvciBic2Rib3ggLSB3aGF0ZXZlciBpcyBpbnN0YWxsaW5nIHRo ZSBiaW5hcmllcyBpbnRvCi0jIHRoZSBlbWJlZGRlZCBzeXN0ZW0gc2hvdWxkIChmb3Igbm93KSBk byB0aGUgbGlua2luZyB0aGVyZS4gVGhpcyBtYXkgY2hhbmdlCi0jIGluIHRoZSBmdXR1cmUuIC1h ZHJpYW4KLSMuaWZuZGVmIENSVU5DSF9TVVBQUkVTU19MSU5LXyR7UH0KLSNMSU5LUys9ICQoQklO RElSKS8kKFBST0cpICQoQklORElSKS8kKFApCi0jLmVuZGlmCi0jLmZvciBBIGluICQoQ1JVTkNI X0FMSUFTXyQoUCkpCi0jLmlmbmRlZiBDUlVOQ0hfU1VQUFJFU1NfTElOS18ke0F9Ci0jTElOS1Mr PSAkKEJJTkRJUikvJChQUk9HKSAkKEJJTkRJUikvJChBKQotIy5lbmRpZgotIy5lbmRmb3IKKy5p Zm5kZWYgQ1JVTkNIX1NVUFBSRVNTX0xJTktfJHtQfQorTElOS1MrPSAkKEJJTkRJUikvJChQUk9H KSAkKEJJTkRJUikvJChQKQorLmVuZGlmCisuZm9yIEEgaW4gJChDUlVOQ0hfQUxJQVNfJChQKSkK Ky5pZm5kZWYgQ1JVTkNIX1NVUFBSRVNTX0xJTktfJHtBfQorTElOS1MrPSAkKEJJTkRJUikvJChQ Uk9HKSAkKEJJTkRJUikvJChBKQorLmVuZGlmCiAuZW5kZm9yCiAuZW5kZm9yCisuZW5kZm9yCiAK IGFsbDogJChQUk9HKQogZXhlOiAkKFBST0cpCg== --0016364c7a13191b8204952f65dc--