[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] xendomains init script



On Thu, Mar 30, 2006 at 06:09:33AM +0200, Florian Kirstein wrote:
> Hi,
> 
> first: thanks for the nice script :) (second: resent, hope my first mail
> using a wrong From: is ignored by the list daemon)
> 
> I had a (more cosmetic) problem with it:
> > The init script is somewhat SUSE-ish, but it should work on LSB
> > compliant and on RH based distros as well.
> Almost for the LSB-Part: LSB defines that the ini script functions
> like "log_success_msg" can be defined as shell aliases:
> http://www.linuxbase.org/spec/refspecs/LSB_3.0.0/LSB-Core-generic/LSB-Core-generic/iniscrptfunc.html
> 
> The problem with that: shell aliases are usually not enabled in scripts
> (so I'll have to blame LSB for allowing this in init script functions)
> and the real problem, they are never used (at least for bash) in
> if/then/else blocks or functions, which is exactly what your script
> does.
> 
> So, IF a distribution decides to implement log_success_msg LSB conforming
> as alias, as current RedHat distributions (like RHEL4 and including
> FC5) do, this will fail and give "log_success_msg: command not found".
> I think this is a bad idea (see above), but can't blame RedHat for
> following LSB by the word. And as you check for LSB first and then
> for redhat, newer redhat distributions will take the LSB case.
> 
> I don't think there's a nice way to make this fully LSB compatible
> as these funtions are usually called within functions, but I'd
> highly suggest to use a "specific before generic" approach for the
> "distribution specific emulations". Means: test for the redhat
> specific part first and then use LSB. For the LSB part I'd add
> an if-test to check if log_success_msg is an alias and use the
> generic emulation in case it is (assuming log_error_msg would be an
> alias too when log_success_msg is).
> 
> I attached a patch doing exactly this, resulting in a clean run on the
> redhat based distributions and fixing it for possible others using an
> alias in their LSB conformity scripts, leaving it unchanged for the
> rest. Hopefully :) 
> 
> @Kurt: as this is your init-script, would you like to review the patch
> and in case you think it's OK submit it to the official source tree?
> Or whoever is responsible for this :)
> 
> (:ul8er, r@y


+1

   Associated bug at Red Hat is
 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=171056

yes this is a problem, but I didn't tried the patch yet, I assume you tested
it on FC5,

Daniel

-- 
Daniel Veillard      | Red Hat http://redhat.com/
veillard@xxxxxxxxxx  | libxml GNOME XML XSLT toolkit  http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine http://rpmfind.net/

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.