MinGWでサクラエディタをビルドできるようにメイクファイルを更新する #351

berryzplus · 2018-08-19T06:49:31Z

目的

最近いれた修正の影響で、Makefileを使ったビルドができなくなっています。
とりあえず、ビルドできる状態に戻します。 Issue #349

確認方法

MinGW64のビルド環境を用意する
自分は pleiades 4.6 Neon の C++ Full 版を使いました。
コマンドプロンプトで https://github.com/sakura-editor/sakura.git をクローンしたフォルダを開く
1で用意した mingw の bin フォルダを path に指定する
自分の場合 D:\eclipse4.6 に解凍してるので path=D:\eclipse4.6\eclipse\mingw\bin;%path% でした。
cd sakura_core ←フォルダ移動
mingw32-make githash stdafx sakura_rc.o ←並行ビルドできないものを先に済ませる
mingw32-make -j4 ←マルチスレッド実行を指定してビルド開始。
-j4の数字は最大プロセス数なので、CPUコアに応じて増減させます。
sakura.exe を起動していたら「全終了」で落としてください。
.\sakura.exe で MinGW 版のEXEが起動します。
現状「起動できること」以上の確認はできません。そこ以降は今後の課題。

berryzplus · 2018-08-19T07:02:54Z

追記
このMakefileはコンフィグ動作を含まないので、
git がパスに入った状態でもちゃんとビルドできます。

コミットリストに kobake さんのコミットが含まれている理由は、
#306 のコミットをベースに作業したからです。
修正の内容はコミットログに書いていますので気になる方はコミットログ見てください。

k-takata · 2018-08-19T07:23:02Z

sakura_core/util/string_ex.cpp

@@ -222,7 +222,9 @@ const char* stristr_j( const char* s1, const char* s2 )
 //                           互換                              //
 // -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- //

-#if (defined(_MSC_VER) && _MSC_VER<1400) || defined(__MINGW32__) //VS2005より前なら
+#if (defined(_MSC_VER) && _MSC_VER<1400) //VS2005より前なら
+|| (defined(__MINGW32__) && defined(MINGW_HAS_SECURE_API) && MINGW_HAS_SECURE_API)


\ での行継続が必要ですね。
あと、MinGWの判定合ってますか？

レビューありがとうございます。

\ での行継続が必要ですね。

ビルドが通ったのは _MSC_VER が定義されていなかったからですね。

あと、MinGWの判定合ってますか？

MinGW側の判定はたぶん合ってると思います。
定義するときは -DMINGW_HAS_SECURE_API=1 としますが、
0とされたときに正しく動くようにする意図です。

sec_api/tchar_s.h などを見ると、#ifdef で判定しています。
そして、MinGW自身がセキュア系関数を持っているはずです。

デバッグできるようにするために設定をいじっていてMinGWの判定式の誤りに気付きました。

#if (defined(_MSC_VER) && _MSC_VER<1400) \ || (defined(__MINGW32__) && (!defined(MINGW_HAS_SECURE_API) || MINGW_HAS_SECURE_API != 1)) //VS2005より前なら

「MINGW_HAS_SECURE_APIが未定義または1以外」が正しい条件でした。
すでにマージ済みなので別PRを投げます。

「本来あるべき」を考えると、これは「サポートしないビルド環境」を示す条件なので「#if ごと削除が妥当」と思っています。./configureやautomakeに相当するもの(cmake)が準備できるまでの暫定処置と考えています。

k-takata · 2018-08-19T07:23:26Z

sakura_core/util/string_ex.h

@@ -144,7 +144,8 @@ int my_strnicmp( const char *s1, const char *s2, size_t n );
 // -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- //

 // VS2005以降の安全版文字列関数
-#if (defined(_MSC_VER) && _MSC_VER<1400) || defined(__MINGW32__) //VS2005より前なら
+#if (defined(_MSC_VER) && _MSC_VER<1400) //VS2005より前なら
+	|| (defined(__MINGW32__) && defined(MINGW_HAS_SECURE_API) && MINGW_HAS_SECURE_API)


