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.

Sign up for GitHub

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

Jump to bottom

[x64対応] master の修正を x64 ブランチにマージする #161

Merged
merged 40 commits into from
Jun 23, 2018
Merged

[x64対応] master の修正を x64 ブランチにマージする #161

merged 40 commits into from
Jun 23, 2018

Conversation

m-tmatma
Copy link
Member

@m-tmatma m-tmatma commented Jun 22, 2018 •
edited
Loading

[x64対応] master の修正を x64 ブランチにマージする

未反映の修正が溜まって来たのでそろそろ同期する

以下のマージ後がいいと思う

beru and others added 26 commits June 10, 2018 03:12
@beru
grep 処理速度改善
e594f61
@beru
grep 処理速度改善
cd5834b
@beru
grep 処理速度改善
4fa304e 
UI確認・結果出力の時間間隔をメンバー変数から定数に変更
CMemory::AllocBuffer() における確保バッファサイズの調整記述を更新
@beru
grep 処理速度改善
282b7b2 
CMemory の _AppendSz と AppendRawData でメモリ再確保サイズを倍々にして再確保頻度を減らす対応
@beru
grep 処理速度改善
b5e4bc9 
CMemory は変更しないように元に戻す
CGrepAgent の変更コードのインデントスタイルを周りに合わせる
@berryzplus
Merge pull request #77 from beru/grep_speed_up
b03ddd7 
grep 処理速度改善 マージします。
@m-tmatma
Compiled HTML Help を appveyor でコンパイルする。
3b9ab5c 
hhc は 成功時に 1, 失敗時に 0 を返す
@kobake
Merge pull request #123 from m-tmatma/feature/appveyor-compile-chm
11cb421 
Compiled HTML Help を appveyor でコンパイルする。
@m-tmatma
#81: bregonig.dll のバイナリを追加する
92bd7a3
@m-tmatma
Merge pull request #136 from m-tmatma/feauture/add-bregonig-binary
1008fc5 
bregonig.dll のバイナリを追加する
@kobake
.bat は常に改行コード CRLF でチェックアウトされるようにする
d53f4f8
@kobake
gitrev.h 生成時に git コマンドの存在と .git ディレクトリの存在を明示的にチェック
96e0ab4
@kobake
SVN Revision 利用箇所を削除し、代わりに GitHash を用いる
38e25c2
@m-tmatma
Merge pull request #146 from kobake/githash-when-nogit
8477158 
preBuild: gitrev.h 生成時に git と .git の存在を明示的にチェック
@kobake
svnrev.h 参照コードを削除
34f4a47 
※ただしここのコンパイルが走ることはないことに注意（別のタイミングでこのブロックは削除することを検討）
@m-tmatma
Merge pull request #147 from kobake/remove-svnrev
55a7b08 
SVN Revision 利用箇所を削除し、代わりに GitHash を用いる
@kobake
GIT_COMMIT_HASH が存在しないとき、バージョン情報ダイアログにバージョン番号が表示されない問題を修正
52c6039 
さらにいうと szMsg 未初期化により実は文字化けすることがありました。それも副産物的に修正。
@kobake
バージョンダイアログのハッシュ文字列は整形せずに直接append
249ea9d
@berryzplus
Merge pull request #150 from kobake/version-dialog-without-githash
53b1d94 
git情報が存在しないとき、バージョンダイアログにバージョン番号が表示されない＆文字化けする問題を修正

マージしちゃいまーす。
@m-tmatma
appveyor で再起動したときにハングする問題の修正を試す
85d39a0 
appveyor/ci#2459
OwenMcDonnell/ProjectA@c6fb53c
@berryzplus
Merge pull request #156 from m-tmatma/feature/fix-appveyor-fail-restart
8b789e1 
appveyor で再起動したときにハングする問題の修正を試す
@m-tmatma
GitURL をバージョン情報に含める
66be2e9
@m-tmatma
インストーラのビルドバッチファイルを追加
be83f6d
@m-tmatma
インストーラのバイナリを成果物に含める
55a8921
@m-tmatma
sakura_lang_en_US.dll を追加
8426f9c
@m-tmatma
#157: GitHash が反映されない問題を修正
1f7a420
@m-tmatma m-tmatma added the x64 x64 対応 label Jun 22, 2018
@m-tmatma m-tmatma added this to the next release milestone Jun 22, 2018
@m-tmatma m-tmatma mentioned this pull request Jun 22, 2018
Closed
m-tmatma and others added 2 commits June 23, 2018 08:51
@m-tmatma
#154: 変数を小文字に変更
639cae2
@kobake
Merge pull request #159 from m-tmatma/feature/fix-sakura_lang_en_US-G…
706855d 
…itHash

sakura_lang_en_US.dll のバージョンにGitHash が含まれない
@berryzplus
Copy link
Contributor

このPRの意図はこんな感じ？

  • master → x64 ブランチ切った
  • x64 ブランチで色々修正した
  • master ブランチで色々修正した
  • master ブランチの変更 → x64 ブランチに取り込みたい　←　いまここ
  • x64 ブランチの変更 → master ブランチに取り込みたい

@kobake
Copy link
Member

kobake commented Jun 23, 2018

以下のマージ後がいいと思う

#144
#158
#159
#163

まだマージされたくないのであれば、PR名に [WIP] って付けておいてもらえると間違ってマージされる事故防げます。

個人的には x64 へ master をマージするタイミングはいつでも良いと思っていますが。

@kobake
Copy link
Member

kobake commented Jun 23, 2018

x64 に対して master をマージする際には「意図」は何も考えなくて良いというのが自分の考えです。むしろ常にマージし続けるくらいでも良いくらい。

@m-tmatma
Copy link
Member Author

インストーラ関連の修正を入れたいと思ってます。

@m-tmatma m-tmatma changed the title [x64対応] master の修正を x64 ブランチにマージする WIP [x64対応] master の修正を x64 ブランチにマージする Jun 23, 2018
@m-tmatma
Merge pull request #163 from m-tmatma/feature/refactor-build-solution
6660f3e 
solution のビルドをバッチファイルの中で行う
@m-tmatma
Merge pull request #158 from m-tmatma/feature/show-remote-URL
4abff1a 
Git remote URL をバージョン情報に含める
@m-tmatma
Merge pull request #144 from m-tmatma/feature/build-installer-on-appv…
30775de 
…eyor

appveyor でのインストーラビルド対応
@berryzplus
Copy link
Contributor

前提条件はクリアされた認識。
WIP付いてるから approve が・・・。

@m-tmatma
Copy link
Member Author

#130 も入れようと思います。

@m-tmatma m-tmatma changed the title WIP [x64対応] master の修正を x64 ブランチにマージする [x64対応] master の修正を x64 ブランチにマージする Jun 23, 2018
@m-tmatma
Copy link
Member Author

#130 で win7 での動作確認が取れて、マージされたらこれもお願いします。
WIP 外しておきます。

@m-tmatma
Copy link
Member Author

#132 も入れときます?

@kobake
Copy link
Member

kobake commented Jun 23, 2018 •
edited
Loading

#132 も入れときます?

どちらでも良いと思います。
この PR がマージされた後にまた x64 <- master の PR 作り直してマージすれば良いけなので。
（x64 <- master マージについては基本的にはレビューも何もなく機械的に approve && merge で良いと思ってます）

@m-tmatma
Copy link
Member Author

#132 も入れときます?

どちらでも良いと思います。

では後にします。

@kobake
Copy link
Member

kobake commented Jun 23, 2018

sakura-editor/management-forum#3 で書きましたが、
明日あたりのタイミングで master に対して x64 をマージすることを考えてまして、
そうなったらもう x64 に対する master マージは不要になるので何も考えなくて良くなります。

kobake
kobake approved these changes Jun 23, 2018
View reviewed changes
Copy link
Member

@kobake kobake left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

事前条件揃ったのでマージします

@kobake kobake merged commit ee9341e into x64 Jun 23, 2018
@ds14050 ds14050 added the x64 x64 対応 label Sep 18, 2018
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019
@kobake
Merge pull request sakura-editor#161 from sakura-editor/master
43810d0 
[x64対応] master の修正を x64 ブランチにマージする
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kobake kobake kobake approved these changes

Assignees
No one assigned
Labels
x64 x64 対応
Projects
None yet
Milestone
v2.4.0
Development

Successfully merging this pull request may close these issues.
6 participants
@m-tmatma @berryzplus @kobake @beru @ds14050 @KENCHjp