Skip to content

Conversation

@vmi
Copy link

@vmi vmi commented Jul 27, 2014

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

@kompiro
Copy link
Owner

kompiro commented Jul 27, 2014

@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ファイル等を残したままです。 なので、この変更はまだ取り込まない方がよいように思います。いかがでしょうか?

@vmi
Copy link
Author

vmi commented Jul 27, 2014

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の警告設定など。設定を共用するなら残した方がよさそう。

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.

2 participants