gmockのターゲットを参照するのをやめる

[berryzplus]

gtestのパッケージ版を利用するに修正したが、
gmockのパッケージ版は現時点で利用できない。

gmockの利用を開始する時点でライブラリをどうするか考えればよいので、
一旦ターゲット参照を削り、gmockを利用していないことが分かるようにする。

#899 より切り出して作成。

#894 で行ったMinGW版でGTestのパッケージ版を利用するようにした修正の改善。

[beru]

特に問題無いと思います。

[ds14050]

#899 に push しました、gmock をソースからビルドするのを省略する方法も考えてください。gmock 外しを決定したならただの時間食いです。

[berryzplus]

#899 に push しました、gmock をソースからビルドするのを省略する方法も考えてください。gmock 外しを決定したならただの時間食いです。

• PR #894 の不備・不整合を正します。 #899 の目的が見えなくなりました。
• gmock をソースからビルドするのを省略する方法も考えてください。 > なんで？
• gmock 外しを決定したなら > 決めるのはぼくじゃないです。「誰かがこうしよう！」と言ったものに同意が付いたらそれが流れになります。今回も「一旦外そう！」に「いいんじゃね」とレスが付いただけです。
• ただの時間食いです。> ビルド時間をもっと削れる方法に気付いちまったぜ、ﾋｬｯﾊｰ と改善提案を行うならスゴイ奴だなぁと素直に思えると思います。

なにがいかんのか分からなくなってきたので @beru さんには申し訳ないですが、このPRのマージは一旦保留としたいと思います。

[ds14050]

#899 は具体的手段の参照先として提示したのみです。

gmock をソースからビルドするのを省略する方法も考えてください。 > なんで？

意味がないからです。もはや gmock はサクラエディタのテストの一部でも道具でもありません。

gmock 外しを決定したなら >決めるのはぼくじゃないです。

@m-tmatma さんの意向が確認できていないのが気がかりですが、@beru さんとの間で同意しました。

[beru]

tests/googletest/CMakeLists.txt を見る限りでは BUILD_GMOCK を OFF にすれば外せそうな感じはありますね。

まぁそれは別としてこのPRを先にマージしても良いんじゃないでしょうか。

[ds14050]

まぁそれは別としてこのPRを先にマージしても良いんじゃないでしょうか。

構いませんが #899 に BUILD_GMOCK に関する記述が rebase 後も残ります。それも自分は構いませんが。

[ds14050]

再度引用。

gmock 外しを決定したなら >決めるのはぼくじゃないです。

PR #894 を master にマージしたときからなし崩しにそういう状態になっています。誰が何をいまさら言っているのか、です。

[beru]

まぁそれは別としてこのPRを先にマージしても良いんじゃないでしょうか。

構いませんが #899 に BUILD_GMOCK に関する記述が rebase 後も残ります。それも自分は構いませんが。

必要だと判断した変更内容ならそれを残すのは良いんじゃないかと思いますよ。

gmock のターゲットを参照するのを止めるのも、BUILD_GMOCK オプションを OFF にするのも、まぁやっておくのが良いんじゃないかな、程度でやりたい人がやれば良いかと思います。ちゃんと早めに対処しておかないと大変な不都合が起きる、という問題では無いと思うので。

[ds14050]

必要だと判断した変更内容ならそれを残すのは良いんじゃないかと思いますよ。

別件だ別件だと言う人がいますので……。PR のタイトルを変えた今は違うかもしれませんが。

[m-tmatma]

この修正って単にソリューションフォルダを設定しないだけのように思いますが。

[m-tmatma]

https://cmake.org/cmake/help/v3.0/prop_tgt/FOLDER.html

[ds14050]

この修正って単にソリューションフォルダを設定しないだけのように思いますが。

はいそれは知っています。>#899 (comment)

しかし @berryzplus さんなりの言い分があるらしいので好きにやってもらっているところです。

[ds14050]

#899 の目的が見えなくなりました。

全体像が見えていなかったりオブラートに包んだりしていてそれほど明らかではなかったかもしれませんが、#899 の立脚点を露悪的に書くとこうです。「#894 はクソで問題を引き起こしてるから、クソにパッチを当てるか、クソを master から取り除くか、今すぐ選べ！」

クソの内容はといえば

1. テストプログラムが特定環境でしか作成できない。
2. gmock 外しという目的外の大きな変更が入っているが、それに適応した修正が入っていないために歪みと無駄を生んでいる。

です。この PR がスピンオフした経緯は #899 (comment) にまとめられています。それに対する自分の考えはこれです> #899 (comment)

良い結果になると思わなかったので言わずにきましたが、かばわれている本人がそれを理解しないのだからわかるように命令します。

#894 は gmock 外しという目的外の大きな変更が混ぜ込まれているので、まずはそれを revert し、Issue を立ててメンバーの意向を確認しなさい。その後 gmock 外しの対応をしてからパッケージを導入するか、gmock のパッケージを用意するかした、#894 の後継 PR を作成しなさい。自分の発言に対して責任を持ち一貫した行動を見せるためにそうしなさい。この PR を #899 から分けなければいけないと主張するなら、事の軽重を考えて当然そうしなければいけません。

[ds14050]

相変わらずの「だんまり」かどうか、すこし待ちます。

[ds14050]

この @berryzplus の行動を他のメンバーが是とするのか、こういう振る舞いがまかり通るのか、ということも見ています。

[berryzplus]

#901 (comment)
一緒に作っていく気持ちがなくなったならプロジェクトを去ればよいと思います。

いままでありがとう。

[KENCHjp]

このプロジェクトというか私は、追い出すことも引き留めることも、破壊的行為が伴っていなければ行うことは毛頭ありませんが、IssueやPR内で、意見交換ではない人格などにかかわるやり取りが散見されるのはリソースの無駄かと思いますので自重していただければと思います。
横やりでしたすいません。

[ds14050]

@KENCHjp さんに見放されるとつらいものがありますが、故のない感情的発言ではなく行動に対する評価ですから、取り下げることはしませんし自重することもできなかったでしょう。不条理がまかり通る不健全な場では正気が保てない人間です。危機感を共有できる人間がいなかったのが残念です。

[ds14050]

盗みをはたらく者とそれを泥棒と呼ぶ者がいて、「そんな風に言うもんじゃありません」とたしなめることは問題を解決しないと思うんですよ。

[KENCHjp]

見放される

言葉を選んだつもりでしたがうまく行かないものですね。
特定のだれかを見放そうなんて意図はございません。
#899 も中身危ういところはありつつもまだ建設的に意見交換されているかとおもいました。
中で berryzplus さんも書かれてましたが、人間完璧ではないし変われない生き物ではないと。
sf.netが何が問題だったかわかってませんが、相手への強要や否定ではなく問題点の提示と、あれば対案の提示などの建設的なやり取りを希望しています。
もう十分やってきて疲れたとおっしゃってるのかもしれませんが、ここは義務ではなく皆がそれぞれリアル生活ののりしろの中で活動していると思うので、ひと呼吸おきつつまた建設的に事を勧めてほしいと願う次第です。

何が泥棒的な行為なのか、それが元でどんな影響があるのか、解決するにはどうすればいいのか。。。

[KENCHjp]

ああ、長い。

m(… . …)m

[berryzplus]

こちらとしては全力でサポートを続けてきたつもりです。

@ds14050 さんの鋭い分析力は高く評価していて、出来ることならこれからもサクラエディタに関わって欲しいと思っています。

先のコメントは、もう仲間としてやって行くつもりがなくなったと判断して書きました。とても残念に思っています。