From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from pigeon.gentoo.org ([208.92.234.80] helo=lists.gentoo.org) by finch.gentoo.org with esmtp (Exim 4.60) (envelope-from ) id 1Rr2d7-0001CD-GJ for garchives@archives.gentoo.org; Sat, 28 Jan 2012 07:23:21 +0000 Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id 089E7E0795; Sat, 28 Jan 2012 07:23:11 +0000 (UTC) Received: from smtp.gentoo.org (smtp.gentoo.org [140.211.166.183]) by pigeon.gentoo.org (Postfix) with ESMTP id C6273E0B2C for ; Sat, 28 Jan 2012 07:22:21 +0000 (UTC) Received: from [10.19.82.217] (193-64-22-30-nat.elisa-mobile.fi [193.64.22.30]) (using TLSv1 with cipher ECDHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) (Authenticated sender: ssuominen) by smtp.gentoo.org (Postfix) with ESMTPSA id DC7AA1B4002 for ; Sat, 28 Jan 2012 07:22:20 +0000 (UTC) Message-ID: <4F23A19C.2030302@gentoo.org> Date: Sat, 28 Jan 2012 09:19:56 +0200 From: Samuli Suominen User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:9.0) Gecko/20120114 Thunderbird/9.0 Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-dev@lists.gentoo.org Reply-to: gentoo-dev@lists.gentoo.org MIME-Version: 1.0 To: gentoo-dev@lists.gentoo.org Subject: Re: [gentoo-dev] useless set*id binaries References: <201201271914.45638.vapier@gentoo.org> <201201271941.13417.vapier@gentoo.org> <4F234A61.3020409@gentoo.org> <201201272049.16394.vapier@gentoo.org> <4F23543D.60601@gentoo.org> <20259.38310.170712.434418@a1i15.kph.uni-mainz.de> In-Reply-To: <20259.38310.170712.434418@a1i15.kph.uni-mainz.de> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Archives-Salt: a81352ba-be77-4065-a5bf-652cab79dde5 X-Archives-Hash: 70edc33ba9732bac9888230b9e178add On 01/28/2012 08:28 AM, Ulrich Mueller wrote: >>>>>> On Sat, 28 Jan 2012, Samuli Suominen wrote: > >> i've improved the situation _a bit_: > >> +*cdrtools-3.01_alpha06-r1 (28 Jan 2012) >> + >> + 28 Jan 2012; Samuli Suominen >> + +cdrtools-3.01_alpha06-r1.ebuild: >> + Change cdda2wav, cdrecord, readcd and rscsi from suid root to sgid >> disk for >> + udev users (note: tested with cdrecord -scanbus) > > This is definitely not an improvement and should be reverted. The suid > root is also needed to elevate cdrecord's scheduling priority. Missed that piece of code and reverted then. Any chance you could be more specific? > if has_version sys-fs/udev; then > fowners root:disk /usr/bin/{cdda2wav,cdrecord,readcd} /usr/sbin/rscsi > fperms u-s,g+s /usr/bin/{cdda2wav,cdrecord,readcd} /usr/sbin/rscsi > fi > > Automagic dependency on udev in src_install? Oh my. I don't consider this as a automagic to be worried about at all. Was bouncing back and forth with 'use kernel_linux' or 'has_version sys-fs/udev', since wasn't sure how other devmanagers have permissions set. But I guess this is now irrelevant since it's reverted.