quick-junit icon indicating copy to clipboard operation
quick-junit copied to clipboard

Cleanup build environment

Open vmi opened this issue 10 years ago • 2 comments

I want to fix Quick JUnit, however I encountered many problems until I could build it. I cleaned up the build environment. Please merge my fix.

Quick JUnitでいくつか修正したい点があったのですが、ビルドできるようになるまで、いろいろとトラブりました。 その結果を切り出してビルド環境をクリーンアップしましたので、マージいただければ幸いです。

vmi avatar Jul 27 '14 04:07 vmi

@vmi pull requestありがとうございます。基本的には修正を取りこんで良さそうな変更に思えましたが 、下記の点が気になっています。

  • org.junit4 -> org.junitへの変更(旧Eclipseをtarget platformにした時、テストがビルドエラーになる)
  • 現在のTychoのデザインは、mavenに合わせて大丈夫か?

それぞれについてコメントをします。

org.junit4->org.junitへの変更

Eclipse Lunaで上記の変更が行われましたが、この変更を取り込むと過去のEclipseではjunit3を利用することになるので、ビルドできません。たぶんこの修正以外は取り込まなくても、とりあえず開発はできそうな気がします。(Warningは残りますが。)

Eclipse Lunaを今回は開発環境として利用したいのでしょうか。target platformとしたいのでしょうか。 target platformというのは、Quick JUnitを動かしたい環境を指します。現在は3.8(Indigo)以降のEclipseをサポートしたいです。

現在のTychoにはtarget-platformのプロジェクトを作成することもできるようです。 開発環境のEclipseをtarget platformにせず、IndigoのEclipseを使うように調整いただけないでしょうか? (target platformがよくわからないようであれば、こちらで調整しますので、コメントを返答し、このまま開発を進めてください。)

現在のm2eやTychoのデザインは、mavenの流儀に合わせて大丈夫か?

.project等のファイルを削除し、.gitignoreに追加されています。以前はこれらのプロジェクトファイルを削除していると、cloneした後にインポートするとき等、調子が悪いことが多々ありました。

JDTチームのプロジェクトを確認しましたが、pom.xmlと共に.projectファイル等を残したままです。 なので、この変更はまだ取り込まない方がよいように思います。いかがでしょうか?

kompiro avatar Jul 27 '14 06:07 kompiro

org.junit4 -> org.junitへの変更(旧Eclipseをtarget platformにした時、テストがビルドエラーになる)

私がプラグインの開発を知らないせいかもしれませんが、Lunaでプロジェクトを取り込んだときに該当部分がエラーになって先に進めませんでした。 何とかならないかちょっと調べてみます。 設定変更(エラー/警告レベルを落とすなど)だけでよければ、その旨ドキュメントに注記する対処でよいと思います。

現在のm2eやTychoのデザインは、mavenの流儀に合わせて大丈夫か?

インポートを必ず「既存Mavenプロジェクト」で実施すれば、必要なファイルは全部自動生成されると認識していて、今のところこれで不具合は経験していません。 ローカルな設定変更/環境変更なのに、プロジェクトファイルがいろいろ更新されて差分が出まくるので、一部の設定は残すにしても必要最小限にしたいところです。個別のファイルに対する意見としては、

  • .projectと.classpathはできるだけ消したい。これらは「Maven」→「プロジェクトの更新」ですぐ差分だらけになる上に、個人環境に依存した形での差異が出やすい。
  • .settings以下については、
    • org.eclipse.core.resources.prefs:主にエンコーディング設定のようなので、残してもよさそう。
    • org.eclipse.m2e.core.prefs:これも自動生成の対象のはずなので、消した方がよいのでは。
    • org.eclipse.jdt.core.prefs:意見保留。
    • org.eclipse.jdt.ui.prefs:JDTの警告設定など。設定を共用するなら残した方がよさそう。

vmi avatar Jul 27 '14 07:07 vmi