From owner-cvs-src@FreeBSD.ORG Tue May 27 23:59:07 2008 Return-Path: Delivered-To: cvs-src@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 29CEA1065674 for ; Tue, 27 May 2008 23:59:07 +0000 (UTC) (envelope-from erikt@midgard.homeip.net) Received: from ch-smtp01.sth.basefarm.net (ch-smtp01.sth.basefarm.net [80.76.149.212]) by mx1.freebsd.org (Postfix) with ESMTP id A9F538FC16 for ; Tue, 27 May 2008 23:59:06 +0000 (UTC) (envelope-from erikt@midgard.homeip.net) Received: from c83-253-25-183.bredband.comhem.se ([83.253.25.183]:60216 helo=falcon.midgard.homeip.net) by ch-smtp01.sth.basefarm.net with esmtp (Exim 4.68) (envelope-from ) id 1K18pP-00059q-3u for cvs-src@FreeBSD.org; Wed, 28 May 2008 01:43:40 +0200 Received: (qmail 30121 invoked from network); 28 May 2008 01:43:35 +0200 Received: from owl.midgard.homeip.net (10.1.5.7) by falcon.midgard.homeip.net with ESMTP; 28 May 2008 01:43:35 +0200 Received: (qmail 25085 invoked by uid 1001); 28 May 2008 01:43:35 +0200 Date: Wed, 28 May 2008 01:43:35 +0200 From: Erik Trulsson To: Maxim Sobolev Message-ID: <20080527234335.GA24995@owl.midgard.homeip.net> References: <200805272004.m4RK4SZt029194@repoman.freebsd.org> <483C7FF2.6000607@FreeBSD.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <483C7FF2.6000607@FreeBSD.org> User-Agent: Mutt/1.5.18 (2008-05-17) X-Originating-IP: 83.253.25.183 X-Scan-Result: No virus found in message 1K18pP-00059q-3u. X-Scan-Signature: ch-smtp01.sth.basefarm.net 1K18pP-00059q-3u 466cedb884eb9e523f96b700360adb9b Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, Xin LI , cvs-all@FreeBSD.org Subject: Re: cvs commit: src/include string.h src/lib/libc/string Makefile.inc memchr.3 memrchr.c src/sys/sys param.h X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 27 May 2008 23:59:07 -0000 On Tue, May 27, 2008 at 02:41:06PM -0700, Maxim Sobolev wrote: > Xin LI wrote: > > delphij 2008-05-27 20:04:27 UTC > > > > FreeBSD src repository > > > > Modified files: (Branch: RELENG_6) > > include string.h > > lib/libc/string Makefile.inc memchr.3 > > sys/sys param.h > > Added files: (Branch: RELENG_6) > > lib/libc/string memrchr.c > > Log: > > MFC: Add memrchr(3). > > I think this is not very good idea to MFC that into stable releases 6.x > and 7.x. The reason is that configure scripts for some packages might > detect up this API and enable it. Which means that some binary-only > packages build for say 6.4 won't work on 6.3 and down. AFAIK, both > forward and backward compatibility is required (or at least desired?) > for stable branches. No, it isn't. Backwards compatibility (in the sense that stuff that worked on 6.x should still work on 6.x+1) is largely required unless there is some very good reason to break compatibility (such very good reasons are quite rare.) Compatibility in the other direction (that everything that works on 6.x+1 will also work on 6.x), while obviously somewhat desirable, has AFAIK never been required and certainly not guaranteed. Such compatibility has almost certainly already been broken several times on all the -STABLE branches by now. > > While it's "nice-to-have" feature, I see no pressing need to MFC this > interface. Nor is there any pressing need not to MFC it. -- Erik Trulsson ertr1013@student.uu.se