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 版でバージョン情報にアルファ版の表示を行う #174

Closed

Conversation

m-tmatma
Copy link
Member

#162: x64 版でバージョン情報にアルファ版の表示を行う

実際に x64 で効果があることを確認するためにはローカルで
upstream の x64 ブランチをマージする必要があります。

@m-tmatma m-tmatma added the x64 x64 対応 label Jun 23, 2018
@m-tmatma m-tmatma added this to the next release milestone Jun 23, 2018
@kobake
Copy link
Member

kobake commented Jun 24, 2018

ここで確認しました。
https://github.com/kobake/sakura-test/pull/2

32bit

64bit

  ----------------------------------------------------------------------------------------
  ---  This is an alpha version and under development. Be careful to use this version. ---
  ----------------------------------------------------------------------------------------

コメント

細かい点ですが、以下のような表示にしたい気持ちがあります。この PR でやるか別 PR でやるかはどちらでも良いですが。

  • 32bit 版 … Ver. 2.3.2.0 Ver. 2.3.2.0 (32bit)
  • 64bit 版 … Ver. 2.3.2.0(Alpha Version)Ver. 2.3.2.0 (64bit Alpha Version)

@m-tmatma
Copy link
Member Author

#90 を再調整した PR でやるのが良いと思います。

@m-tmatma
Copy link
Member Author

x64 ブランチを master にマージすると
広く利用される可能性があるので
ユーザーがアルファ版と認識できれば
とりあえずは ok と思ってます。

@m-tmatma
Copy link
Member Author

あと x64 対応の為にインストーラの対応が要りますね。

@kobake
Copy link
Member

kobake commented Jun 24, 2018

自分が使っているソフトの版が 32bit か 64bit かは確認できるようにしておきたいです。
32bit alpha というものが出る可能性もありますし。

#90 は今コンフリクト解消してますけど、ちょっとこの解消作業が発生してるの理不尽ですね・・・
あのPR、特にあの時点ではリジェクトされる理由なかったと思うのですが・・・

@m-tmatma
Copy link
Member Author

32bit 版 … Ver. 2.3.2.0 → Ver. 2.3.2.0 (32bit)
64bit 版 … Ver. 2.3.2.0(Alpha Version) → Ver. 2.3.2.0 (64bit Alpha Version)

#179 の形式とは異なりますが、
具体的にどのようなものがいいかというのはありますか?

@m-tmatma
Copy link
Member Author

#90 は文字コード変換が無ければ、特に問題はありませんでした。

@kobake
Copy link
Member

kobake commented Jun 25, 2018

#90 はかなり前に文字コード変換抜いた版をプッシュしていました

@kobake
Copy link
Member

kobake commented Jun 25, 2018

32bit 版 … Ver. 2.3.2.0 → Ver. 2.3.2.0 (32bit)
64bit 版 … Ver. 2.3.2.0(Alpha Version) → Ver. 2.3.2.0 (64bit Alpha Version)

#179 の形式とは異なりますが、
具体的にどのようなものがいいかというのはありますか?

#179 で bit 数が入るのであればそれで良いです。
あと ( の前には空白が入っていて欲しい、というのがありましたが、これは細かい点なので入ってなくてもマージすると思います。
それ以外は特にこだわりないです。

@m-tmatma
Copy link
Member Author

#179 マージしました。

@kobake
Copy link
Member

kobake commented Jun 25, 2018

x64 と master の統合前にこの PR をマージしたい (sakura-editor/management-forum#3 (comment)) ということでしたら、この PR が x64 とコンフリクトしないように調整いただきたいです。

@kobake
Copy link
Member

kobake commented Jun 25, 2018

おそらく master に対する PR としていることが選択として間違っていて、これは x64 に対する PR であるべきだったと思います。master 統合後のマージでも良いという話であれば master 宛ての PR でも良かったのですが。

@m-tmatma m-tmatma changed the title x64 版でバージョン情報にアルファ版の表示を行う [ボツ] x64 版でバージョン情報にアルファ版の表示を行う Jun 25, 2018
@m-tmatma
Copy link
Member Author

この PR は捨てて #182 を作成しました。

@m-tmatma m-tmatma closed this Jun 25, 2018
@m-tmatma m-tmatma deleted the feature/add-alpha-about-dialog branch July 8, 2018 22:48
@ds14050 ds14050 added the x64 x64 対応 label Sep 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
x64 x64 対応
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants