The problem with that is it requires the enableDelayedExpansion to be set, which may not be a big deal. Also we still do the setlocal in an @if "%OS%" == "Windows_NT" setlocal block. If we could get rid of these NT tests then that might work.

On 05/16/2013 10:41 PM, Nicklas Karlsson wrote:

When googling for the issue, http://ss64.com/nt/delayedexpansion.html comes up (and usage of !varname!), havent't tested it though.
On Fri, May 17, 2013 at 4:51 AM, James R. Perkins <jperkins@redhat.com> wrote:
Hello All,
There was an issue found with the Windows batch scripts if the JBOSS_HOME path had a space or a ) in it. I created a fix and PR to fix the issue [1].

If anyone has any good or better ideas on how to make this cleaner please let me know. We are toying with the idea of just using an EXE instead of batch scripts, but any input from someone with more Windows batch scripting experience than me is welcome.

[1]: https://github.com/wildfly/wildfly/pull/4520
-- James R. Perkins JBoss by Red Hat
_______________________________________________
wildfly-dev mailing list
wildfly-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/wildfly-dev
--
Nicklas Karlsson, +358 40 5062266
Vaakunatie 10 as 7, 20780 Kaarina

-- 
James R. Perkins
JBoss by Red Hat