上と同様。

上と同様、修正します。

k-takata · 2018-08-19T07:27:14Z

sakura_core/StdAfx.h

+#  include <sec_api/tchar_s.h>
+#  include <sec_api/wchar_s.h>
+#  include <sec_api/string_s.h>
+#  include <sec_api/stdio_s.h>


これらのヘッダーファイルは tchar.h などから自動的にインクルードされるものであって、明示的にインクルードするものではないのではないと思います。VCにもあるファイルをインクルードしておけば #ifdef を使わずともVCと同じようにコンパイルできるはずですし。
_mingw.h も普通は windows.h をインクルードしておけばよいと思います。

自動インクルードされてくれる自身がなかったので明示的に書きました。
外してみてビルド通りそうなら外してしまおうと思います。

#ifdef __MINGW32___ のブロックごと外してビルドできましたので外してしまいました。

swprintf_s関数が見つからないという以下エラーの対応 io/CFileLoad.cpp:93:52: error: 'swprintf_s' was not declared in this scope swprintf_s(buf, _countof(buf), L"%I64u", megabytes);

…られる記述を訂正する rvalueの非const参照を宣言してはならないという以下エラーの対応 macro/CMacroFactory.cpp:92:37: error: invalid initialization of non-const reference of type 'std::_List_iterator<CMacroManagerBase* (*)(const wchar_t*)>&' from an rvalue of type 'std::__cxx11::list<CMacroManagerBase* (*)(const wchar_t*)>::iterator {aka std::_List_iterator<CMacroManagerBase* (*)(const wchar_t*)>}' auto& c_it = m_mMacroCreators.begin();

sakura-editor#303　CNativeA/CNativeW に対する AppendString のフォーマット書式対応版メソッドを準備以下2ファイルのコンパイルエラーの対応 mingw32-make mem/CNativeW.o mingw32-make mem/CNativeA.o

古いMinGWのWindowsSDKに含まれていないCOMのIDを独自に定義している。 GNU 6.2.0には既に取り込まれているので、定義があるとビルドエラーになる。

berryzplus · 2018-08-23T11:45:51Z

レビューをおねがいします。

m-tmatma · 2018-08-23T12:32:17Z

sakura_core/mem/CNativeA.cpp

-		sprintf_s(cbuf, _countof(cbuf), "AppendStringF error. errno = %d", e);
-		throw std::exception(cbuf);
+		DEBUG_TRACE(L"AppendStringF error. errno = %d", e);
+		throw std::exception();


これは visual studio での動作を変えてしまうのでは?

目的はビルドできるようにすることなので、コードの目的を変えない範囲で変更しています。
変更前）エラー情報を例外に詰めて投げる・・・catchするコードがないのでそのまま落ちる
変更後）エラー情報をデバッガに出力しエラーを投げる・・・catchするコードがないのでそのまま落ちる

std::exceptionのコンストラクタがconstexprでない文字列を受け付けないのが原因なので、恒久対応としては専用の例外クラスを自作することになると思っています。

チケットを作っておいてもらえますか?

#356 作りました。

m-tmatma · 2018-08-23T12:32:41Z

mingw32-make clean すると以下のエラーが出ました。

make (e=2): 指定されたファイルが見つかりません。
Makefile:451: recipe for target 'clean' failed
mingw32-make: *** [clean] Error 2

mingw32-make githash を実行した後にもう一度 mingw32-make githash すると以下のエラーが表示されます。
githash.bat は Visual studio で実行した場合やコマンドラインで実行した場合は二回実行しても
以下のエラーが表示されてないので mingw32-make から実行された場合に状況が異なっているのだと思います。

パラメーターの形式が誤っています - "sakura_core\githash.h.tmp"
../sakura_core\githash.h was not updated.

またビルド方法を markdown にまとめてほしいです。

m-tmatma · 2018-08-23T12:42:12Z

libwinpthread-1.dll のパスが通っていないと起動できない点を記載する必要があると思います。

