admin@glassfish.java.net

Please review fix for 6467835

From: Sivakumar Thyagarajan <Sivakumar.Thyagarajan_at_Sun.COM>
Date: Thu, 14 Sep 2006 16:55:33 +0530

Could you please review this fix for issue 6467835?
http://blrshare.india.sun.com/~st148833/codereview/6467835/

Issue: The varhome option was not sent while trying to init the
imqbroker, during domain creation, and this resulted in the imqbrokerd
using the default varhome [domain1's varhome] always.

Thanks
--Siva.

uw-diff for non-swan users:
> Index: servermgmt/src/java/com/sun/enterprise/admin/servermgmt/RepositoryManager.java
> ===================================================================
> RCS file: /cvs/glassfish/admin/servermgmt/src/java/com/sun/enterprise/admin/servermgmt/RepositoryManager.java,v
> retrieving revision 1.6
> diff -u -w -r1.6 RepositoryManager.java
> --- servermgmt/src/java/com/sun/enterprise/admin/servermgmt/RepositoryManager.java 8 May 2006 16:27:12 -0000 1.6
> +++ servermgmt/src/java/com/sun/enterprise/admin/servermgmt/RepositoryManager.java 14 Sep 2006 11:23:12 -0000
> @@ -771,6 +771,8 @@
> final List cmdInput = new ArrayList();
> cmdInput.add(broker.getAbsolutePath());
> cmdInput.add("-init");
> + cmdInput.add("-varhome");
> + cmdInput.add(mqVarHome.getAbsolutePath());
> ProcessExecutor pe = new ProcessExecutor
> ((String[])cmdInput.toArray(new String[cmdInput.size()]));
> pe.execute(false, false);