Re: [Cdk-jchempaint] call for software review

[<ra...@ar...>]

On Thu, Jun 21, 2012 at 02:33:04PM -0700, Cyrus Harmon wrote:
> Do you have any concise/up-to-date build instructions that go along with this?

No, just ant or ant dist-applet. There is no mechanics in build.xml
to run single tests and you will get errors from a dozen older tests
if you run all tests (they also require manual intervention which
shouldn't be; this is issue #62), but you can run single tests from
Eclipse fine. More important would be the code review, however.

> I assume I should be using the CDK from git://github.com/cdk/cdk.git
> 
> But should I be using the HEAD or some particular tag/branch?

1.4.10 please.

> What do you mean about the pull request being applied against master? This will be applied to jchempaint, not cdk, right?

Yes, right. It's possible to have pull requests that are applied
to a branch, like the other PR with JCP that, if applied, goes into
the cdkapi-1.5 branch.

Thanks for your efforts!