Aside from reformatting the msbuild file for better readability, there are following enhancements:
.rsp file generation is generating lines using ItemGroup functionality, which results in a more readable target (IMHO). As a result, usage of %0d%0a for newline was dropped.
More human-readable error message for when dotnet codegen --version fails, and a message when it succeeds.
Not-swallowing non-0 exit code for main invocation as per #102
An error instead of a warning for when the command exited with 0 but the .GeneratedFileList.txt file doesn't exist. It's an error, because by design on a successful run the file won't be created only if the file list path was not passed as a parameter, and it's passed in the target provided by us.
Only the dbcd71d commit is actually the fix
Aside from reformatting the msbuild file for better readability, there are following enhancements:
.rsp
file generation is generating lines usingItemGroup
functionality, which results in a more readable target (IMHO). As a result, usage of%0d%0a
for newline was dropped.dotnet codegen --version
fails, and a message when it succeeds.0
but the.GeneratedFileList.txt
file doesn't exist. It's an error, because by design on a successful run the file won't be created only if the file list path was not passed as a parameter, and it's passed in the target provided by us.