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

githash.bat のエラー耐性を上げる #360

Merged
merged 3 commits into from
Aug 25, 2018

Conversation

m-tmatma
Copy link
Member

@m-tmatma m-tmatma commented Aug 25, 2018

githash.bat のエラー耐性を上げる

  • 引数に / が含まれていても動くようにする
  • カレントディレクトリが意図していない場所になっていても動作するようにする

- 引数に / が含まれていても動くようにする
- カレントディレクトリが意図していない場所になっていても動作するようにする
@m-tmatma m-tmatma added this to the next release milestone Aug 25, 2018
@m-tmatma m-tmatma changed the title githash.bat のエラー耐性を改善する githash.bat のエラー耐性を上げる Aug 25, 2018

@echo.
@echo ---- Make githash.h ----

@rem ensure to be in the proper directory
pushd %~dp0
Copy link
Contributor

@berryzplus berryzplus Aug 25, 2018

Choose a reason for hiding this comment

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

%~dp0をダブルクォートで囲うべきだと思います。

ないとは思いますが C:\Documents and Settings\berryzplus\My Documents\sakuraにクローンした場合、
pushd C:\Documents と解釈されてエラーになる余地があります。

@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.

修正しました。

確認しました。

@m-tmatma m-tmatma merged commit 5aef243 into sakura-editor:master Aug 25, 2018
@m-tmatma m-tmatma deleted the feature/improve-githash-bat branch August 25, 2018 13:18
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019
…thash-bat

githash.bat のエラー耐性を上げる
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants