From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [PATCH 0/7 v3] tools/hotplug: systemd changes for 4.5 Date: Tue, 6 Jan 2015 10:05:42 +0000 Message-ID: <1420538742.28863.98.camel@citrix.com> References: <1418988333-5404-1-git-send-email-olaf@aepfle.de> <20141219191032.GB9213@laptop.dumpdata.com> <20141222080639.GA6139@aepfle.de> <20141231153106.GA2928@laptop.dumpdata.com> <20150105212206.GA3869@l.oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20150105212206.GA3869@l.oracle.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: Konrad Rzeszutek Wilk Cc: Olaf Hering , wei.liu2@citrix.com, mcgrof@do-not-panic.com, ian.jackson@eu.citrix.com, xen-devel@lists.xen.org, m.a.young@durham.ac.uk, anthony.perard@citrix.com List-Id: xen-devel@lists.xenproject.org On Mon, 2015-01-05 at 16:22 -0500, Konrad Rzeszutek Wilk wrote: > However there is a question in there for Ian: > > "The place of the wrapper is currently LIBEXEC_BIN, it has to be > decided what the final location is supposed to be. IanJ wants it in > "/etc". > " > > IanJ - any specific reasons for having it in /etc instead of > LIBEXEC_BIN? IIRC Ian explained this is the course of the thread. It's because an administrator might reasonably want to edit the file to apply local configuration. It is in effect a configuration file masquerading as a script. > > The workaround is to document what the 'context' is .. or whatever > > else is needed to make this work. > > Such as this might be good (Or perhaps move it to the INSTALL file) I think the Release Notes are the right place for this sort of information. e.g. http://wiki.xenproject.org/wiki/Xen_Project_4.4_Release_Notes#Known_issues Ian.