sakura-editor / sakura

SAKURA Editor (Japanese text editor for MS Windows)
https://sakura-editor.github.io/
Other
1.25k stars 163 forks source link

[x64] x64 版が alpha 版であることがわかるようにする #162

Closed m-tmatma closed 6 years ago

m-tmatma commented 6 years ago

x64 版が alpha 版であることがわかるようにする

https://github.com/sakura-editor/management-forum/issues/3#issuecomment-399272193

x64 用の成果物のzipファイル名、zip 内部のフォルダ名に alpha とかいう文字列を
入れるといいと思います。

https://github.com/sakura-editor/management-forum/issues/3#issuecomment-399296734

git clone してきて自前ビルドする人が x64 を選んでビルドする可能性もあるので、
そのあたりは README に「x64 版はまだ未完成である」ことを注記として入れる
のに加えて、#pragma で x64 ビルド時メッセージにも「x64 版はまだ未完成である」
ことが分かるような Warning を組み込む等の対応を自分は考えています。
(現状は既存コードの Warning が出まくりますけど、それらの Warning を
ぜんぶ解消したとしてもまだ x64 対応完了というわけではないはずなので、
対応完了までは明示的な Warning を組み込んでおくべきと考えている)
あとバージョン情報(タイトルバー表示も含む)にも alpha って入れたいですね。

https://github.com/sakura-editor/management-forum/issues/3#issuecomment-399311108

x64 かどうかで条件コンパイル切らないで
別のマクロ (alpha と入れるかどうか) を用意しておいて、それを x64 のときに
有効にするのが良いと思います。そうすることで必要無くなったら簡単に削除
できるようにできます。
m-tmatma commented 6 years ago

PR は以下の 2つに分ける

PR1個目

PR2個目

m-tmatma commented 6 years ago

161 の後で着手するのがいいと思う。

m-tmatma commented 6 years ago

x64 用の成果物のzipファイル名、zip 内部のフォルダ名に alpha とかいう文字列を入れる

こんな感じを想定

alpha-test.patch.txt

m-tmatma commented 6 years ago

PR1個目

173 で対応

m-tmatma commented 6 years ago

PR2個目

174 で対応

m-tmatma commented 6 years ago

PR2個目

174 で対応

174 は捨てて、 #182 を作成しました。

m-tmatma commented 6 years ago

182 がマージされた。