Date: Fri, 31 Oct 2014 17:54:07 -0700 From: Garrett Cooper <yaneurabeya@gmail.com> To: =?iso-8859-1?Q?Dag-Erling_Sm=F8rgrav?= <des@des.no> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, Xin LI <delphij@FreeBSD.org> Subject: Re: svn commit: r273919 - head/etc/rc.d Message-ID: <0577B462-91E9-4E46-BF9E-085B50EDE09C@gmail.com> In-Reply-To: <86bnor9202.fsf@nine.des.no> References: <201410312220.s9VMKRaO096404@svn.freebsd.org> <86bnor9202.fsf@nine.des.no>
next in thread | previous in thread | raw e-mail | index | archive | help
--Apple-Mail=_DF1385D1-9A8E-43B3-A633-733B5E8017FC Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=iso-8859-1 On Oct 31, 2014, at 17:51, Dag-Erling Sm=F8rgrav <des@des.no> wrote: > Xin LI <delphij@FreeBSD.org> writes: >> Log: >> rc.d/geli should not depend on random, as the attach functionality >> do not require additional entropy to function. >>=20 >> It would create a circular dependency (not immediately obvious: >> geli provides 'disks' and requires 'random' as of r273872, >> 'random' requires 'FILESYSTEMS', 'FILESYSTEMS' requires 'root', >> 'root' requires 'swap', and finally 'swap' requires 'disk'). >=20 > My first instinct was: shouldn't rcorder warn about this? It turns = out > that it does, but in a very cryptic fashion: >=20 > des@nine ~head/etc/rc.d% svn up -qr273871 =20 > des@nine ~head/etc/rc.d% rcorder * >/dev/null > des@nine ~head/etc/rc.d% svn up -qr273872 =20 > des@nine ~head/etc/rc.d% rcorder * >/dev/null > rcorder: Circular dependency on file `zfs'. > rcorder: Circular dependency on provision `mountcritlocal' in file = `zfs'. > rcorder: Circular dependency on provision `mountcritlocal' in file = `var'. > rcorder: Circular dependency on provision `mountcritlocal' in file = `FILESYSTEMS'. > rcorder: Circular dependency on provision `root' in file = `FILESYSTEMS'. > des@nine ~head/etc/rc.d% svn up -qr273919 =20 > des@nine ~head/etc/rc.d% rcorder * >/dev/null > des@nine ~head/etc/rc.d%=20 rcorder errors are indeed cryptic/confusing. I can quickly add a = testcase for this if you like (basically check for all the scenarios = that need to be covered via /etc/rc). Does that sound good? Thanks! --Apple-Mail=_DF1385D1-9A8E-43B3-A633-733B5E8017FC Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename=signature.asc Content-Type: application/pgp-signature; name=signature.asc Content-Description: Message signed with OpenPGP using GPGMail -----BEGIN PGP SIGNATURE----- Comment: GPGTools - https://gpgtools.org iQEcBAEBCgAGBQJUVC8vAAoJEMZr5QU6S73eH4AH/0x96qlsz3AdSPeCxTjNU1cE 7xJPuln401K/ORTt28vnzyw8uPQ2ldvfzqmwuR5ijSl3o1lW/jMWMPbwCuSkFP2w xgviOSsqzdqhPifOb5VCoRP95kuCyq9+k9Cdrs9Q3jlC7ijBAla7cxYCGtRK2xQW qgVI4v/a+/P1Q+rHS2XC5fh9qBXUH/U7Ybq5CrAfPCh57BpUfWtArmYHyGubcDew m/c172ERAiG035IJeTdoBC/dX2NMCOwQiy3wxNkSq7OCMeLirv04GK5tOOZIHaaZ HXPBcyFQfAtB5ugILC4/rjIWznaTMZg3+n6WdJs6643fJ2qLCeSHlHlmhjohoZY= =AsJe -----END PGP SIGNATURE----- --Apple-Mail=_DF1385D1-9A8E-43B3-A633-733B5E8017FC--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?0577B462-91E9-4E46-BF9E-085B50EDE09C>