From: SourceForge.net <no...@so...> - 2010-05-11 11:59:02
|
Patches item #2998955, was opened at 2010-05-09 11:27 Message generated for change (Comment added) made by rajarshi 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: Accepted >Status: Closed >Resolution: Fixed 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: Rajarshi Guha (rajarshi) Date: 2010-05-11 07:59 Message: Aa, thanks for catching that - I had actually done those, but forgot to include the patch. Anyway, applied and pushed to master.(log shows the same commit twice as I forgot to commit the first oatch the first time and so had to reset and do both) ---------------------------------------------------------------------- Comment By: Egon Willighagen (egonw) Date: 2010-05-11 03: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 |