From owner-cvs-all@FreeBSD.ORG Tue May 27 09:09:53 2008 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id D91E1106566C; Tue, 27 May 2008 09:09:53 +0000 (UTC) (envelope-from rwatson@FreeBSD.org) Received: from cyrus.watson.org (cyrus.watson.org [209.31.154.42]) by mx1.freebsd.org (Postfix) with ESMTP id 936478FC21; Tue, 27 May 2008 09:09:53 +0000 (UTC) (envelope-from rwatson@FreeBSD.org) Received: from fledge.watson.org (fledge.watson.org [209.31.154.41]) by cyrus.watson.org (Postfix) with ESMTP id 3569346B49; Tue, 27 May 2008 05:09:53 -0400 (EDT) Date: Tue, 27 May 2008 10:09:53 +0100 (BST) From: Robert Watson X-X-Sender: robert@fledge.watson.org To: "Bjoern A. Zeeb" In-Reply-To: <20080527084233.T65662@maildrop.int.zabbadoz.net> Message-ID: <20080527100846.M35026@fledge.watson.org> References: <20080527084233.T65662@maildrop.int.zabbadoz.net> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed Cc: cvs-src@freebsd.org, Niclas Zeising , src-committers@freebsd.org, cvs-all@freebsd.org Subject: Re: cvs commit: src/etc/mtree BSD.include.dist BSD.usr.dist src/include Makefile src/lib Makefile src/lib/libatm Makefile atm_addr.c cache_key.c ioctl_subr.c ip_addr.c ip_checksum.c libatm.h timer.c src/rescue/rescue Makefile src/sbin/atm Makefile .. 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: Tue, 27 May 2008 09:09:54 -0000 On Tue, 27 May 2008, Bjoern A. Zeeb wrote: > On Tue, 27 May 2008, Niclas Zeising wrote: > >>> Remove netatm from HEAD as it is not MPSAFE and relies on the now removed >>> NET_NEEDS_GIANT. netatm has been disconnected from the build for ten >>> months in HEAD/RELENG_7. Specifics: >>> >> [SNIP more commit details] >> >> Just a question. Shouldn't the WITHOUT_ATM option in src.conf(5) be removed >> as well? If that't the case, remove src/tools/build/options/WITHOUT_ATM and >> regen src.conf. > > Oh, I think we only have two more ATM implementations left in the tree (but > I could be wrong about the number;-) This was going to be my initial response also, but a glimpse of the source tree suggests that WITHOUT_ATM is not used anymore: rwatson@zoo:/zoo/rwatson/netperf/src% glimpse -H . WITHOUT_ATM | grep -v CVS | grep -v glimpse /zoo/rwatson/netperf/src/share/man/man5/src.conf.5: .It Va WITHOUT_ATM /zoo/rwatson/netperf/src/share/man/man5/src.conf.5: .\" from FreeBSD: src/tools/build/options/WITHOUT_ATM,v 1.1 2006/03/21 07:50:49 ru Exp /zoo/rwatson/netperf/src/tools/build/options/WITHOUT_ATM: .\" $FreeBSD: src/tools/build/options/WITHOUT_ATM,v 1.1 2006/03/21 07:50:49 ru Exp $ So either we should GC it, or we should refer to it elsewhere? :-) Robert N M Watson Computer Laboratory University of Cambridge