From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Vrabel Subject: Re: [PATCH] tools/pygrub: store kernels in /var/run/pygrub Date: Thu, 17 Apr 2014 12:33:29 +0100 Message-ID: <534FBC09.40507@cantab.net> References: <1397731926-29636-1-git-send-email-olaf@aepfle.de> <534FB383.5060806@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <534FB383.5060806@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Andrew Cooper Cc: Olaf Hering , Ian.Jackson@eu.citrix.com, Ian.Campbell@citrix.com, xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org On 17/04/14 11:57, Andrew Cooper wrote: > On 17/04/14 11:52, Olaf Hering wrote: >> Move location of temporary bootfiles from /var/run/xend/boot to >> /var/run/pygrub. If the directory does not exist create it, unless >> --output-directory= was specified. >> >> The reason for this change is that all entrys below /var/run have to be >> created at runtime in case /var/run is cleared on every boot. >> >> Signed-off-by: Olaf Hering >> --- >> >> New code not runtime tested as pygrub, just as hello_world.py... > > I don't think it is valid in the slightest for programs like this to be > mkdir()ing their environment. It is very much a packaging responsibility. /var/run may not be persistent so this can't be done by the packaging. David