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 で再起動したときにハングする問題の修正を試す #156

Merged

Conversation

m-tmatma
Copy link
Member

@m-tmatma m-tmatma commented Jun 22, 2018

appveyor で再起動したときにハングする問題の修正を試す

以下により init: の代わりに、install: を使う

appveyor/ci#2456 (comment)
For now try switching the problematic sequence to install: instead of init:

appveyor/ci#2459
OwenMcDonnell/ProjectA@c6fb53c

@m-tmatma m-tmatma added 🐛bug🦋 ■バグ修正(Something isn't working) CI appveyor など CI 関連 【ChangeLog除外】 labels Jun 22, 2018
@m-tmatma m-tmatma added this to the next release milestone Jun 22, 2018
@m-tmatma
Copy link
Member Author

ビルド通りました!
レビュープリーズ

Copy link
Contributor

@berryzplus berryzplus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTMです。
参照URLで内容は分かったけど、できればもちょっと説明プリーズw

@m-tmatma
Copy link
Member Author

説明欄に追加しました。
ただ init: の何が問題で、 install: だといいのかはわかってません。

とりあえずビルドが通るようになるのを優先しました。

@m-tmatma
Copy link
Member Author

参照URLで内容は分かったけど、できればもちょっと説明プリーズw

上記説明で問題なければマージしちゃってください。

@berryzplus
Copy link
Contributor

Initとは別にinstallというフェーズがあるみたいです。参照URLにそんな雰囲気の英文があった感じです。問題なさそうなのでマージしちゃいます。

@berryzplus berryzplus merged commit 8b789e1 into sakura-editor:master Jun 22, 2018
@m-tmatma m-tmatma deleted the feature/fix-appveyor-fail-restart branch June 22, 2018 12:10
@m-tmatma
Copy link
Member Author

Initとは別にinstallというフェーズがあるみたいです。参照URLにそんな雰囲気の英文があった感じです。

https://www.appveyor.com/docs/appveyor-yml/
のコメントに以下とある。

# scripts that are called at very beginning, before repo cloning
init:
  - git config --global core.autocrlf input

# scripts that run after cloning repository
install:
  # by default, all script lines are interpreted as batch
  - echo This is batch
  # to run script as a PowerShell command prepend it with ps:
  - ps: Write-Host 'This is PowerShell'
  # batch commands start from cmd:
  - cmd: echo This is batch again
  - cmd: set MY_VAR=12345

そもそも、clone した後に実行される install に書くのが正しそう。

以下を参考にしたがそれが間違っていたのかも。
appveyor/ci#846 (comment)

@ds14050 ds14050 added 🐛bug🦋 ■バグ修正(Something isn't working) CI appveyor など CI 関連 【ChangeLog除外】 labels Sep 18, 2018
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019
…or-fail-restart

appveyor で再起動したときにハングする問題の修正を試す
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛bug🦋 ■バグ修正(Something isn't working) CI appveyor など CI 関連 【ChangeLog除外】
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants