TEMP_GIT_SHORT_COMMIT_HASH および TEMP_GIT_COMMIT_HASH を廃止する #1193
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR の目的
TEMP_GIT_SHORT_COMMIT_HASH および TEMP_GIT_COMMIT_HASH を廃止する
カテゴリ
PR の背景
以下の一連の対応により、 APPVEYOR 依存変数を使わなくなった。
#1183
#1184
#1185
#1186
#1187
APPVEYOR 依存変数を削除する対応の中で、#886 (comment) の際に既存動作を変えないために #969 で TEMP_GIT_COMMIT_HASH と TEMP_GIT_SHORT_COMMIT_HASH を導入した。
#1183 (comment)
#1190 (comment)
の一環で TEMP_GIT_SHORT_COMMIT_HASH および TEMP_GIT_COMMIT_HASH を廃止する
PR のメリット
変数の定義が減るのでシンプルになる。
PR のデメリット (トレードオフとかあれば)
デメリットではないが、zip アーカイブを作成するときに
SHORTHASH
という変数の定義をGIT_SHORT_COMMIT_HASH
の変数に置き換えたので、ローカルビルドしたときにzip ファイル名にコミットハッシュの短縮形が入る。(現状はローカルビルドの場合は入っていない)
↑ もともと TEMP_GIT_SHORT_COMMIT_HASH および TEMP_GIT_COMMIT_HASH は
APPVEYOR_REPO_COMMIT および APPVEYOR_SHORTHASH の廃止の際に仕様が変わるのを
防ぐためのものだった。
※ この PR は CI ビルドの場合は影響はない。
PR の影響範囲
関連チケット
#886
#969
#1183
#1184
#1185
#1186
#1187
参考資料