From owner-cvs-all@FreeBSD.ORG Mon Feb 6 16:26:28 2006 Return-Path: X-Original-To: cvs-all@FreeBSD.org Delivered-To: cvs-all@FreeBSD.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 90E2E16A420; Mon, 6 Feb 2006 16:26:28 +0000 (GMT) (envelope-from ceri@submonkey.net) Received: from shrike.submonkey.net (cpc2-cdif2-3-1-cust208.cdif.cable.ntl.com [82.31.78.208]) by mx1.FreeBSD.org (Postfix) with ESMTP id 13D7D43D49; Mon, 6 Feb 2006 16:26:27 +0000 (GMT) (envelope-from ceri@submonkey.net) Received: from ceri by shrike.submonkey.net with local (Exim 4.60 (FreeBSD)) (envelope-from ) id 1F69C7-000HKv-0i; Mon, 06 Feb 2006 16:26:27 +0000 Date: Mon, 6 Feb 2006 16:26:26 +0000 From: Ceri Davies To: Xin LI Message-ID: <20060206162626.GJ21499@submonkey.net> Mail-Followup-To: Ceri Davies , Xin LI , src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org References: <200602060649.k166nvJL015878@repoman.freebsd.org> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="7ZAtKRhVyVSsbBD2" Content-Disposition: inline In-Reply-To: <200602060649.k166nvJL015878@repoman.freebsd.org> X-PGP: finger ceri@FreeBSD.org User-Agent: Mutt/1.5.11 Sender: Ceri Davies Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/usr.sbin/sysinstall tcpip.c X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 06 Feb 2006 16:26:28 -0000 --7ZAtKRhVyVSsbBD2 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Feb 06, 2006 at 06:49:57AM +0000, Xin LI wrote: > delphij 2006-02-06 06:49:57 UTC >=20 > FreeBSD src repository >=20 > Modified files: > usr.sbin/sysinstall tcpip.c=20 > Log: > Attempt to fix DHCP address acquisition which was broken by rev. 1.133. > | @@ -596,9 +596,12 @@ netconfig: > | ipv4_enable =3D TRUE; > | if (ipv4_enable) { > | sprintf(ifn, "%s%s", VAR_IFCONFIG, devp->name); > | - if (use_dhcp) > | - sprintf(temp, "DHCP %s", extras); > | - else > | + if (use_dhcp) { > | + if (strlen(extras) > 0) > | + sprintf(temp, "DHCP %s", extras); > | + else > | + sprintf(temp, "DHCP"); > | + } else > | sprintf(temp, "inet %s %s netmask %s", > | ipaddr, extras, netmask); > | variable_set2(ifn, temp, 1); Can you explain the failure mode? network.subr doesn't deal with trailing spaces or something? Ceri --=20 That must be wonderful! I don't understand it at all. -- Moliere --7ZAtKRhVyVSsbBD2 Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.2 (FreeBSD) iD8DBQFD53iyocfcwTS3JF8RAmWmAJ0S/LRBd1tK74WKKpbZAEWrGumFSgCeP7a7 GVT4n4CKJKeAsfwEiROj7HM= =UG1n -----END PGP SIGNATURE----- --7ZAtKRhVyVSsbBD2--