[PATCH] D133483: Add command line argument parsing to the Windows packaging script.

[Carlos Alberto Enciso via Phabricator via llvm-commits]

CarlosAlbertoEnciso added inline comments.


================
Comment at: llvm/utils/release/build_llvm_release.bat:295
+    )
+    set "arg_name=%aux:~2,250%"
+  ) else (
----------------
hans wrote:
> I'm probably missing something, but for `--type=release` it seems like this would set `arg_name` to `type=release`? Or does it get split into the name and value in some way?
> 
> Or, it looks like the code would maybe work for `--type release`, but in that case the example above would need updating.
> I'm probably missing something, but for `--type=release` it seems like this would set `arg_name` to `type=release`? Or does it get split into the name and value in some way?
> 

The `=` sign in the command line is interpreted as argument separator. The same for space, comma and semicolon.

`--type=release` and `type release` are splitted in 2 arguments `--type` and `release`.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D133483/new/

https://reviews.llvm.org/D133483