History | Log In     View a printable version of the current page.  
Issue Details (XML | Word | Printable)

Key: MAVEN-1466
Type: Bug Bug
Status: Open Open
Priority: Minor Minor
Assignee: Unassigned
Reporter: Albert Davidson Chou
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Maven 1

install_repo.bat can require inconsistent quoting on the command line

Created: 14/Oct/04 06:13 PM   Updated: 13/Apr/06 05:52 PM
Component/s: documentation
Affects Version/s: 1.0
Fix Version/s: None

Time Tracking:
Original Estimate: 1 hour
Original Estimate - 1 hour
Remaining Estimate: 1 hour
Remaining Estimate - 1 hour
Time Spent: Not Specified
Remaining Estimate - 1 hour

Environment: Windows XP Professional SP 1


 Description  « Hide
The install_repo.bat script in version 1.0 builds up its argument word by word and then puts double quotes around the value thereby built up wherever it is used. However, this implementation makes using the script kind of weird. On Windows I have to quote the command anyway if it contains spaces, e.g.,

"%MAVEN_HOME%\bin\install_repo.bat"

and I would expect to have to quote its argument for similar reasons. But as it stands now, install_repo.bat requires the following quoting syntax if both MAVEN_HOME and HOME (or HOMEPATH) contain spaces:

"%MAVEN_HOME%\bin\install_repo.bat" %HOME%\.maven\repository

Note the inconsistency in quoting.

The Windows batch language doesn't treat quotes the way most shells do; the quotes are literally part of the string rather than just a meta-character that signifies that contained whitespace is literal rather than a command or argument separator. The syntax %~1 can be used to produce a non-quoted and fully-path-expanced version of whatever argument %1 was, at least in more recent versions of Windows (2000, XP, 2003). No guarantees that this feature is present in Windows 9x....



 All   Comments   Work Log   Change History      Sort Order: Ascending order - Click to sort in descending order
There are no comments yet on this issue.