また sakura.exe は起動できましたが、テキストの編集画面は出ませんでした。
出ましたか?

berryzplus · 2018-08-23T13:55:54Z

このPRの目的はビルドできるようにすることです。

mingw32-make clean すると以下のエラーが出ました。

ビルドできないことに対する対処なのでcleanには手を入れてません。
ぼくが知る限り、昔からそうです。
今回のPRのスコープ外です・・・簡単に直せないか試してみます。

パラメーターの形式が誤っています - "sakura_core\githash.h.tmp"
../sakura_core\githash.h was not updated.

最後の行しか見ていませんでした。
1回目 was updated と出ますが「パラメータの形式が誤ってます」も出るみたいです。
調査して対応します。

またビルド方法を markdown にまとめてほしいです。

あとでコミット追加します。準備できたら声かけますのでよろしくお願いします。

berryzplus · 2018-08-23T14:06:33Z

libwinpthread-1.dll のパスが通っていないと起動できない点を記載する必要があると思います。

このPRのスコープ外と考えていました。
markdownの説明に追記しときます。

また sakura.exe は起動できましたが、テキストの編集画面は出ませんでした。
出ましたか?

出ませんでした。
このMakefileはRelease版しか作れないので、デバッグでの原因調査もまだしていません。
このPRのスコープ外と考えています。
「とりあえずexeができる」が目標到達点です。

berryzplus · 2018-08-23T17:00:23Z

mingw32-make clean すると以下のエラーが出ました。

ビルドできないことに対する対処なのでcleanには手を入れてません。
ぼくが知る限り、昔からそうです。
今回のPRのスコープ外です・・・簡単に直せないか試してみます。

パス区切りがスラッシュになっていることが原因でした。
スラッシュを含むパスを del などのコマンドに渡すにはダブルクォートで囲う必要があります。
元々はMacからクロスコンパイルするためのMakefileのようなんですが、暫定対処としてwindowsでビルド＆クリーンができるようにしました。

m-tmatma · 2018-08-23T22:33:31Z

あとでコミット追加します。準備できたら声かけますのでよろしくお願いします。

別にチケット作ってもいいですし、
#349 の説明に項目足すのでもいいです。

m-tmatma · 2018-08-23T22:42:45Z

元々はMacからクロスコンパイルするためのMakefileのようなんですが、暫定対処としてwindowsでビルド＆クリーンができるようにしました。

直っているのは確認しましたが、贅沢をいうと
mingw32-make clean した後に mingw32-make clean すると
以下のようなメッセージが出るのも直してほしいです。

.\sakura_core\sakura.exe が見つかりませんでした。

m-tmatma · 2018-08-23T22:44:22Z

最後の行しか見ていませんでした。
1回目 was updated と出ますが「パラメータの形式が誤ってます」も出るみたいです。
調査して対応します。

直っているのを確認しました。

m-tmatma · 2018-08-23T22:45:56Z

sakura_core/Makefile

@@ -433,7 +433,7 @@ Funccode_enum.h: $(HEADERMAKE) Funccode_x.hsrc
 	$(HEADERMAKE) -in=../sakura_core/Funccode_x.hsrc -out=../sakura_core/Funccode_enum.h -mode=enum -enum=EFunctionCode

 githash:
-	cmd /c $(CURDIR)/../sakura/githash.bat ../sakura_core
+	cmd /c $(CURDIR)/../sakura/githash.bat ..\\sakura_core


githash.bat の中で / を置換する方法でもいいと思います。

m-tmatma · 2018-08-23T22:46:17Z

またビルド方法を markdown にまとめてほしいです。

確認しました。

berryzplus · 2018-08-24T14:15:10Z

mingw32-make clean した後に mingw32-make clean すると
以下のようなメッセージが出るのも直してほしいです。
.\sakura_core\sakura.exe が見つかりませんでした。

対応しました。

delコマンドが出しているメッセージでした。
削除対象のファイル群をまとめてdelに渡していたのを、1ファイルずつ存在確認しながら削除するように書き換えました。ファイル削除の体感速度はあまり変わってないように思います。（変更内容からすると遅くなるはず・・・。）

