From: SourceForge.net <no...@so...> - 2010-05-11 07:05:51
|
Patches item #2998955, was opened at 2010-05-09 17:27 Message generated for change (Comment added) made by egonw You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=2998955&group_id=20024 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: master Group: Needs Review Status: Open Resolution: None Priority: 5 Private: No Submitted By: Rajarshi Guha (rajarshi) >Assigned to: Rajarshi Guha (rajarshi) Summary: remove R-CDK bridge code Initial Comment: This patch remoes the qsar/model package since the R-CDK bridge is hardly used and unecessarily expands the scope of the library. Patch removes sources, test sources, jar files. Due to the idiotic SF limits, you can get this patch from http://rguha.net/0001-Removed-the-R-based-model-package-and-associated-jar.patch ---------------------------------------------------------------------- >Comment By: Egon Willighagen (egonw) Date: 2010-05-11 09:05 Message: Rajarshi, I had to make a follow up patch to fix compiling of the test-qsar module... your patch looks fine otherwise. I signed off your patch, and left mine in the 144-removeJRI branch: http://github.com/egonw/cdk/commits/144-removeJRI Please check my patch, sign off if you are OK, and commit both to the repository. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=2998955&group_id=20024 |