Hrm... I guess it would help if I actually included the patch... oops =)

----8<----
Index: AutoDeployer.java
===================================================================
RCS file: /products/cvs/ejboss/jboss/src/main/org/jboss/ejb/AutoDeployer.java,v
retrieving revision 1.11
diff -u -r1.11 AutoDeployer.java
--- AutoDeployer.java   2000/12/07 15:44:08     1.11
+++ AutoDeployer.java   2001/03/06 22:38:49
@@ -262,7 +262,7 @@
       // Add URLs to list
       while (urls.hasMoreTokens())
       {
-         String url = urls.nextToken();
+         String url = urls.nextToken().trim();

          // Check if directory
          File urlFile = new File(url.startsWith ("file:") ? url.substring (5) : url);
---->8----

--jason

On Tue, 6 Mar 2001, Jason Dillon wrote:

> Hello, I ran into a small issue with AutoDeployer when trying to specify a
> list of URLs that contained some extra white space similar to this:
>
>   <mbean code="org.jboss.ejb.AutoDeployer"
>        name="EJB:service=AutoDeployer">
>     <attribute name="Deployer">J2EE:service=J2eeDeployer</attribute>
>     <attribute name="URLs">
>       /some/path/somebean.jar,
>       /some/path/someotherbean.jar
>     </attribute>
>   </mbean>
>
> I modified the AutoDeployer.startService() method to trim each URL returned
> from the StringTokenizer and it fixed this problem.
>
> Below is a patch (ala 'cvs diff -u') of AutoDeployer with this change made.
> I think that this minor change helps the readability of the jboss.jcml file
> when specify more than one URL to deploy.
>
> I hope that this change would be acceptable for integrating with the main
> code line.
>
> Thanks,
>
> --jason
>
>


Reply via email to