search

sakura-editor / sakura

SAKURA Editor (Japanese text editor for MS Windows)
https://sakura-editor.github.io/
Other
1.23k stars 162 forks source link

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

Open m-tmatma opened 4 years ago

m-tmatma commented 4 years ago

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.

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

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

berryzplus commented 4 years ago

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

---- 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 commented 4 years ago
  • TEMP_GIT_SHORT_COMMIT_HASH : 8文字。なんで？
  • TEMP_GIT_COMMIT_HASH フル桁と同じものな気がする。
  • TEMP_GIT_XXXってなんでしたっけ？ｗ

1193 を投げました。

m-tmatma commented 4 years ago
  • CI_REPO_NAME は GITHUB_REPO_NAME な気がする。

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

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

berryzplus commented 4 years ago

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 commented 4 years ago

あかん、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 commented 4 years ago

ちがうな・・・。

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 と同じ気がする

m-tmatma commented 4 years ago

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

でいいと思います。

berryzplus commented 4 years ago

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)

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