<div class="gmail_quote">On Wed, Dec 9, 2009 at 4:24 PM, Steven Boscarine <span dir="ltr">&lt;<a href="mailto:steven.boscarine@childrens.harvard.edu">steven.boscarine@childrens.harvard.edu</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">



  

<div bgcolor="#ffffff" text="#000000">
Fine with me.  I never even plan on using the ANT script anyway...just
trying to make it easier for you to document.<br></div></blockquote><div><br>Yep, this is certainly a migration effort ;) <br><br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div bgcolor="#ffffff" text="#000000">
<br>
If it were me, I&#39;d prefer a fail message telling me to set JBOSS_HOME
and just handle everything with OS variables and not even have a
properties file to edit.</div></blockquote><div><br>Good point, that is the correct message. Perhaps something like:<br><br>Please set the JBOSS_HOME environment variable. Alternatively, you can set jboss.home in build.properties. <br>
 </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div bgcolor="#ffffff" text="#000000"><br>
I don&#39;t really have much of an opinion on the subject.  I&#39;ll apply your
suggestions and then maybe the ant strategy should be architected by an
actual ANT user. <br></div></blockquote><div><br>I&#39;ll apply this change. I&#39;m waiting on an <a href="http://oss.sonatype.org">oss.sonatype.org</a> account anyway.<br><br>-Dan<br></div></div><br>-- <br>Dan Allen<br>
Senior Software Engineer, Red Hat | Author of Seam in Action<br>Registered Linux User #231597<br><br><a href="http://mojavelinux.com">http://mojavelinux.com</a><br><a href="http://mojavelinux.com/seaminaction">http://mojavelinux.com/seaminaction</a><br>
<a href="http://www.google.com/profiles/dan.j.allen">http://www.google.com/profiles/dan.j.allen</a><br>