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

Revert "patchunicode 1006 と 1047 の取り込み" #745

Closed
wants to merge 1 commit into from

Conversation

ds14050
Copy link
Contributor

@ds14050 ds14050 commented Jan 3, 2019

Reverts #738

#738 はコミット著者に架空の(他人の?)メールアドレスが記録されているため一旦リバートし、正しい著者アドレスを付けた同じ変更を #744 で再コミットする予定です。

@beru
Copy link
Contributor

beru commented Jan 3, 2019

修正コミットを上乗せで積んでも結局過去のコミットはそのままでそこに付いてるメアドは変わらないので意味無いと思います。

@beru
Copy link
Contributor

beru commented Jan 3, 2019

Reverts #738

#738 はコミット著者に架空の(他人の?)メールアドレスが記録されているため一旦リバートし、正しい著者アドレスを付けた同じ変更を #744 で再コミットする予定です。

モチベが良く分かりませんが複数のPRにわざわざ分ける意味も無いと思います。

何らかのこだわり?で履歴を直したいなら owner の方に一時的に設定を変えて force push 操作をしてもらうように依頼するのが良いと思います。

メアドくらいあんまり気にする意味無いと思うんですけどね…。

@m-tmatma
Copy link
Member

m-tmatma commented Jan 3, 2019

何らかのこだわり?で履歴を直したいなら owner の方に一時的に設定を変えて force push 操作をしてもらうように依頼するのが良いと思います。

マージ後の master (991c72b) をベースに #742 を作成しているので、force push は避けたいです。

@m-tmatma
Copy link
Member

m-tmatma commented Jan 3, 2019

メアドくらいあんまり気にする意味無いと思うんですけどね…。

それは、patch を作成された原作者に対する礼儀でやっているのだと思います。

@m-tmatma
Copy link
Member

m-tmatma commented Jan 3, 2019

#738 適用前 991c72ba1824c815b1c6027c23fe06bf95e29191~ (9e4d9ef)
と この PR 適用後 2d81025sakura_core\cmd\CViewCommander_Cursor.cpp
が一致するのを確認しました。

@beru
Copy link
Contributor

beru commented Jan 3, 2019

それは、patch を作成された原作者に対する礼儀でやっているのだと思います。

変更内容の大元の PatchUnicode #1006 の Creator が ds14050 さん本人なので、その分については過去の自分に対するリスペクト(ちょっと何言ってるかわからない)ととして、引退されたMocaさんに対する礼儀という事でしょうか?

@m-tmatma
Copy link
Member

m-tmatma commented Jan 3, 2019

作者が違う人というつもりで書きました。
違うのですか?

@m-tmatma
Copy link
Member

m-tmatma commented Jan 3, 2019

パッチを作ったのが、自分自身ならこだわらなくていいと思います。

@beru
Copy link
Contributor

beru commented Jan 3, 2019

取り消し対象の #738コミット2つですが、
cf22a4b は ds14050 さん本人が作られた Patch Unicode #1006 の分で、 c718cfe は 引退されたらしいMoca さんが作られた Patch Unicode #1047 の分です。

@ds14050
Copy link
Contributor Author

ds14050 commented Jan 4, 2019

メアドくらいあんまり気にする意味無いと思うんですけどね…。

それは、patch を作成された原作者に対する礼儀でやっているのだと思います。

メアドが Git における身分証明(簡易版)だから「くらい」ではないと思うんです。コミット改竄を考えていたときは ~@mail.sf.net という専有していないメールアドレスに対するリファレンスで他人を驚かせる可能性も考えていました。

複数のPRにわざわざ分ける意味も無いと思います。

これには一晩経って気がつきました。

:::「Merge pull request #738 from beru/patchunicode_1006_1047」から派生する「revert_and_recommit_pr738 」ブランチを作成して
git co -b revert_and_recommit_pr738 991c72ba1824c815b1c6027c23fe06bf95e29191
::: PR #738 の2番目のコミットをリバート
git revert c718cfec50e983a2e8a27c1e6a70f01674ff2c0b
::: PR #738 の1番目のコミットをリバート
git revert cf22a4b04a12fae4f787e693b65c746225addcff
:::: PR #744 「リバート予定の #738 の再コミットバージョンです」の1番目のコミットをピックアップ
git cherry-pick fffb5b02916a83a6b1574a242f16996503a3c234
:::: PR #744 「リバート予定の #738 の再コミットバージョンです」の2番目のコミットをピックアップ
git cherry-pick d157ecf5826d3c9de3173a43872221255fe693b7

するところまでやりましたが、ここでモチベーションが尽きました。

ブランチを作り直しても直さなくてももう一度他者の手を借りないと一連の作業が完了しませんし、自分で書きましたがメールアドレスはあくまでも簡易な識別情報でした。

内容を確認して Approve してくださった @m-tmatma さんには PR を引っ込めることもまた失礼なことですが、確認が不十分なまま #738 を Approve した後悔を引きずってこれは取り下げます。

@ds14050 ds14050 closed this Jan 4, 2019
@k-takata k-takata deleted the revert-738-patchunicode_1006_1047 branch January 4, 2019 03:54
@k-takata
Copy link
Member

k-takata commented Jan 4, 2019

取り下げとのことですので、ブランチは消しておきました。

@ds14050
Copy link
Contributor Author

ds14050 commented Jan 4, 2019

ありがとうございます。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants