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バッジ追加、AppVeyor URL修正 #34

Merged
merged 2 commits into from
Jun 2, 2018
Merged

Conversation

kobake
Copy link
Member

@kobake kobake commented Jun 1, 2018

内容

  • README 見出し直下に AppVeyorバッジ追加
  • README 内の AppVeyor の URL が一部間違っていたのを修正

対象 Isssue

対応手順

AppVeyor に sakura-editor ユーザでログインした状態で以下の設定ページを開く。

この中にある Branch sample markdown code から取得できる以下の Markdown をコピーして README の見出し直下に挿入。

[![Build status](https://ci.appveyor.com/api/projects/status/xlsp22h1q91mh96j/branch/master?svg=true)](https://ci.appveyor.com/project/sakuraeditor/sakura/branch/master)

残存エラー

https://ci.appveyor.com/project/sakuraeditor/sakura/build/1.0.28

fatal: destination path 'C:\projects\sakura' already exists and is not an empty directory.
Command exited with code 128

#15 でこのエラーは解消したつもりでしたが、まだ直ってなかったみたいです。
clone_folder を yml 側で指定する等で対処できないかどうか検証します。

@kobake kobake changed the title AppVeyorバッジ追加、AppVeyor URL修正 [WIP] AppVeyorバッジ追加、AppVeyor URL修正 Jun 1, 2018
@berryzplus
Copy link
Contributor

ログを見るとgit cloneが2回行われているように見えます。

https://ci.appveyor.com/project/sakuraeditor/sakura/build/1.0.28

1Build started
2Set-WinSystemLocale ja-JP
3Start-Sleep -s 5
4Restart-Computer
5git clone -q --branch=appveyor-badge https://github.com/sakura-editor/sakura.git C:\projects\sakura
6git clone -q --branch=appveyor-badge https://github.com/sakura-editor/sakura.git C:\projects\sakura
7fatal: destination path 'C:\projects\sakura' already exists and is not an empty directory.
8Command exited with code 128

REMかも知れないので微妙ですが・・・。

昨日参考パッチを上げてみたとき、1回目のgit cloneでコケたので問題自体は残ってそうです。
https://ci.appveyor.com/project/sakuraeditor/sakura/build/1.0.30

@m-tmatma
Copy link
Member

m-tmatma commented Jun 2, 2018

#15 でこのエラーは解消したつもりでしたが、まだ直ってなかったみたいです。
clone_folder を yml 側で指定する等で対処できないかどうか検証します。

こんなの見つけました。

https://github.com/Robpol86/githubBackup/blob/9f19a60af76289e739efe51880ebf2bf2544e8d0/appveyor.yml

@kobake kobake force-pushed the appveyor-badge branch 2 times, most recently from 646bbc7 to 600fe75 Compare June 2, 2018 13:09
@m-tmatma
Copy link
Member

m-tmatma commented Jun 2, 2018

https://www.appveyor.com/docs/environment-variables/
に書いてある変数が利用できるみたいです。

AppVeyor ビルド時に以下のようなエラーが発生することがあったので、それへの対策。

https://ci.appveyor.com/project/sakuraeditor/sakura/build/1.0.28
--------------------------------------------------------------------------------
fatal: destination path 'C:\projects\sakura' already exists and is not an empty directory.
Command exited with code 128
--------------------------------------------------------------------------------
@kobake kobake changed the title [WIP] AppVeyorバッジ追加、AppVeyor URL修正 AppVeyorバッジ追加、AppVeyor URL修正 Jun 2, 2018
@kobake
Copy link
Member Author

kobake commented Jun 2, 2018

情報ありがとうございます!

appveyor.yml での clone_folder 指定で git clone される場所が毎回変わるようになりました。これで git clone エラーは起こらなくなるはずです。(AppVeyor に対するマナーとしてこういうやり方がアリかナシかは微妙な気もしますが。どうなんだろう……)

clone_folder: C:\projects\$(APPVEYOR_ACCOUNT_NAME)\$(APPVEYOR_PROJECT_NAME)_$(APPVEYOR_BUILD_VERSION)

@kobake
Copy link
Member Author

kobake commented Jun 2, 2018

@m-tmatma
内容問題なければマージお願いします。(AppVeyor は m-tmatma さんが詳しそうなので m-tmatma さんから見て問題なければ大丈夫そう、という判断)

@m-tmatma
Copy link
Member

m-tmatma commented Jun 2, 2018

内容問題なければマージお願いします。

https://github.com/sakura-editor/sakura/blob/appveyor-badge/README.md

を確認して、バッジが機能しているのを確認しました。

@m-tmatma m-tmatma merged commit 8b67043 into master Jun 2, 2018
@m-tmatma
Copy link
Member

m-tmatma commented Jun 2, 2018

マージ後に https://github.com/sakura-editor/sakura を見たときに バッジが Pending になっているのを確認しました。

@kobake
Copy link
Member Author

kobake commented Jun 2, 2018

build failing 表示も(たまたま)確認できました。
エラー起こったことはあまり良くないですが、一応エラー時にエラー表示が出ることも確認できて良かった(?)です。

肝心のエラー内容ですが、、
https://ci.appveyor.com/project/sakuraeditor/sakura/build/1.0.49

git clone -q --branch=master https://github.com/sakura-editor/sakura.git C:\projects\sakuraeditor\sakura_1.0.49
fatal: destination path 'C:\projects\sakuraeditor\sakura_1.0.49' already exists and is not an empty directory.
Command exited with code 128

これ以上どうすれば、、という感……

@m-tmatma
Copy link
Member

m-tmatma commented Jun 2, 2018

build failing 表示も(たまたま)確認できました。
エラー起こったことはあまり良くないですが、一応エラー時にエラー表示が出ることも確認できて良かった(?)です。

バッジは動いているので、このプルリクエストは再オープンせずに
clone 失敗の件は #43 に登録しました。

@kobake
Copy link
Member Author

kobake commented Jun 2, 2018

エラーバッジのままだと見栄えが悪いので、手動で AppVeyor ビルド再度走らせて成功状態にしておきました。

@m-tmatma m-tmatma added this to the next release milestone Jun 4, 2018
@kobake kobake deleted the appveyor-badge branch June 6, 2018 13:58
@m-tmatma m-tmatma added the CI appveyor など CI 関連 【ChangeLog除外】 label Jun 12, 2018
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019
AppVeyorバッジ追加、AppVeyor URL修正
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

Successfully merging this pull request may close these issues.

3 participants