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 作業集約場所:2018-06-27 をもって master に統合する #86

Merged
merged 24 commits into from
Jun 26, 2018

Conversation

kobake
Copy link
Member

@kobake kobake commented Jun 10, 2018

本 PR の目的

  • x64 に集約されたコミット内容 (コミット群, master との差分等) を俯瞰するための場所です。
  • x64 対応が一通り済んだら、この PR のマージにより master に対して x64 全変更内容を適用します。

x64 対応の作業について

  • 本 PR (x64 ブランチ) に対して直接コミットを積むようなことは原則しません。
  • x64 ブランチに対してコミットを積みたい場合は x64 ブランチから派生した作業ブランチ (fork 推奨) を作成し、x64 に対する PR を作成してください。
  • fork したリポジトリで作業する場合は、作業する前にfork したリポジトリおよび x64 ブランチを最新にしてください。

fork したリポジトリを最新にする方法 (以下のサイトの説明にはないですが、最後に自分の fork リポジトリに git push する必要があります。

作業が重複しないように、着手済みの作業は分かりやすいように宣言しましょう

  • 複数人による分担作業が想定されます。作業被りを防ぐため、何らかの作業に着手する際にはそれが分かるように Issue を立てる等してください(単に Issue を立てるだけでなく、着手済みが分かるような記載をしてください。記載方法は各自おまかせでよしなにやりましょう)

x64 関連は x64 のラベルをつける

x64 x64 対応

@m-tmatma
Copy link
Member

fork したリポジトリで作業する場合の注意、参考情報を説明欄に追加しました。

@m-tmatma
Copy link
Member

着手済みが分かるような記載をしてください。

Assignees に自分を割り当てればいいのでは?

@m-tmatma
Copy link
Member

Assignees に自分を割り当てればいいのでは?

そうすれば
https://github.com/sakura-editor/sakura/issues
の画面でフィルタリングすることもできます。

こんな感じ
https://github.com/sakura-editor/sakura/issues/assigned/kobake

@kobake
Copy link
Member Author

kobake commented Jun 10, 2018

着手済みが分かるような記載をしてください。

Assignees に自分を割り当てればいいのでは?

Assignees を設定すること自体は良いと思います。

ただ、本来の Assignees の意味合いとしては着手済みかどうかというよりも
「着手する予定」「着手してもらう予定」「着手してもらいたい」「着手済み」いろんな解釈ができるので、今回は明示的に「着手済み」が分かるようにしたかったという意図があります。

ラベルで管理する選択もありますが、今回の x64 対応ぐらいでしか着手済み状態の管理ってやらなそうなので、タイトル記載で済ませられる範囲と考え、タイトル記載で済ませてます。

@m-tmatma m-tmatma added the x64 x64 対応 label Jun 10, 2018
@m-tmatma
Copy link
Member

x64 のラベルを作りました。
x64 x64 対応

kobake and others added 4 commits June 11, 2018 11:57
※ MakefileMake の x64 のビルド構成はないので表面化しないが潜在バグなので修正しておく
[x64対応] MakefileMake に対する C4477 の警告修正
x64 に対して master をマージ(定期的にやっていく)
@m-tmatma m-tmatma added this to the next release milestone Jun 12, 2018
kobake and others added 3 commits June 13, 2018 23:48
[x64対応] master の修正を x64 ブランチにマージする
x64 に対して master をマージ(定期的にやっていく)
@m-tmatma
Copy link
Member

X64 対応は完了してないですが、
x64 ブランチとmaster との同期の手間が大きいので
一旦 x64ブランチを master に マージしますか?

@kobake
Copy link
Member Author

kobake commented Jun 15, 2018

同期の手間、大きいですか?
個人的には手間には感じてません。
どのような点で手間に感じているのかご説明いただけると納得しやすいです。

理想としては x64 対応完了してからマージするのが好ましいですが、作業の支障が大きいと感じるのであれば今の時点でマージしてしまう選択も視野に入れて良いと思います。

複数プロセッサによるコンパイルを x64 でも有効にする
@m-tmatma
Copy link
Member

同期の手間、大きいですか?

x64 ブランチがマージされていれば
#103 の対応をしたあと、#108 で不足分を対応する必要がなく、一発の対応で済みました。
#124 のテストをするときも別途ブランチが別れているために x64 用対応をする必要があります。

単にマージするだけだといいのですが、追加対応が必要な場合に二度手間になります。

@kobake
Copy link
Member Author

kobake commented Jun 18, 2018

同期の手間、大きいですか?

x64 ブランチがマージされていれば
#103 の対応をしたあと、#108 で不足分を対応する必要がなく、一発の対応で済みました。
#124 のテストをするときも別途ブランチが別れているために x64 用対応をする必要があります。

単にマージするだけだといいのですが、追加対応が必要な場合に二度手間になります。

そのあたりの手間は想定していたのですが、#40 の時点の議論ではブランチ運用を推したい雰囲気を察したのでブランチ運用としていました(手間がかかることは想定の上であの時点では推しているのだと捉えていました)。僕としては master でやってもブランチでやってもどちらでも良いと思っています。

選択肢としては以下が考えられると思いますがどうでしょう(以下以外の案もあればおっしゃってください)。

  • 手間はかかるけど現状のスタイルを続行
  • x64 は対応途中であることを明示した上で今の x64 ブランチは master にマージし、x64 対応は master 上で行う
  • x64 に対するマージ条件を緩くする(approve 不要にする等)ことでマージ待ちの時間を減らす(問題ないことが自明なマージであれば自己マージを許容する)

@kobake
Copy link
Member Author

kobake commented Jun 18, 2018

↑Issue番号打ち間違えたのでコメント修正

@m-tmatma
Copy link
Member

どうでもいいですが、この PR の番号が 86 になっている (x86 を連想)

@kobake kobake changed the title [WIP] x64 作業集約場所 x64 作業集約場所:2018-06-27 をもって master に統合する Jun 26, 2018
@kobake
Copy link
Member Author

kobake commented Jun 26, 2018

現時点でのブランチ運用の是非 #3 の提案に伴い、x64 ブランチを master へ一旦マージし、以後しばらくは master で作業します。

@sakura-editor/sakura-developers
どなたか approve と merge をお願いします。
x64 ブランチ(動作検証済み)からの単純なマージですので、レビューは不要です。

マージ後のイメージは #183 となります。

仮にこのマージで後から問題が発生することがあったとしても事後対応で良いでしょう。
今のフェーズではその程度の品質で良いです。作業効率を優先しましょう。

@kobake kobake added the IMPORTANT 早急に解消すべきもの label Jun 26, 2018
@m-tmatma m-tmatma merged commit 204b7ec into master Jun 26, 2018
@m-tmatma
Copy link
Member

マージしました。

@ds14050 ds14050 added x64 x64 対応 IMPORTANT 早急に解消すべきもの labels Sep 18, 2018
@m-tmatma m-tmatma deleted the x64 branch May 12, 2019 09:58
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019
x64 作業集約場所:2018-06-27 をもって master に統合する
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
IMPORTANT 早急に解消すべきもの x64 x64 対応
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants