Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

appveyor の成果物にビルド番号を含めるか検討する #165

Closed
m-tmatma opened this issue Jun 23, 2018 · 5 comments
Closed

appveyor の成果物にビルド番号を含めるか検討する #165

m-tmatma opened this issue Jun 23, 2018 · 5 comments
Labels
CI appveyor など CI 関連 【ChangeLog除外】
Milestone

Comments

@m-tmatma
Copy link
Member

appveyor の成果物にビルド番号を含めるか検討する

#114 (comment)

ローカルでの区別しやすさを考えると
ビルド番号を含めてもいいかも
デメリットはダウンロードしたzip に対して自動で何かしたいときに邪魔になること

#114 (comment)

> ビルド番号を含めてもいいかも
これの判断は難しいところです。一旦別件として検討しましょう。

#109 (comment)

既存の有名なプロダクトがどのような方針を採っているのか気になってます

@m-tmatma m-tmatma added the CI appveyor など CI 関連 【ChangeLog除外】 label Jun 23, 2018
@m-tmatma
Copy link
Member Author

#177 の修正(d265094) に添付のパッチを適用することで可能なのを確認

build-number-version.patch.txt

@m-tmatma
Copy link
Member Author

#187 (comment)

余談ですが個人的にはやっぱり artifacts ZIPファイル名に githash または PR 番号(これは原理的に難しそうな気がする)が含まれているとレビュー都合的にありがたいなーと思っています。

どちらも可能だと思います。

#167 (comment)

を見ていただくと

環境変数で githash は APPVEYOR_PULL_REQUEST_HEAD_COMMIT あるいは APPVEYOR_REPO_COMMIT で取れますし、 PR 番号は APPVEYOR_PULL_REQUEST_NUMBER で取れます。

@m-tmatma
Copy link
Member Author

環境変数で githash は APPVEYOR_PULL_REQUEST_HEAD_COMMIT あるいは APPVEYOR_REPO_COMMIT で取れますし、 PR 番号は APPVEYOR_PULL_REQUEST_NUMBER で取れます。

上記の環境変数を参照して以下の変数を設定している箇所に追加すればいいです。

set BASENAME=sakura-%platform%-%configuration%%SUFFIX%

ZIP ファイル名だけ変えたければ以下です。

set OUTFILE=%BASENAME%.zip

@kobake
Copy link
Member

kobake commented Jun 27, 2018

PR 番号含められるの良いですね。BASENAME, OUTFILE 両方変わっているのが個人的には好ましいです。

@m-tmatma
Copy link
Member Author

m-tmatma commented Jul 4, 2018

#199 で対応

@m-tmatma m-tmatma closed this as completed Jul 4, 2018
@ds14050 ds14050 added the CI appveyor など CI 関連 【ChangeLog除外】 label Sep 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI appveyor など CI 関連 【ChangeLog除外】
Projects
None yet
Development

No branches or pull requests

3 participants