# Please review fixes for 6664457

3 replies [Last post]
dlpharness
Offline
Joined: 2007-07-26

Dawn Phillips

Done.

https://jtharness.dev.java.net/source/browse/jtharness/trunk/code/src/co...

dawn

jtharness@mobileandembedded.org wrote:
> One comment...
>
> In command/excludeList.html:
>
> You expand on how on how to format the command. I think it would be nice to draw a conclusion for the user - no file path can contain an space - C:\Program Files\foo\bar for example...
> [Message sent by forum member 'bkurotsu' (bkurotsu)]
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: interest-unsubscribe@jtharness.dev.java.net
> For additional commands, e-mail: interest-help@jtharness.dev.java.net
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: interest-unsubscribe@jtharness.dev.java.net

bkurotsu
Offline
Joined: 2004-12-13

One comment...

In command/excludeList.html:

You expand on how on how to format the command. I think it would be nice to draw a conclusion for the user - no file path can contain an space - C:\Program Files\foo\bar for example...

bkurotsu
Offline
Joined: 2004-12-13

And no further review necessary for merge into trunk...