From owner-cvs-all@FreeBSD.ORG Tue May 29 07:51:33 2007 Return-Path: X-Original-To: cvs-all@FreeBSD.org Delivered-To: cvs-all@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 87E4216A4AC; Tue, 29 May 2007 07:51:33 +0000 (UTC) (envelope-from brian@Awfulhak.org) Received: from storm.uk.FreeBSD.org (storm.uk.FreeBSD.org [194.242.157.42]) by mx1.freebsd.org (Postfix) with ESMTP id 3927613C483; Tue, 29 May 2007 07:51:33 +0000 (UTC) (envelope-from brian@Awfulhak.org) Received: from store.lan.Awfulhak.org (store.lan.Awfulhak.org [172.16.0.35]) by storm.uk.FreeBSD.org (8.14.1/8.14.1) with ESMTP id l4T7pUoU049427; Tue, 29 May 2007 08:51:30 +0100 (BST) (envelope-from brian@Awfulhak.org) Received: from store.lan.Awfulhak.org (localhost.localdomain [127.0.0.1]) by localhost (Postfix) with SMTP id AB63E1957C6C; Tue, 29 May 2007 07:51:30 +0000 (GMT) Received: from gw.Awfulhak.org (gw.lan.Awfulhak.org [172.16.0.1]) by store.lan.Awfulhak.org (Postfix) with ESMTP id 71DE31957C61; Tue, 29 May 2007 07:51:22 +0000 (GMT) Received: from dev.lan.Awfulhak.org (brian@dev.lan.Awfulhak.org [172.16.0.5]) by gw.Awfulhak.org (8.14.1/8.14.1) with ESMTP id l4T7pIRV006808; Tue, 29 May 2007 00:51:19 -0700 (PDT) (envelope-from brian@Awfulhak.org) Date: Tue, 29 May 2007 00:51:18 -0700 From: Brian Somers To: Roman Bogorodskiy Message-ID: <20070529005118.63a62eb4@dev.lan.Awfulhak.org> In-Reply-To: <20070528045306.GA987@underworld.novel.ru> References: <200705251345.l4PDjoEW059652@repoman.freebsd.org> <20070527233522.E13311@maildrop.int.zabbadoz.net> <20070528045306.GA987@underworld.novel.ru> X-Mailer: Claws Mail 2.9.1 (GTK+ 2.10.12; i386-portbld-freebsd7.0) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-1.4 required=5.0 tests=ALL_TRUSTED autolearn=failed version=3.1.8 X-Spam-Checker-Version: SpamAssassin 3.1.8 (2007-02-13) on gw.lan.Awfulhak.org Cc: cvs-src@FreeBSD.org, "Bjoern A. Zeeb" , cvs-all@FreeBSD.org, src-committers@FreeBSD.org Subject: Re: cvs commit: src/usr.sbin/ppp command.c ppp.8.m4 radius.c radius.h 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, 29 May 2007 07:51:33 -0000 On Mon, 28 May 2007 08:53:06 +0400 Roman Bogorodskiy wrote: > Bjoern A. Zeeb wrote: > > > On Fri, 25 May 2007, Roman Bogorodskiy wrote: > > > > >novel 2007-05-25 13:45:49 UTC > > > > > > FreeBSD src repository (ports committer) > > > > > > Modified files: > > > usr.sbin/ppp command.c ppp.8.m4 radius.c radius.h > > > Log: > > > Add a new option for ppp.conf: rad_port_id. It allows to > > > change the way of what ppp submits to the RADIUS server > > > as NAS-Port-Id. Possible options are: the PID of the process > > > owning the corresponding interface, tun(4) interface number, > > > interface index (as it would get returned by if_nametoindex(3)), > > > or it's possible to keep the default behavior. Check the ppp(8) > > > manual page for details. > > > > > > PR: bin/112764 > > > Submitted by: novel (myself) > > > Reviewed by: flz > > > Approved by: flz > > > MFC after: 1 month > > > > > > Revision Changes Path > > > 1.307 +27 -1 src/usr.sbin/ppp/command.c > > > > @@ -144,6 +144,7 @@ > > #define VAR_IPV6CPRETRY 37 > > #define VAR_RAD_ALIVE 38 > > #define VAR_PPPOE 39 > > +#define VAR_PORT_ID 40 > > > > Was there some reason the VAR_ and the NEG_ defines were in > > different ranges? If so there is a conflict now and maybe we > > should move the NEG_ defines to 128+? > > I cannot see any conflicts, however it doesn't mean it could not exist. > Anyway, it seems to be a good idea to move NEG_ to 128+ to avoid > possible confusing and have a room for other 'set' we would want to add > in the future. > > A simple patch attached. > > > > 1.326 +20 -1 src/usr.sbin/ppp/ppp.8.m4 > > > 1.54 +27 -10 src/usr.sbin/ppp/radius.c > > > 1.22 +6 -0 src/usr.sbin/ppp/radius.h > > > > > > > -- > > Bjoern A. Zeeb bzeeb at Zabbadoz dot NeT > > _______________________________________________ > > cvs-all@freebsd.org mailing list > > http://lists.freebsd.org/mailman/listinfo/cvs-all > > To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org" > > Roman Bogorodskiy The only reason these "look" like the same namespace was to debug stuff when I first developed it. The namespace is different for each function that the values are passed to, so VAR_* is only used by SetVariable() and NEG_* is only used by NegotiateSet(). These defines should probably be enums with no value assignments. -- Brian Somers Don't _EVER_ lose your sense of humour !