On 08/24/12 19:34, Ben Walton wrote:
+1.  This also looks obviously correct.  It's probably good form to
put the RC_ stuff there still but this is a solaris 10 only (I think?)
package so those are really only a formality.  They don't hurt
anything though so I'd leave them in.

Yes, I reached the same conclusion.

Do we still care to check /opt/csw/etc/csw.conf too?  I'm ok with
leaving that behind but there are sites that might use it.  All of our
CAS scripts check both locations too (/opt/csw/etc and then
/etc/opt/csw to give local preference over possibly NFS-shared).

Done, attached the patch to apply above others.

Other than that, it looks sound to me.  Thanks for sharing this!

You're welcome. It's going to be quite useful to me as well :-)

Laurent
--- cswmysql5.tpl.old	Mon Aug 27 09:54:26 2012
+++ cswmysql5.tpl	Mon Aug 27 09:57:54 2012
@@ -195,6 +195,11 @@
 ### START OPENCSW ARCHITECTURE SELECTION
 # Source the configuration
 # This is used to define the binary architeture to run
+
+# Check the old location first
+[ -r @prefix@/etc/csw.conf ] && . @prefix@/etc/csw.conf
+
+# Check the current location, its content overrides the previous one
 [ -r @sysconfdir@/csw.conf ] && . @sysconfdir@/csw.conf
 
 # If the package-specific variable is not set, try the default
_______________________________________________
devel mailing list
[email protected]
https://lists.opencsw.org/mailman/listinfo/devel

Reply via email to