m-tmatma · 2018-08-24T21:20:26Z

対応しました。

確認しました。

これもスコープ外なのかもしれませんが、
mingw32-make clean しても以下ファイルは削除されませんでした。

sakura_core\Funccode_define.h
sakura_core\Funccode_enum.h
sakura_core\githash.h

berryzplus · 2018-08-24T23:27:10Z

これもスコープ外なのかもしれませんが、
mingw32-make clean しても以下ファイルは削除されませんでした。
•sakura_core\Funccode_define.h
•sakura_core\Funccode_enum.h
•sakura_core\githash.h

必要だと思ったので対応しました。
VC++側プロジェクト設定が上記ファイルの削除に未対応なのは別件としたいです。

m-tmatma · 2018-08-25T06:49:26Z

必要だと思ったので対応しました。
VC++側プロジェクト設定が上記ファイルの削除に未対応なのは別件としたいです。

確認しました。

…ingw_makefile MinGWでサクラエディタをビルドできるようにメイクファイルを更新する

This was referenced Aug 19, 2018

Makefile調整：入力ファイルUTF-8指定、GitHash出力 #306

Closed

btool 配下のソースコードを UTF-8 (BOM付き) に変換 #304

Closed

btoolフォルダを削除する #352

Merged

k-takata reviewed Aug 19, 2018

View reviewed changes

berryzplus mentioned this pull request Aug 19, 2018

appveyor で MingW ビルドする #309

Closed

kobake and others added 15 commits August 20, 2018 20:33

入力ファイルのエンコーディング指定変更: cp932 -> utf-8

fcc9538

svnrev 出力関連を githash 出力に差し替え

f92466c

-fexec-charset は utf-8 ではなく cp932 に戻す

519bf89

sakura_rc ビルドを UTF-8 指定に変更 (RCTOOLは用いない)

28805ba

svnrev⇒githashの横展開

6861472

sakura_rc.rcのutf8化対応

300bc7c

プリコンパイルヘッダーの依存関係を修正する

5b8576d

exeとcppの依存関係を更新

8b7ba73

MinGWでセキュリティ強化版CRT関数を使えるようにする

c9f59d8

swprintf_s関数が見つからないという以下エラーの対応 io/CFileLoad.cpp:93:52: error: 'swprintf_s' was not declared in this scope swprintf_s(buf, _countof(buf), L"%I64u", megabytes);

暗黙のnarrow_castがgccに怒られる問題に対する対処

25dd45b

古いMinGWに含まれていないCOMのIDを除去

11c5ff3

古いMinGWのWindowsSDKに含まれていないCOMのIDを独自に定義している。 GNU 6.2.0には既に取り込まれているので、定義があるとビルドエラーになる。

