APPVEYOR_REPO_COMMIT および APPVEYOR_SHORTHASH への参照を廃止、代替変数を参照して Azure Pipelines 対応の準備をする

[m-tmatma]

PR の目的

APPVEYOR_REPO_COMMIT および APPVEYOR_SHORTHASH への参照を廃止、代替変数を参照して Azure Pipelines 対応の準備をする
(#886 の PR を作り直したもの)

カテゴリ

• リファクタリング
• CI関連
  • Appveyor
  • Azure Pipelines

PR の背景

#821 で Azure Pipelines で環境変数の定義を検討中だが、
Azure Pipelinse の Predefined variables では
$(Build.SourceVersion) (YMLの場合) あるいは BUILD_SOURCEVERSION (バッチファイルの場合)
という appveyor と別の環境変数が割り当てられている。

Azure Pipelines でも同様に commit Hash をもとにした処理を実装しやすいようにするために、APPVEYOR_SHORTHASH および APPVEYOR_REPO_COMMIT を
直接参照するのをやめる。

PR のメリット

Azure Pipelines 対応の準備ができる

PR のデメリット (トレードオフとかあれば)

なし

PR の影響範囲

以下が変わるという意味ではなく影響を受ける可能性です。

• バージョンダイアログ
• バージョンリソース
• ビルド時の環境変数
• githash.h
• zip ファイル名

関連チケット

#886
#821

参考資料

[AppVeyorBot]

✅ Build sakura 1.0.2067 completed (commit dc6c1af57d by @m-tmatma)

[AppVeyorBot]

✅ Build sakura 1.0.2068 completed (commit 62295b0a07 by @m-tmatma)

[berryzplus]

ざっくり版変更概要

新環境変数	旧環境変数	azure pipelines でも取れる取得元
TEMP_GIT_COMMIT_HASH	APPVEYOR_REPO_COMMIT	git show -s --format=%h
TEMP_GIT_SHORT_COMMIT_HASH	APPVEYOR_SHORTHASH	git show -s --format=%H

git showコマンドは、最新コミットの詳細を見るためのコマンド。
-sは--no-patchの省略形で、diffを表示させないという意味です。
--formatは--prettyの別名で、出力整形式を指定します。

コマンドは元々ローカルビルドで動いていたものだから問題はないと思っています。

観点として、廃止されるものがすべて新設されるものに置き換わっているか否かを見る感じになります。

気になった点

廃止される環境変数でソース全体をGrepしました。
（旧環境変数が残っている≠ＮＧです。）

□検索条件  "APPVEYOR_REPO_COMMIT"
検索対象       *.*
フォルダ       C:\work\sakura
除外ファイル   
除外フォルダ   
    (サブフォルダも検索)
    (単語単位で探す)
    (英大文字小文字を区別しない)
    (文字コードセットの自動判別)
    (一致した行を出力)


C:\work\sakura\appveyor_env.py(44,4)  [UTF-8]: 		"APPVEYOR_REPO_COMMIT",
C:\work\sakura\appveyor_env.py(97,10)  [UTF-8]: 					if "APPVEYOR_REPO_COMMIT" in self.env:
C:\work\sakura\appveyor_env.py(102,18)  [UTF-8]: 							self.env["APPVEYOR_REPO_COMMIT"],
C:\work\sakura\appveyor_env.py(110,18)  [UTF-8]: 							self.env["APPVEYOR_REPO_COMMIT"],
4 個が検索されました。

これは、appveyor専用のpythonスクリプトだから対応しなくて良い、という理解でいいですかね？
SHORTHASHのほうは一致ありませんでした。

appveyor_env.pyに旧変数が残っているのが意図通りであれば「問題なし」の判断です 😄

[m-tmatma]

そうです