GCC でビルドできない

ckw-mod / ckw-mod

ckw is yet another command prompt.

http://ckw-mod.github.io/

GNU General Public License v2.0

108 stars 20 forks source link

Closed wantora closed 11 years ago

wantora commented 11 years ago

(#13 を issue として再登録させていただきました。)

下記の問題があり GCC 4 でビルド出来ません。

参考として以前修正したものをおいておきます。 wantora/ckw-mod@7d9d9da9cbb0c8a9513e8bf1cea78b7dd310d46e

southly commented 11 years ago

個人的に作るときは C++ の仕様にない _s 付き関数は使わないのですが、 C11 に含まれたみたいですし、そのうち gcc でも使えるようになると期待してお茶を濁す方向で対応しようかなと思います。

southly/ckw-mod@0fb4ad7465d9db0ee47bd02cce447ca63d8523e4 (gcc 4.6.2)

southly commented 11 years ago

警告に対する対処やセキュリティの向上はこの先対応していくということで。

wantora commented 11 years ago

おお、ありがとうございます。一応こちらでも GCC 4.6.2 (mingw32) でビルドができることを確認しました。

一つ気になるのですが、GCC で STARTF_TITLEISLINKNAME の部分を無視していいのでしょうか？ GCC でビルドした時だけ問題が発生するのはまずい気がします。 (Windows 7 x64 は持っていないので確認できないのですが…)

southly commented 11 years ago

私の環境は Win7 64bit なのですが問題ありませんでした。 64bit アプリとしてビルドしたときに問題だったのかなと思っていたのですが。

wantora commented 11 years ago

なるほど。

でも、x64対応の MinGW-w64 もあるので一応対応しておいたほうがいいと思うのですが、どうでしょうか。(まあ、MinGW-w64 はサポートしない、というのもアリですが)

southly commented 11 years ago

MinGW-w64 のバイナリを TDM-GCC からダウンロードして対応しておきました。

southly/ckw-mod@0709e1eaef0fab3b5e6d674ec346803c4211f424

STARTF_TITLEISLINKNAME の部分はあってもなくても変わらないようでしたので消すのはやめにしました。

wantora commented 11 years ago

素晴らしいです。

southly commented 11 years ago

一応 MinGW-w64 でもビルドできるようにしましたが、サポートするとは言い難いです。おまけ的な位置づけだと思ってください。

southly commented 11 years ago

修正をマージしておきました。

wantora commented 11 years ago

マージありがとうございます。