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

GIT_COMMIT_HASH が存在しない場合にバージョン番号が表示されなくなる箇所がある #148

Closed
kobake opened this issue Jun 19, 2018 · 6 comments
Labels
🐛bug🦋 ■バグ修正(Something isn't working)
Milestone

Comments

@kobake
Copy link
Member

kobake commented Jun 19, 2018

参考
https://github.com/sakura-editor/sakura/pull/147/files

@m-tmatma
GIT_COMMIT_HASH が定義されていないとバージョン情報自体がなくなります。
GIT_COMMIT_HASH が定義されていないとき GIT_COMMIT_HASH の以外の部分は残すべきだと思います。

@m-tmatma
Copy link
Member

#147 がマージされた。

@yoshinrt
Copy link
Contributor

yoshinrt commented Jun 30, 2018

git インストールがビルド要件でしたら無視してください.
git コマンドが利用できない条件でビルドすると .rc でエラーになるようです.

重大度レベル	コード	説明	プロジェクト	ファイル	行	抑制状態
エラー	RC2104	undefined keyword or key name: GIT_SHORT_COMMIT_HASH	sakura_lang_en_US	D:\DDS\sakura\sakura_lang_en_US\sakura_lang_rc.rc	2823	
エラー	RC2104	undefined keyword or key name: GIT_SHORT_COMMIT_HASH	sakura	D:\DDS\sakura\sakura_core\sakura_rc.rc	2821	

defdb57 で再現しました.

ご検討をよろしくお願いいたします.
(ちなみに自分は cygwin git を使用しているので,パスが通っていない状態でビルドしております)

@berryzplus
Copy link
Contributor

gitにパスを通すことはビルド要件である認識です。

@berryzplus berryzplus reopened this Jun 30, 2018
@kobake
Copy link
Member Author

kobake commented Jun 30, 2018

ビルド要件としては明示してないですが、ZIPでソース一式ダウンロードしてビルドする人もいるので、自分としては git 無くてもビルドできるようにはしておきたいと思っています。

少なくとも少し前の master では git 無しでもビルド出来ていました。

現時点の master について、ちょっとこちらでもビルド試してみますね。

@berryzplus
Copy link
Contributor

...(Closed になってるissueに重要っぽいコメントが入ってたのでreopenしました。)

閉じるタイミングはお任せします。

@kobake
Copy link
Member Author

kobake commented Jun 30, 2018

#191 で対応しました。これがマージされたらこの Isssue は再クローズします。

@ds14050 ds14050 added the 🐛bug🦋 ■バグ修正(Something isn't working) label Sep 18, 2018
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this issue Jun 11, 2019
Git情報がなくてもビルドできるようにする (Close sakura-editor#148)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛bug🦋 ■バグ修正(Something isn't working)
Projects
None yet
Development

No branches or pull requests

5 participants