Maven 2 & 3

mvn.bat detection of 4NT syntax error

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 2.0.4
  • Fix Version/s: 2.0.8
  • Component/s: Command Line
  • Labels:
    None
  • Environment:
    Windows XP & 4NT 8.0
  • Complexity:
    Intermediate
  • Number of attachments :
    1

Description

Copied from the 4NT 8.0 Documentation:

======================================================================================
"Detecting 4NT or Take Command

From a batch file, you can determine if 4NT or TC is loaded by testing for the variable function @EVAL, with a test like this:

if "%@eval[2 + 2]%" == "4" echo %_cmdproc is loaded!

This test can never succeed in CMD.EXE. (Other variable functions could also be used for the same purpose.) The _CMDPROC internal variable tells you which specific command processor is running.
=======================================================================================

The mvn.bat in 2.0.4 contains:

@REM – 4NT shell
if "%eval[2+2]" == "4" goto 4NTArgs

It should be:

@REM – 4NT shell
if "%@eval[2+2]" == "4" goto 4NTArgs

In case, you don't see it, the first one is missing the @ in between the % and eval

I will attach a patch in a moment.

Issue Links

Activity

Hide
Jason van Zyl added a comment -

Someone with some Windoze Skilz want to check this.

Show
Jason van Zyl added a comment - Someone with some Windoze Skilz want to check this.
Hide
John Casey added a comment -

How does this relate to MNG-2503? I just applied the diff between that attached mvn.bat and the current trunk, with minor differences...

It's been applied to both trunk and the maven-2.0.x branch...can someone with Windows take a look at these two issues, and give us some insight?

Show
John Casey added a comment - How does this relate to MNG-2503? I just applied the diff between that attached mvn.bat and the current trunk, with minor differences... It's been applied to both trunk and the maven-2.0.x branch...can someone with Windows take a look at these two issues, and give us some insight?
Hide
Brian Fox added a comment -

patch already applied. Windows bat works fine.

Show
Brian Fox added a comment - patch already applied. Windows bat works fine.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: