From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by mx1.pokylinux.org (Postfix) with ESMTP id 336FC4C811FB for ; Thu, 27 Jan 2011 18:34:26 -0600 (CST) Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga101.jf.intel.com with ESMTP; 27 Jan 2011 16:34:25 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.60,388,1291622400"; d="scan'208";a="701484543" Received: from unknown (HELO [10.255.14.102]) ([10.255.14.102]) by orsmga001.jf.intel.com with ESMTP; 27 Jan 2011 16:34:11 -0800 Message-ID: <4D420F02.8050306@intel.com> Date: Thu, 27 Jan 2011 16:34:10 -0800 From: Saul Wold User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.10) Gecko/20100621 Fedora/3.0.5-1.fc13 Lightning/1.0b2pre Thunderbird/3.0.5 MIME-Version: 1.0 To: "Tian, Kevin" References: <4D41B2B1.2050206@intel.com> <625BA99ED14B2D499DC4E29D8138F1504DB5E056FB@shsmsx502.ccr.corp.intel.com> In-Reply-To: <625BA99ED14B2D499DC4E29D8138F1504DB5E056FB@shsmsx502.ccr.corp.intel.com> Cc: "poky@pokylinux.org" Subject: Re: [PATCH 2/5] cronie: enable multi-user crontab usage and make cron environment complete X-BeenThere: poky@yoctoproject.org X-Mailman-Version: 2.1.13 Precedence: list List-Id: Poky build system developer discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 28 Jan 2011 00:34:26 -0000 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit On 01/27/2011 04:07 PM, Tian, Kevin wrote: >> From: Wold, Saul >> Sent: Friday, January 28, 2011 2:00 AM >>> + >>> +pkg_postinst_${PN} () { >>> + if [ "x$D" != "x" ] ; then >>> + exit 1 >>> + fi >>> + >>> + # below setting is necessary to allow normal user using crontab >>> + >>> + # add 'crontab' group and setgid for crontab binary >>> + grep crontab /etc/group || addgroup crontab >>> + chown root:crontab /usr/bin/crontab >>> + chmod 2755 /usr/bin/crontab >>> + >>> + # allow 'crontab' group write to /var/spool/cron >>> + chown root:crontab /var/spool/cron >>> + chmod 770 /var/spool/cron >>> + >>> + chmod 600 /etc/crontab >>> } >> Kevin, >> >> I am going to hold off pulling this and the at changes. >> >> Is there a specific reason that you are creating this as a post install >> operation for the permission management, both this and the at recipes >> are doing this. Please remember that adding additional items that run >> at first boot slow things down. >> >> Is it possible to run these permission changes at actual install time, >> instead of on the target. >> > > I thought about this when baking the patch, and finally went to this way because > there's chown operation and new group needs to be created. This has to be done > on the target, as you'll see in other similar recipes like dbus, hal, ... > Since this is a similar theme, is there a bbclass or method that could be written in order to handle adding users/groups to the passwd/group file during sysroot creation time, prior to packaging? Will pseudo handle this correctly? Maybe for 1.0 we go this route, but we should think more about this, in order to create the users and groups we need. I think Ke may run into this also for the rootless X work he is doing. Sau! > Thanks > Kevin >