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

sakura_lang_en_US.dll の MinGW 向け Makefile を更新する #369

Merged

Conversation

berryzplus
Copy link
Contributor

最近の修正に合わせてビルド方法を修正します。

#351 で sakura_core/Makefileに行った修正の横展開です。
#352 btool フォルダ削除 の前に実施すべき修正です。

最近の修正に合わせて依存関係を修正。
@berryzplus berryzplus changed the title langdllのMakefile更新 sakura_lang_en_US.dll の MinGW 向け Makefile を更新する Aug 26, 2018
@m-tmatma
Copy link
Member

RCTOOLDIR と RCTOOLDIR の変更も削除しては?
sakura_core\Makefile と sakura_lang_en_US\Makefile の両方に含まれています。


depend: svnrev
depend:
$(CXX) -E -MM -w $(DEFINES) $(CXXFLAGS) *.cpp */*.cpp */*/*.cpp > depend.mak
Copy link
Member

Choose a reason for hiding this comment

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

この depend のルールいりますか?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

この depend のルールいりますか?

おそらく不要ですが、cmakeが吐くMakefileに含まれています。(作る慣習がある?)
とりあえず「 svnrev に依存します」を除去するだけにとどめたいです。

@m-tmatma
Copy link
Member

RCTOOLDIR と RCTOOLDIR の変更も削除しては?
sakura_core\Makefile と sakura_lang_en_US\Makefile の両方に含まれています。

これは #371 でやるんですね。

@berryzplus
Copy link
Contributor Author

これは #371 でやるんですね。

同じ変更内容なのでまとめてやるのが良いかなと考えて分けました。

@m-tmatma m-tmatma merged commit 52f2520 into sakura-editor:master Aug 26, 2018
@berryzplus berryzplus deleted the feature/update_lang_dll_makefile branch August 26, 2018 13:31
@m-tmatma m-tmatma added this to the next release milestone Sep 9, 2018
@m-tmatma m-tmatma added the MinGW MinGW label Sep 9, 2018
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019
…ang_dll_makefile

sakura_lang_en_US.dll の MinGW 向け Makefile を更新する
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
MinGW MinGW
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants