Add reproducer for #1706 #1752
Add reproducer for #1706 #1752
Conversation
| @"--debug" | ||
| }; | ||
| var args = new ProcessRunnerArguments(batchName, true) { CmdLineArgs = expected, WorkingDirectory = testDir }; | ||
| var success = runner.Execute(args); |
There was a problem hiding this comment.
What is passed to the Test.bat looks like this:
^^^"^^^-^^^D^^^s^^^o^^^n^^^a^^^r^^^.^^^s^^^c^^^a^^^n^^^A^^^l^^^l^^^F^^^i^^^l^^^e^^^s^^^=^^^t^^^r^^^u^^^e^^^"^^^ ^^^"^^^-^^^D^^^p^^^r^^^o^^^j^^^e^^^c^^^t^^^.^^^s^^^e^^^t^^^t^^^i^^^n^^^g^^^s^^^=^^^D^^^:^^^\^^^D^^^e^^^v^^^L^^^i^^^b^^^T^^^e^^^s^^^t^^^\^^^C^^^l^^^a^^^s^^^s^^^L^^^i^^^b^^^r^^^a^^^r^^^y^^^T^^^e^^^s^^^t^^^.^^^s^^^o^^^n^^^a^^^r^^^q^^^u^^^b^^^e^^^\^^^o^^^u^^^t^^^\^^^s^^^o^^^n^^^a^^^r^^^-^^^p^^^r^^^o^^^j^^^e^^^c^^^t^^^.^^^p^^^r^^^o^^^p^^^e^^^r^^^t^^^i^^^e^^^s^^^"^^^ ^^^"^^^-^^^-^^^f^^^r^^^o^^^m^^^=^^^S^^^c^^^a^^^n^^^n^^^e^^^r^^^M^^^S^^^B^^^u^^^i^^^l^^^d^^^/^^^5^^^.^^^1^^^3^^^.^^^1^^^"^^^ ^^^"^^^-^^^-^^^d^^^e^^^b^^^u^^^g^^^"
Double quotes surround every argument, and every character is then escaped by prepending ^^^. This works somewhat but breaks under certain conditions. What precisely the conditions are is not clear to me. The log from the issue indicates a Portuguese local (CP860), which may have an impact too.
| @"@echo off | ||
| REM The sonar-scanner.bat uses %* to pass the argument to javac.exe | ||
| echo %* | ||
| REM Because of the escaping, the single arguments are somewhat broken on echo. A workaround is to add some new lines for some reason. |
There was a problem hiding this comment.
This should be fixed by #1751
|
Kudos, SonarCloud Quality Gate passed!
|
| @"-Dsonar.scanAllFiles=true", | ||
| @"-Dproject.settings=D:\DevLibTest\ClassLibraryTest.sonarqube\out\sonar-project.properties", | ||
| @"--from=ScannerMSBuild/5.13.1", | ||
| @"--debug" |
There was a problem hiding this comment.
Taken from the community ticket logs:
-Dsonar.scanAllFiles=true -Dproject.settings=D:\DevLibTest\ClassLibraryTest\.sonarqube\out\sonar-project.properties --from=ScannerMSBuild/5.13 --debug <sensitive data removed>
<sensitive data removed> indicates that there might be something passed that caused the failure but isn't in the logs.
martin-strecker-sonarsource
requested a review
from costin-zaharia-sonarsource
github-actions
bot
assigned costin-zaharia-sonarsource and unassigned martin-strecker-sonarsource
github-actions
bot
assigned martin-strecker-sonarsource and unassigned costin-zaharia-sonarsource
Reproducer for #1706