Hi Nina, Hi Egon, On 21.04.2011 21:27, Nina Jeliazkova wrote: > Egon, > > On 21 April 2011 21:29, Egon Willighagen <[email protected]> wrote: > >> On Thu, Apr 21, 2011 at 7:25 PM, Nina Jeliazkova <[email protected]> >> wrote: >>> >>> Done. Please note , no dependencies are defined for this artifact >>> (although obviously it should depend on core cdk packages), and not >>> tested so far. >>> >>> <dependency> >>> <groupId>org.openscience.cdk</groupId> >>> <artifactId>cdk-jchempaint</artifactId> >>> <version>18</version> >>> </dependency>
Thanks a lot! Egon might know better, but if dependencies are not strictly defined at the moment - we will add them later. >>> Accepted - I would really appreciate help of splitting JChemPaint >>> into dependencies. SWT dependency is a good example indeed - we are using >>> JChemPaint code in a swing application and in a web services application, >>> and neither needs SWT... >>> How do we start - perhaps a hint from JChemPaint developers where/how the >>> dependencies are defined ? >> >> Use this branch: >> >> https://github.com/egonw/cdk/tree/13-unsorted-patches >> >> It's CDK 1.3.8 + CDK-JChemPaint 18, with the module approach you >> expect from the CDK. > > Thanks - one question to clarify for myself. > > As CDK-JChemPaint is a set of patches, does it change anything in the > cdk modules, or just adds new classes? I suspect it changes CDK a bit. On the other hand, if complete merging of JChemPaint into CDK core is a matter of short time -- let's wait until final rollout (in order not to mess up with branches and patches). Thanks again for very quick solution! -- With best regards, Dmitry ------------------------------------------------------------------------------ Fulfilling the Lean Software Promise Lean software platforms are now widely adopted and the benefits have been demonstrated beyond question. Learn why your peers are replacing JEE containers with lightweight application servers - and what you can gain from the move. http://p.sf.net/sfu/vmware-sfemails _______________________________________________ Cdk-user mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/cdk-user
