From owner-freebsd-x11@FreeBSD.ORG Wed Apr 18 10:50:07 2007 Return-Path: X-Original-To: freebsd-x11@freebsd.org Delivered-To: freebsd-x11@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 430E516A400; Wed, 18 Apr 2007 10:50:07 +0000 (UTC) (envelope-from dejan.lesjak@ijs.si) Received: from mail.ijs.si (mailman.ijs.si [193.2.4.66]) by mx1.freebsd.org (Postfix) with ESMTP id BF8B613C458; Wed, 18 Apr 2007 10:50:06 +0000 (UTC) (envelope-from dejan.lesjak@ijs.si) Received: from localhost (mail.ijs.si [193.2.4.66]) by mail.ijs.si (Postfix) with ESMTP id 84201192D9F; Wed, 18 Apr 2007 12:50:05 +0200 (CEST) DKIM-Signature: v=0.5; a=rsa-sha1; c=relaxed/simple; d=ijs.si; s=ja-200610; t=241372776; bh=CjHIQ55vwjcht5vDv/7lblgOqdg=; h=DomainKey-Signature: X-Virus-Scanned:Received:Received:Received:From:To:Subject:Date: User-Agent:Cc:References:In-Reply-To:MIME-Version:Content-Type: Content-Transfer-Encoding:Content-Disposition:Message-Id; b=UiLFaR Sy0+bcR3GM2XzOjp0YioT0jk+72GJDFQZO1OXyzkY2t3F36m+njivV+Iw3iK8PSbaq2 FeCLbyy9y9cRRxH6BW/UggYAAwq/C0mhJDWAeWMFuIUw04U9L2LJbJi DomainKey-Signature: a=rsa-sha1; s=ja-200610; d=ijs.si; c=nofws; q=dns; h=x-virus-scanned:received:from:to:subject:date:user-agent:cc: references:in-reply-to:mime-version:content-type: content-transfer-encoding:content-disposition:message-id; b=Mapf1Jizfr6PdCAXWqynoKDLaYNQ2wuPJS10CPthOilIXLEELCHmD/dFA6wlIn+YU uIj9SDrL+MhzPBZiN+LOmvhlYUeNcJri8EgkkyaHuFhCmwdzZBAZ2NWkYc1uRj0 X-Virus-Scanned: amavisd-new at ijs.si Received: from mail.ijs.si ([193.2.4.66]) by localhost (mail.ijs.si [127.0.0.1]) (amavisd-new, port 10026) with ESMTP id pomOaLEFxlu1; Wed, 18 Apr 2007 12:50:03 +0200 (CEST) Received: from edina.ijs.si (edina.ijs.si [193.2.4.3]) by mail.ijs.si (Postfix) with ESMTP id B0AD3192D69; Wed, 18 Apr 2007 12:50:02 +0200 (CEST) Received: from f9pc04.ijs.si (f9pc04.ijs.si [194.249.156.4]) by edina.ijs.si (Postfix) with ESMTP id 90954510DF; Wed, 18 Apr 2007 12:50:02 +0200 (CEST) From: Dejan Lesjak To: freebsd-x11@freebsd.org Date: Wed, 18 Apr 2007 12:49:59 +0200 User-Agent: KMail/1.9.6 References: <200704161141.l3GBfrcY049525@freefall.freebsd.org> <46253C95.3030808@FreeBSD.org> <20070417220046.GC44061@ice.42.org> In-Reply-To: <20070417220046.GC44061@ice.42.org> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Content-Disposition: inline Message-Id: <200704181250.01295.dejan.lesjak@ijs.si> Cc: Stefan `Sec` Zehl , Florent Thoumie Subject: Re: ports/109497: x11-servers/xorg-fontserver rc.d/xfs.sh script missing "-user" X-BeenThere: freebsd-x11@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: X11 on FreeBSD -- maintaining and support List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 18 Apr 2007 10:50:07 -0000 On Wednesday 18 of April 2007, Stefan `Sec` Zehl wrote: > On Tue, Apr 17, 2007 at 22:31 +0100, Florent Thoumie wrote: > > Stefan `Sec` Zehl wrote: > > > Hi, > > > > > > On Mon, Apr 16, 2007 at 11:41 +0000, Florent Thoumie wrote: > > >> Synopsis: x11-servers/xorg-fontserver rc.d/xfs.sh script missing > > >> "-user" > > >> > > >> State-Changed-From-To: open->closed > > >> State-Changed-By: flz > > >> State-Changed-When: Mon Apr 16 11:40:38 UTC 2007 > > >> State-Changed-Why: > > >> I just checked and other OS'es seem to run it as root as well. > > >> > > >> If this is a real concern to you, just set xfs_flags="-user nobody" in > > >> /etc/rc.conf. I think this is what you're looking for. > > > > > > Please note that if you set "xfs_user=" to something, the default > > > rc.subr will already try to do something with it, and (silently) fail > > > to start xfs at all. I do think fixing this would be more user friendly > > > -- besides, its only a two-line patch anyway, and it doesn't even > > > change the default of running as root. > > > > > > But if think it's important to refuse this change, I can certainly live > > > without that patch. > > > > This is not what I said, please re-read my message. > > Ok. I did. > > As far as I can tell, your message had two points. > > 1: Others run it as root. > 2: I can run it as non-root if I want to by using xfs_flags=... > > If that isn't what you said, please rephrase, as I must have > misunderstood you. Please be patient, as english is not my native > language. Therefore let me also rephrase my last answer. > > First regarding your two points: > > re 1: > - I'm not asking to change the default. So what other OSs run it as is > not relevant. > > re 2: > - If you still want to reject the patch I sent, I can live with it as > users searching for it will hopefully find the workaround documented > in this PR. > > My additional points I was trying to make: > > - If someone currently sets "xfs_user=" in rc.conf, this makes xfs fail > silently. I think it would be great if it instead would just work. It certainly should not fail silently if someone has unsupported option in rc.conf. If this is the case it should be fixed. But note that we are rather busy trying to get xorg 7.2 into ports so if you could try the script again after the upgrade and if it fails bring it up again then, that would be lovely. > - It is only two lines, so no bloat, and it won't hurt anyone. It's not so much the number of lines - it's rather that it seems this option is not that commonly used and that there already is a mechanism to do that with existing options (namely xfs_flags). > Hope that clears it up, Well, it rather confuses me :-) I didn't expect the thing to fail if you set some option that the script does not even look at, but yes, it does clear up your problem. Dejan