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

Rewrite for Portability #3

Merged
merged 7 commits into from
Sep 6, 2014
Merged

Conversation

a4lg
Copy link
Contributor

@a4lg a4lg commented Sep 6, 2014

このプルリクエストは、MonoDevelop に最低限対応させるための変更と、ソースコードのスタイル統一を図るための変更を含みます。
各コミットの概要は以下のとおり。

  • NuGet.exe を本当に実行可能にする (Cygwin git においてチェックアウトしたワークツリー内でビルドできない問題の修正)
  • XML ファイルの拡張子を小文字に (UNIX スタイルへの修正)
  • 日本語のコメントとエラーメッセージを削除 (いずれも自動生成部分; NuGet 関連のエラーメッセージは対応する英語のエラーメッセージに置換)
  • MonoDevelop のために .gitignore パターンを追加
  • MonoDevelop のために依存関係をダウングレード (6.0.3 → 4.5.11)
  • MonoDevelop のためのソリューションを作成し、PCL プロジェクトを分離する
  • ソースコードスタイルの修正 (行末スペースの削除 [ただし NuGet 取得の ChainingAssertion を除く])

Visual Studio 2013 の最新アップデートと MonoDevelop 4.0 において、問題なく (警告すら無しに) コンパイルされ、動作する (テストをすべて通過する) ことを確認しました。

本当は CoreTweet と同じ流儀にまでしたいのですが (NuGet を埋め込みにせず submodule 等の形にする、マトモな Makefile ですべてが完結するようにする)、そのために必要な変更は割と大きめです。
そのため、PCL 非対応の Mono/MonoDevelop において最低限ビルドさせ、動作させることが可能な変更 (とソースコードを公開しても恥ずかしくないようにするための変更) のみを採用しました。

azyobuzin added a commit that referenced this pull request Sep 6, 2014
@azyobuzin azyobuzin merged commit 80320c5 into CoreTweet:master Sep 6, 2014
@azyobuzin
Copy link
Member

ありがとうございます!そういえば mono 版の nuspec はそもそも CoreTweet のほうでも使ってないので削除したほうがいいかもしれないです。

@a4lg
Copy link
Contributor Author

a4lg commented Sep 6, 2014

あ、そうだったのですか。今のところ CoreTweet と違って nuspec に深……くもない依存関係はつけてないので、CoreTweetSupplement-Mono.nuspec を削除するだけで大丈夫です (というかお任せします)。

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.

None yet

2 participants