SonarQube 対応を入れたが appveyor ビルドがこけている

[m-tmatma]

バグ内容

#510, #674 で SonarQube 対応を入れたが appveyor ビルドがこけている

再現手順

https://ci.appveyor.com/project/sakuraeditor/sakura/builds/24167955

[m-tmatma]

以下で環境変数 SONAR_QUBE_TOKEN で判断している。

sakura/build-sln.bat

Lines 48 to 54 in 30e020c

	if "%SONAR_QUBE_TOKEN%" == "" (
	@echo "%CMD_MSBUILD%" %SLN_FILE% /p:Platform=%platform% /p:Configuration=%configuration% /t:"Build" %EXTRA_CMD% %LOG_OPTION%
	"%CMD_MSBUILD%" %SLN_FILE% /p:Platform=%platform% /p:Configuration=%configuration% /t:"Build" %EXTRA_CMD% %LOG_OPTION%
	) else (
	@echo "%BUILDWRAPPER_EXE%" --out-dir %~dp0bw-output "%CMD_MSBUILD%" %SLN_FILE% /p:Platform=%platform% /p:Configuration=%configuration% /t:"Rebuild" %LOG_OPTION%
	"%BUILDWRAPPER_EXE%" --out-dir %~dp0bw-output "%CMD_MSBUILD%" %SLN_FILE% /p:Platform=%platform% /p:Configuration=%configuration% /t:"Rebuild" %LOG_OPTION%
	)

appveyor の設定で SONAR_QUBE_TOKEN を定義していたので SonarQube の解析が走るが
azure pipelines とは違って、choco install "msbuild-sonarqube-runner" -y を実行していないので
解析に失敗する。

[m-tmatma]

appveyor の設定で SONAR_QUBE_TOKEN 等の環境変数の定義を削除して、
リビルドかけた。

https://ci.appveyor.com/project/sakuraeditor/sakura/builds/24168614

[m-tmatma]

これ azure pipelines 側でもあかん気がする。
SONAR_QUBE で切り替えたつもりだったが、実際には SONAR_QUBE_TOKEN で切り替えていた。
つまり、通常のビルドでも SonarQube が走る。

[berryzplus]

これ azure pipelines 側でもあかん気がする。
SONAR_QUBE で切り替えたつもりだったが、実際には SONAR_QUBE_TOKEN で切り替えていた。
つまり、通常のビルドでも SonarQube が走る。

枝タスクのconditionでスケジュール実行かどうかを見ているから実害はないのでは？と思ったりするけど調べてみないと分からない雰囲気ですね。

[berryzplus]

build-wrapperがないからコケるのか。

[m-tmatma]

これ azure pipelines 側でもあかん気がする。
SONAR_QUBE で切り替えたつもりだったが、実際には SONAR_QUBE_TOKEN で切り替えていた。
つまり、通常のビルドでも SonarQube が走る。

枝タスクのconditionでスケジュール実行かどうかを見ているから実害はないのでは？と思ったりするけど調べてみないと分からない雰囲気ですね。

azure pipelines の 設定で SONAR_QUBE_TOKEN は常に定義されているので
通常の VS2017 の JOB のビルド走った時に build-sln.bat を使ってビルドしているので
SonarQube の解析が走ってしまう。

Fork からの PR では SONAR_QUBE_TOKEN の値は無効なので SonarQube は走らないですが
master にマージされたビルドでは SONAR_QUBE_TOKEN が有効なので通常の JOB で
SonarQube が走ります。

[m-tmatma]

#879 で修正 PR を送りました。

[m-tmatma]

appveyor の設定で SONAR_QUBE_TOKEN 等の環境変数の定義を削除して、
リビルドかけた。

https://ci.appveyor.com/project/sakuraeditor/sakura/builds/24168614

これはパスしました。

[m-tmatma]

#879 をマージしました。