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

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

Closed
m-tmatma opened this issue Jun 22, 2018 · 7 comments
Closed
Labels
CI appveyor など CI 関連 【ChangeLog除外】 document ドキュメント x64 x64 対応
Milestone

Comments

@m-tmatma
Copy link
Member

m-tmatma commented Jun 22, 2018

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

sakura-editor/management-forum#3 (comment)

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

sakura-editor/management-forum#3 (comment)

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

sakura-editor/management-forum#3 (comment)

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

@m-tmatma m-tmatma added x64 x64 対応 document ドキュメント CI appveyor など CI 関連 【ChangeLog除外】 labels Jun 22, 2018
@m-tmatma m-tmatma added this to the next release milestone Jun 22, 2018
@m-tmatma
Copy link
Member Author

PR は以下の 2つに分ける

PR1個目

  • x64 用の成果物のzipファイル名、zip 内部のフォルダ名に alpha とかいう文字列を入れる
  • x64 用の成果物のzipの中に x64 版が alpha 版であることがわかるような注意喚起のテキストファイルを入れておく。
  • README に「x64 版はまだ未完成である」ことを注記として入れる

PR2個目

  • バージョン情報(タイトルバー表示も含む)にも alpha って入れる
  • x64 かどうかで条件コンパイル切らないで別のマクロ (alpha と入れるかどうか)で切り替えてx64 のときに有効にする
  • #pragma で x64 ビルド時(alpha のマクロ定義時) に警告を出す。(appveyor のためにメッセージは英語とする)

@m-tmatma
Copy link
Member Author

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

@m-tmatma
Copy link
Member Author

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

こんな感じを想定

alpha-test.patch.txt

@m-tmatma
Copy link
Member Author

PR1個目

#173 で対応

@m-tmatma
Copy link
Member Author

PR2個目

#174 で対応

@m-tmatma
Copy link
Member Author

PR2個目

#174 で対応

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

@m-tmatma
Copy link
Member Author

#182 がマージされた。

berryzplus pushed a commit to berryzplus/sakura-editor that referenced this issue Jul 5, 2018
@ds14050 ds14050 added x64 x64 対応 document ドキュメント CI appveyor など CI 関連 【ChangeLog除外】 labels Sep 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI appveyor など CI 関連 【ChangeLog除外】 document ドキュメント x64 x64 対応
Projects
None yet
Development

No branches or pull requests

2 participants