Hmmm. On the one hand I do agree with Rajarshi that a whole class for this is a bit heavyweight, but it does open the door for other properties for the library to be made programatically available.
The problem is : what? The only thing that occurs to me is what modules are available - but the code wouldn't compile if the module wasn't there. In any case, you could probably use the classloader for that. The CDKConstants class is a bit of a god class - if that's the right anti-pattern - but I'm not sure it's the largest building in CDK city... http://gilleain.blogspot.com/2008/11/cdk-city.html gilleain On Thu, Jul 22, 2010 at 2:18 PM, Egon Willighagen <[email protected]> wrote: > Hi all CDK-users, > > it is time for a short popularity poll: please do reply, whatevery > your opinion is. > > Rajarshi and I want to introduce a method to get the version number of > a particular CDK release. We both came up with two difference patches: > > Patch 1: > add a static string field to CDKConstants that gets updated by the > build script, but leads to one extra change and commit for each > version > > Patch 2: > use the version in the already existing build.props, and have a new > class with the CDK.getVersion() method. This has the obvious downside > that we have another extra class, but does not require code changes > with each new version. > > Both for pros and cons... please discuss your favorite approach, > further ideas, etc. > > One particular question would be: is there other information about the > library you like to have available via the API? > > Your input would be very much appreciated! > > Egon > > > ---------- Forwarded message ---------- > From: Rajarshi Guha <[email protected]> > Date: Thu, Jul 22, 2010 at 3:03 PM > Subject: Re: [Cdk-devel] mol2 reader not working > To: Egon Willighagen <[email protected]> > Cc: "Developers forum for discussion about the Chemistry Development > Kit (CDK)" <[email protected]> > > On Thu, Jul 22, 2010 at 2:45 AM, Rajarshi wrote: >>> The following we should take to the mailing list, I guess... >>> >>> 3018684 Add a constant to indicate library version >>> >>> Please check my comments, and see how I can make my patch acceptable >>> for you... would it help if I added more functionality to the CDK >> >> Your approach makes sense, but I still think a whole class extra class >> for a version string is a bit overkill. Lets see what the mailling >> ist has to say? > > -- > Rajarshi Guha > NIH Chemical Genomics Center > > > > -- > Post-doc @ Uppsala University > Proteochemometrics / Bioclipse Group of Prof. Jarl Wikberg > Homepage: http://egonw.github.com/ > Blog: http://chem-bla-ics.blogspot.com/ > PubList: http://www.citeulike.org/user/egonw/tag/papers > > ------------------------------------------------------------------------------ > This SF.net email is sponsored by Sprint > What will you do first with EVO, the first 4G phone? > Visit sprint.com/first -- http://p.sf.net/sfu/sprint-com-first > _______________________________________________ > Cdk-user mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/cdk-user > ------------------------------------------------------------------------------ This SF.net email is sponsored by Sprint What will you do first with EVO, the first 4G phone? Visit sprint.com/first -- http://p.sf.net/sfu/sprint-com-first _______________________________________________ Cdk-user mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/cdk-user
