From owner-svn-src-all@freebsd.org Sun Feb 28 12:29:44 2016 Return-Path: Delivered-To: svn-src-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id DE8DEAA708F; Sun, 28 Feb 2016 12:29:44 +0000 (UTC) (envelope-from brooks@spindle.one-eyed-alien.net) Received: from spindle.one-eyed-alien.net (spindle.one-eyed-alien.net [199.48.129.229]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id BD9001B82; Sun, 28 Feb 2016 12:29:44 +0000 (UTC) (envelope-from brooks@spindle.one-eyed-alien.net) Received: by spindle.one-eyed-alien.net (Postfix, from userid 3001) id BB8845A9F0E; Sun, 28 Feb 2016 12:20:09 +0000 (UTC) Date: Sun, 28 Feb 2016 12:20:09 +0000 From: Brooks Davis To: Slawa Olhovchenkov Cc: Bryan Drewery , src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-9@freebsd.org Subject: Re: svn commit: r296146 - stable/9/sys/dev/filemon Message-ID: <20160228122008.GA27425@spindle.one-eyed-alien.net> References: <201602272057.u1RKvYfF035338@repo.freebsd.org> <20160228102728.GA94639@zxy.spb.ru> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="YZ5djTAD1cGYuMQK" Content-Disposition: inline In-Reply-To: <20160228102728.GA94639@zxy.spb.ru> User-Agent: Mutt/1.5.24 (2015-08-30) X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 28 Feb 2016 12:29:45 -0000 --YZ5djTAD1cGYuMQK Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sun, Feb 28, 2016 at 01:27:28PM +0300, Slawa Olhovchenkov wrote: > On Sat, Feb 27, 2016 at 08:57:34PM +0000, Bryan Drewery wrote: >=20 > > Author: bdrewery > > Date: Sat Feb 27 20:57:34 2016 > > New Revision: 296146 > > URL: https://svnweb.freebsd.org/changeset/base/296146 > >=20 > > Log: > > MFC r242088: > > =20 > > Desupport pre-FreeBSD 7.1. > >=20 > > Modified: > > stable/9/sys/dev/filemon/filemon.c > > Directory Properties: > > stable/9/sys/ (props changed) > > stable/9/sys/dev/ (props changed) > > -#if __FreeBSD_version < 701000 >=20 > Not specialy to this commit, just for information: what purpose to > have in src tree, already divided to branches, pieces of code under > #if? >=20 > Of couse, question not about external maintained code. Filemon started as externally maintained code and initially it was a bit ambiguous which direction changes would flow. Sometimes it's also useful to allow code in HEAD to just work when copied to stable so people can see if their problem is fixed without having to merge too much. In general though, we should remove code like this in things maintained entirely in tree. -- Brooks --YZ5djTAD1cGYuMQK Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQEcBAEBAgAGBQJW0uX3AAoJEKzQXbSebgfAlNIIAKCcVI5XPitD/GJoiddzvm2A Kcg7SH0KBA6TkUwMxGqeaW+zzgSgsPqXD1AmNhvpaSexG9Ue5VqsOJoxakajEvAt j5ZE7ypUAq6neNb4gXoNHEmyIH/1fKLsC87QFhkCP49AAi+gga9ywcD0g9df9N+m m9ZOpAJwLRe/NBI1AnKMeNylvCL9jp9u1Cgi7oXOc2SjquemkR90X+KszhH5E/j+ LFRsPdMdoFZ7pg7stQ8QYNHY1rGZ+LsNZb342kK77UgsNYyevi/SEgGdwfVD5zN5 6iuoW1zXrUGppRkey9Z0OPVaEgPVxQimjtQi83ESMYlmA/IFQBx0TO3Gg01+I40= =mXzP -----END PGP SIGNATURE----- --YZ5djTAD1cGYuMQK--