githash.batのログ出力がおかしい(#1183 の指摘)

[m-tmatma]

Originally posted by @berryzplus in https://github.com/sakura-editor/sakura/pull/1183#issuecomment-583844746

PRのレビュー結果には関係しませんが、気付いたことのメモです。

appveyor のビルドログ

---- Make githash.h ----
checking GIT_SHORT_COMMIT_HASH, GIT_COMMIT_HASH
GIT_SHORT_COMMIT_HASH : eec7d9a
GIT_COMMIT_HASH       : eec7d9a00fe0f226dc245daa47a81a09b651e65e
TEMP_GIT_SHORT_COMMIT_HASH : eec7d9a
TEMP_GIT_COMMIT_HASH       : eec7d9a00fe0f226dc245daa47a81a09b651e65e
GIT_REMOTE_ORIGIN_URL : https://github.com/sakura-editor/sakura.git
GIT_TAG_NAME          : 

CI_REPO_NAME          : sakura-editor/sakura
CI_ACCOUNT_NAME             : sakuraeditor
CI_BUILD_VERSION            : 1.0.2614
CI_BUILD_NUMBER             : 2614
CI_BUILD_URL                : https://ci.appveyor.com/project/sakuraeditor/sakura/build/1.0.2614

GITHUB_COMMIT_URL           : https://github.com/sakura-editor/sakura/commit/eec7d9a00fe0f226dc245daa47a81a09b651e65e
GITHUB_COMMIT_URL_PR_HEAD   : https://github.com/sakura-editor/sakura/pull/1183/commits/63f6683199a96ddc41d3b34d3d82add248ddfc81
GITHUB_PR_HEAD_COMMIT       : 63f6683199a96ddc41d3b34d3d82add248ddfc81
GITHUB_PR_HEAD_SHORT_COMMIT : 63f66831

APPVEYOR_URL          : https://ci.appveyor.com
APPVEYOR_PROJECT_SLUG : sakura
        1 file(s) moved.
..\sakura_core\githash.h was updated.

• GIT_SHORT_COMMIT_HASH 7文字。
• GIT_COMMIT_HASH 多分フル桁。
• TEMP_GIT_SHORT_COMMIT_HASH : 8文字。なんで？
• TEMP_GIT_COMMIT_HASH フル桁と同じものな気がする。
• TEMP_GIT_XXXってなんでしたっけ？ｗ
• CI_XXX のグループとGITHUB_XXX のグループは、出力順を入れ替えたほうが見やすくなる気配です。
• CI_REPO_NAME は GITHUB_REPO_NAME な気がする。
• GITHUB_COMMIT_URL_PR_HEAD を GITHUB_PR_HEAD_URL にしたら GITHUB_PR_HEAD_ の頭が揃うような。
• GITHUB_PR_HEAD_COMMIT ⇒ GITHUB_PR_HEAD_COMMIT_HASH （似た名前の変数に合わせる）
• GITHUB_PR_HEAD_SHORT_COMMIT ⇒ GITHUB_PR_HEAD_SHORT_COMMIT_HASH （似た名前の変数に合わせる）
• GITHUB_XXX のグループ内の並び順が、コミットハッシュ⇒URLの順だと分かりやすい気配です。

これ書いてどうする？ってのは考えてませんでしたが、 出力をきれいに揃えてみて初めて見えてくる気付きもあるのかな？と思ってます。

Originally posted by @berryzplus in https://github.com/sakura-editor/sakura/pull/1183#issuecomment-583844746

[berryzplus]

自分のなかで収まりがよい順番に並べ替えてみました。

---- Make githash.h ----
GIT_REMOTE_ORIGIN_URL : https://github.com/sakura-editor/sakura.git
GIT_SHORT_HASH        : eec7d9a0
GIT_COMMIT_HASH       : eec7d9a00fe0f226dc245daa47a81a09b651e65e

GITHUB_PR_URL         : https://github.com/sakura-editor/sakura/pull/1183/commits
GITHUB_PR_SHORT_HASH  : 63f66831
GITHUB_PR_COMMIT_HASH : 63f6683199a96ddc41d3b34d3d82add248ddfc81

GIT_TAG_NAME          : 

CI_BUILD_URL          : https://ci.appveyor.com/project/sakuraeditor/sakura/build/1.0.2614
APPVEYOR_URL          : https://ci.appveyor.com
CI_ACCOUNT_NAME       : sakuraeditor
APPVEYOR_PROJECT_NAME : sakura
CI_BUILD_VERSION      : 1.0.2614
CI_BUILD_NUMBER       : 2614
        1 file(s) moved.
..\sakura_core\githash.h was updated.

並べ替えだけじゃなく、変数の整理もしている感じです。 特に「こうしなければ！」なノリではないので、必要は変数を削っちゃってるかもしれません。

[m-tmatma]

• TEMP_GIT_SHORT_COMMIT_HASH : 8文字。なんで？
• TEMP_GIT_COMMIT_HASH フル桁と同じものな気がする。
• TEMP_GIT_XXXってなんでしたっけ？ｗ

#1193 を投げました。

[m-tmatma]

• CI_REPO_NAME は GITHUB_REPO_NAME な気がする。

https://github.com/sakura-editor/sakura/blob/fdb92d440f33fabe5171f6fadd9a7fcd064a3a81/zipArtifacts.bat#L41-L47

の部分が GitHub にしか事実上対応できない気がする。

[berryzplus]

GITHUB_TAG_HEAD_URLというのを作るのもアリかも知れない・・・（言いっぱなし

変数名	意味
GITHUB_HEAD_URL	最新masterのHEADコミットURLを指す
GITHUB_PR_HEAD_URL	PRのHEADコミットURLを指す
GITHUB_TAG_HEAD_URL	リリースタグのHEADコミットURLを指す

バージョン情報ダイアログに表示する（or リンクさせる）のは、この中のどれか1つでいいような。（実装大変そうですが :cry:

[berryzplus]

あかん、TAG HEADなんてないんやね。

https://github.com/sakura-editor/sakura/releases/tag/v2.4.0-beta4 を表示したときに出るリンクに紐付けられないかと思っただけです。

Ver2.4.0 beta4 (Unicode版) @KENCHjp KENCHjp released this Feb 15, 2020 · 8 commits to master since this release

これ、ということになるのかな？ https://github.com/sakura-editor/sakura/compare/v2.4.0-beta4...master

[berryzplus]

ちがうな・・・。

https://github.com/sakura-editor/sakura/releases/tag/v2.4.0-beta4/commits/13ebfd36d5cda933bfa9681b91e5cbe544c2628f という形式が使えればそれが正しい気がする。

しかし、このURLで表示されるページは https://github.com/sakura-editor/sakura/releases/tag/v2.4.0-beta4 と同じ気がする

• https://github.com/sakura-editor/sakura/releases/tag/v2.4.0-beta4/commits/13ebfd36d5cda933bfa9681b91e5cbe544c2628f
• https://github.com/sakura-editor/sakura/releases/tag/v2.4.0-beta4

[m-tmatma]

https://github.com/sakura-editor/sakura/tree/v2.4.0-beta4

でいいと思います。

[berryzplus]

https://github.com/sakura-editor/sakura/tree/v2.4.0-beta4

でいいと思います。

じゃ、それで（そういうのがあるのも知らんかったｗ

PR #1201 で出してるリッチエディット採用の提案は、コピペ用バージョン表記の分かりやすい形式を模索するためのものだったりします。

現在の形式

サクラエディタ   v2.4.0.0 32bit DEBUG dev
(GitHash xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx)
(GitURL http://github.com/ACOUNT_NAME/PROJECT.git)

変更後の最新master例

サクラエディタ   v2.4.0.0 32bit
(URL https://github.com/sakura-editor/sakura/commit/eec7d9a0)

変更後のPRビルド例

サクラエディタ   v2.4.0.0 32bit
(URL https://github.com/sakura-editor/sakura/pull/1183/commits/63f66831)

変更後のタグ（＝リリース）ビルド例

サクラエディタ   v2.4.0.0 32bit
(URL https://github.com/sakura-editor/sakura/tree/v2.4.0-beta4)

バージョン情報ダイアログの真ん中あたりにあるリンクを並べ替えないといかんですが、コピペ用バージョン情報にソースを識別できるリンクを埋め込むプランは良さげだと思っています。