--copyTests should work with no argument, or improve doc

[bill]

I found that

{{{{ ./build.sh --release --releaseDir /workspace/trunk-release --profile standard --copyTests }}}

gives an error message, while the deprecated syntax works:

./build.sh --release --releaseDir /workspace/trunk-release --profile standard copyTests=true

[Rawld Gill]

copyTests can have three values:

1. falsy: don't copy anything tagged as a test.
2. truthy, but not "build": copy anything marked as a test, but don't apply any transforms
3. "build": treat a test modules like any other module and apply all appropriate transforms

So, --copyTests without any arguments is not allowed. --copyTests true or --copyTests 1 and so on should work fine

I can certainly see how that no arg to copyTests should imply true. Bill--what's your feedback...would you like that enhancement?

[bill]

Oh I see... well it's just that the error message was confusing:

$ ./build.sh --release --releaseDir /workspace/trunk-release --profile standard --copyTests
error(343) Illegal command line argument. switch: --copyTests; position: 7

It sounds like --copyTests is illegal.

And the help message is also misleading:

OPTIONS
...
     
     --writeProfile <filename> write the processed profile to filename
     
     --copyTests               Copy test files and the DOH package.

It tells me that options like "writeProfile" take an argument, but --copyTests doesn't.

So... yeah, would be nice to make --copyTests work with no argument, or otherwise fix the error message and help message.

[Rawld Gill]

In [31277]:

improve error reporting; fixes #15715; !strict