レビュー指摘対応 (#if ディレクティブの改行には \ を付ける)

98ac13c

レビュー指摘対応 (自動インクルードされるファイルは明示でインクルードしなくてもいい)

8789964

berryzplus force-pushed the feature/update_mingw_makefile branch from 8b9e422 to 8789964 Compare August 20, 2018 11:44

m-tmatma reviewed Aug 23, 2018

View reviewed changes

berryzplus changed the title ~~MinGWでサクラエディタをビルドできるようにメイクファイルを更新する~~ WIP MinGWでサクラエディタをビルドできるようにメイクファイルを更新する Aug 23, 2018

レビュー指摘対応（パラメータが正しくありません。と出ている）

2d45280

berryzplus added 2 commits August 24, 2018 00:46

mingw32-make cleanできるようにする

e1ee546

MinGWビルドについて追記

b78aec4

berryzplus changed the title ~~WIP MinGWでサクラエディタをビルドできるようにメイクファイルを更新する~~ MinGWでサクラエディタをビルドできるようにメイクファイルを更新する Aug 23, 2018

m-tmatma reviewed Aug 23, 2018

View reviewed changes

mingw32-make clean時のcould not found fileを抑制する

9a3e316

berryzplus mentioned this pull request Aug 24, 2018

独自例外を定義して使うようにしたい #356

Closed

berryzplus mentioned this pull request Aug 24, 2018

MinGW向けメイクファイルでサクラエディタをビルドできるようにする #349

Closed

9 tasks

ビルドで生成されるファイルをmake cleanの対象に入れる

c737e5d

berryzplus mentioned this pull request Aug 24, 2018

ビルドで生成されるファイルをクリーンで削除するようにしたい #358

Closed

m-tmatma approved these changes Aug 25, 2018

View reviewed changes

berryzplus merged commit f3825cf into sakura-editor:master Aug 25, 2018

berryzplus deleted the feature/update_mingw_makefile branch August 25, 2018 07:19

berryzplus mentioned this pull request Aug 25, 2018

MINGW_HAS_SECURE_APIが無効か判定する式の修正 #363

Merged

m-tmatma added the MinGW MinGW label Aug 25, 2018

This was referenced Aug 26, 2018

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

Merged

MakefileMake を #18 の変更に対応させる #39

Closed

m-tmatma added this to the next release milestone Sep 10, 2018

ds14050 added the MinGW MinGW label Sep 18, 2018

HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019

Merge pull request sakura-editor#351 from berryzplus/feature/update_m…

c343e02

…ingw_makefile MinGWでサクラエディタをビルドできるようにメイクファイルを更新する

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

MinGWでサクラエディタをビルドできるようにメイクファイルを更新する #351

MinGWでサクラエディタをビルドできるようにメイクファイルを更新する #351

berryzplus commented Aug 19, 2018

berryzplus commented Aug 19, 2018

k-takata Aug 19, 2018

berryzplus Aug 19, 2018

k-takata Aug 19, 2018

berryzplus Aug 25, 2018

k-takata Aug 19, 2018

berryzplus Aug 19, 2018

k-takata Aug 19, 2018

berryzplus Aug 19, 2018

berryzplus Aug 19, 2018

berryzplus commented Aug 23, 2018

m-tmatma Aug 23, 2018

berryzplus Aug 23, 2018 •

edited

Loading

m-tmatma Aug 23, 2018

berryzplus Aug 24, 2018

m-tmatma commented Aug 23, 2018

m-tmatma commented Aug 23, 2018

berryzplus commented Aug 23, 2018

berryzplus commented Aug 23, 2018

berryzplus commented Aug 23, 2018

m-tmatma commented Aug 23, 2018

m-tmatma commented Aug 23, 2018 •

edited

Loading

m-tmatma commented Aug 23, 2018

m-tmatma Aug 23, 2018

m-tmatma commented Aug 23, 2018

berryzplus commented Aug 24, 2018

m-tmatma commented Aug 24, 2018

berryzplus commented Aug 24, 2018

m-tmatma commented Aug 25, 2018

MinGWでサクラエディタをビルドできるようにメイクファイルを更新する #351

MinGWでサクラエディタをビルドできるようにメイクファイルを更新する #351

Conversation

berryzplus commented Aug 19, 2018

目的

確認方法

berryzplus commented Aug 19, 2018

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

berryzplus commented Aug 23, 2018

Choose a reason for hiding this comment

berryzplus Aug 23, 2018 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

m-tmatma commented Aug 23, 2018

m-tmatma commented Aug 23, 2018

berryzplus commented Aug 23, 2018

berryzplus commented Aug 23, 2018

berryzplus commented Aug 23, 2018

m-tmatma commented Aug 23, 2018

m-tmatma commented Aug 23, 2018 • edited Loading

m-tmatma commented Aug 23, 2018

Choose a reason for hiding this comment

m-tmatma commented Aug 23, 2018

berryzplus commented Aug 24, 2018

m-tmatma commented Aug 24, 2018

berryzplus commented Aug 24, 2018

m-tmatma commented Aug 25, 2018

berryzplus Aug 23, 2018 •

edited

Loading

m-tmatma commented Aug 23, 2018 •

edited

Loading