From: SourceForge.net <no...@so...> - 2008-12-08 08:09:44
|
Patches item #2405759, was opened at 2008-12-08 09:09 Message generated for change (Tracker Item Submitted) made by Item Submitter You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=2405759&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: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Egon Willighagen (egonw) Assigned to: Nobody/Anonymous (nobody) Summary: upgrade to jni-inchi 0.5 Initial Comment: Sam has released a new version of his bridge to the InChI libraries, and version 0.5 adds support for 64 bit systems, as well as OS/X, and on top of that InChIKey generation. Attached a patch for the cdk-1.2.x/ branch. Please verify by two senior developers (and anyone else), as I like to see it included in the cdk-1.2.x/ branch, which is formally in freeze. The patch does not change any API methods, except for adding a getInChIKey() method (see patch). A unit test assertion for InChIKey generation is added. The full git branch is up here: http://github.com/egonw/cdk/commits/newJNIInChI ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=2405759&group_id=20024 |
From: SourceForge.net <no...@so...> - 2008-12-08 12:20:13
|
Patches item #2405759, was opened at 2008-12-08 09:09 Message generated for change (Settings changed) made by egonw You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=2405759&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: None >Group: Needs Review Status: Open Resolution: None Priority: 5 Private: No Submitted By: Egon Willighagen (egonw) Assigned to: Nobody/Anonymous (nobody) Summary: upgrade to jni-inchi 0.5 Initial Comment: Sam has released a new version of his bridge to the InChI libraries, and version 0.5 adds support for 64 bit systems, as well as OS/X, and on top of that InChIKey generation. Attached a patch for the cdk-1.2.x/ branch. Please verify by two senior developers (and anyone else), as I like to see it included in the cdk-1.2.x/ branch, which is formally in freeze. The patch does not change any API methods, except for adding a getInChIKey() method (see patch). A unit test assertion for InChIKey generation is added. The full git branch is up here: http://github.com/egonw/cdk/commits/newJNIInChI ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=2405759&group_id=20024 |
From: SourceForge.net <no...@so...> - 2008-12-10 15:31:43
|
Patches item #2405759, was opened at 2008-12-08 03:09 Message generated for change (Comment added) made by rajarshi You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=2405759&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: None Group: Needs Review Status: Open Resolution: None Priority: 5 Private: No Submitted By: Egon Willighagen (egonw) Assigned to: Nobody/Anonymous (nobody) Summary: upgrade to jni-inchi 0.5 Initial Comment: Sam has released a new version of his bridge to the InChI libraries, and version 0.5 adds support for 64 bit systems, as well as OS/X, and on top of that InChIKey generation. Attached a patch for the cdk-1.2.x/ branch. Please verify by two senior developers (and anyone else), as I like to see it included in the cdk-1.2.x/ branch, which is formally in freeze. The patch does not change any API methods, except for adding a getInChIKey() method (see patch). A unit test assertion for InChIKey generation is added. The full git branch is up here: http://github.com/egonw/cdk/commits/newJNIInChI ---------------------------------------------------------------------- Comment By: Rajarshi Guha (rajarshi) Date: 2008-12-10 10:31 Message: Patch looks fine to me. Has this been tested on OS X or Linux 64 bit? ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=2405759&group_id=20024 |
From: SourceForge.net <no...@so...> - 2008-12-10 19:19:49
|
Patches item #2405759, was opened at 2008-12-08 09:09 Message generated for change (Comment added) made by egonw You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=2405759&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: None Group: Needs Review Status: Open Resolution: None Priority: 5 Private: No Submitted By: Egon Willighagen (egonw) Assigned to: Nobody/Anonymous (nobody) Summary: upgrade to jni-inchi 0.5 Initial Comment: Sam has released a new version of his bridge to the InChI libraries, and version 0.5 adds support for 64 bit systems, as well as OS/X, and on top of that InChIKey generation. Attached a patch for the cdk-1.2.x/ branch. Please verify by two senior developers (and anyone else), as I like to see it included in the cdk-1.2.x/ branch, which is formally in freeze. The patch does not change any API methods, except for adding a getInChIKey() method (see patch). A unit test assertion for InChIKey generation is added. The full git branch is up here: http://github.com/egonw/cdk/commits/newJNIInChI ---------------------------------------------------------------------- Comment By: Egon Willighagen (egonw) Date: 2008-12-10 20:19 Message: Excellent point! We need 64bit and OS/X Nightly servers :) Ummm... what we also need, is a Nightly meta server, which keeps track of different instances of nightly's... and maybe put (comparable) test results in a table... which would make it easier to compare platform dependencies, as well as an easy way to autocompare (comparable) branches... say a patch branch for cdk1.2.x/... not so much thinking about comparing trunk with cdk1.2.x/ here... ---------------------------------------------------------------------- Comment By: Rajarshi Guha (rajarshi) Date: 2008-12-10 16:31 Message: Patch looks fine to me. Has this been tested on OS X or Linux 64 bit? ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=2405759&group_id=20024 |
From: SourceForge.net <no...@so...> - 2009-01-20 19:19:01
|
Patches item #2405759, was opened at 2008-12-08 09:09 Message generated for change (Comment added) made by egonw You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=2405759&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: None Group: Needs Review Status: Open Resolution: None Priority: 5 Private: No Submitted By: Egon Willighagen (egonw) Assigned to: Nobody/Anonymous (nobody) Summary: upgrade to jni-inchi 0.5 Initial Comment: Sam has released a new version of his bridge to the InChI libraries, and version 0.5 adds support for 64 bit systems, as well as OS/X, and on top of that InChIKey generation. Attached a patch for the cdk-1.2.x/ branch. Please verify by two senior developers (and anyone else), as I like to see it included in the cdk-1.2.x/ branch, which is formally in freeze. The patch does not change any API methods, except for adding a getInChIKey() method (see patch). A unit test assertion for InChIKey generation is added. The full git branch is up here: http://github.com/egonw/cdk/commits/newJNIInChI ---------------------------------------------------------------------- >Comment By: Egon Willighagen (egonw) Date: 2009-01-20 20:18 Message: OS/X works, 32 bit and 64 bit. Linux 64bit has been reported not to work from within Bioclipse. ---------------------------------------------------------------------- Comment By: Egon Willighagen (egonw) Date: 2008-12-10 20:19 Message: Excellent point! We need 64bit and OS/X Nightly servers :) Ummm... what we also need, is a Nightly meta server, which keeps track of different instances of nightly's... and maybe put (comparable) test results in a table... which would make it easier to compare platform dependencies, as well as an easy way to autocompare (comparable) branches... say a patch branch for cdk1.2.x/... not so much thinking about comparing trunk with cdk1.2.x/ here... ---------------------------------------------------------------------- Comment By: Rajarshi Guha (rajarshi) Date: 2008-12-10 16:31 Message: Patch looks fine to me. Has this been tested on OS X or Linux 64 bit? ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=2405759&group_id=20024 |
From: SourceForge.net <no...@so...> - 2009-01-20 21:16:28
|
Patches item #2405759, was opened at 2008-12-08 03:09 Message generated for change (Comment added) made by rajarshi You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=2405759&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: None Group: Needs Review Status: Open Resolution: None Priority: 5 Private: No Submitted By: Egon Willighagen (egonw) Assigned to: Nobody/Anonymous (nobody) Summary: upgrade to jni-inchi 0.5 Initial Comment: Sam has released a new version of his bridge to the InChI libraries, and version 0.5 adds support for 64 bit systems, as well as OS/X, and on top of that InChIKey generation. Attached a patch for the cdk-1.2.x/ branch. Please verify by two senior developers (and anyone else), as I like to see it included in the cdk-1.2.x/ branch, which is formally in freeze. The patch does not change any API methods, except for adding a getInChIKey() method (see patch). A unit test assertion for InChIKey generation is added. The full git branch is up here: http://github.com/egonw/cdk/commits/newJNIInChI ---------------------------------------------------------------------- >Comment By: Rajarshi Guha (rajarshi) Date: 2009-01-20 16:16 Message: Looks good - it's be useful to include a note in the docs regarding 64 bit Linux (I don't have a 64 bit OS anymore to test on) ---------------------------------------------------------------------- Comment By: Egon Willighagen (egonw) Date: 2009-01-20 14:18 Message: OS/X works, 32 bit and 64 bit. Linux 64bit has been reported not to work from within Bioclipse. ---------------------------------------------------------------------- Comment By: Egon Willighagen (egonw) Date: 2008-12-10 14:19 Message: Excellent point! We need 64bit and OS/X Nightly servers :) Ummm... what we also need, is a Nightly meta server, which keeps track of different instances of nightly's... and maybe put (comparable) test results in a table... which would make it easier to compare platform dependencies, as well as an easy way to autocompare (comparable) branches... say a patch branch for cdk1.2.x/... not so much thinking about comparing trunk with cdk1.2.x/ here... ---------------------------------------------------------------------- Comment By: Rajarshi Guha (rajarshi) Date: 2008-12-10 10:31 Message: Patch looks fine to me. Has this been tested on OS X or Linux 64 bit? ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=2405759&group_id=20024 |
From: SourceForge.net <no...@so...> - 2009-02-06 12:13:27
|
Patches item #2405759, was opened at 2008-12-08 09:09 Message generated for change (Comment added) made by shk3 You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=2405759&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: None Group: Needs Review Status: Open Resolution: None Priority: 5 Private: No Submitted By: Egon Willighagen (egonw) Assigned to: Nobody/Anonymous (nobody) Summary: upgrade to jni-inchi 0.5 Initial Comment: Sam has released a new version of his bridge to the InChI libraries, and version 0.5 adds support for 64 bit systems, as well as OS/X, and on top of that InChIKey generation. Attached a patch for the cdk-1.2.x/ branch. Please verify by two senior developers (and anyone else), as I like to see it included in the cdk-1.2.x/ branch, which is formally in freeze. The patch does not change any API methods, except for adding a getInChIKey() method (see patch). A unit test assertion for InChIKey generation is added. The full git branch is up here: http://github.com/egonw/cdk/commits/newJNIInChI ---------------------------------------------------------------------- >Comment By: Stefan Kuhn (shk3) Date: 2009-02-06 13:13 Message: Looks good. Not tested the 64bit stuff working. ---------------------------------------------------------------------- Comment By: Rajarshi Guha (rajarshi) Date: 2009-01-20 22:16 Message: Looks good - it's be useful to include a note in the docs regarding 64 bit Linux (I don't have a 64 bit OS anymore to test on) ---------------------------------------------------------------------- Comment By: Egon Willighagen (egonw) Date: 2009-01-20 20:18 Message: OS/X works, 32 bit and 64 bit. Linux 64bit has been reported not to work from within Bioclipse. ---------------------------------------------------------------------- Comment By: Egon Willighagen (egonw) Date: 2008-12-10 20:19 Message: Excellent point! We need 64bit and OS/X Nightly servers :) Ummm... what we also need, is a Nightly meta server, which keeps track of different instances of nightly's... and maybe put (comparable) test results in a table... which would make it easier to compare platform dependencies, as well as an easy way to autocompare (comparable) branches... say a patch branch for cdk1.2.x/... not so much thinking about comparing trunk with cdk1.2.x/ here... ---------------------------------------------------------------------- Comment By: Rajarshi Guha (rajarshi) Date: 2008-12-10 16:31 Message: Patch looks fine to me. Has this been tested on OS X or Linux 64 bit? ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=2405759&group_id=20024 |
From: SourceForge.net <no...@so...> - 2009-02-06 12:31:22
|
Patches item #2405759, was opened at 2008-12-08 09:09 Message generated for change (Settings changed) made by egonw You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=2405759&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: None >Group: Accepted >Status: Closed >Resolution: Fixed Priority: 5 Private: No Submitted By: Egon Willighagen (egonw) Assigned to: Nobody/Anonymous (nobody) Summary: upgrade to jni-inchi 0.5 Initial Comment: Sam has released a new version of his bridge to the InChI libraries, and version 0.5 adds support for 64 bit systems, as well as OS/X, and on top of that InChIKey generation. Attached a patch for the cdk-1.2.x/ branch. Please verify by two senior developers (and anyone else), as I like to see it included in the cdk-1.2.x/ branch, which is formally in freeze. The patch does not change any API methods, except for adding a getInChIKey() method (see patch). A unit test assertion for InChIKey generation is added. The full git branch is up here: http://github.com/egonw/cdk/commits/newJNIInChI ---------------------------------------------------------------------- >Comment By: Egon Willighagen (egonw) Date: 2009-02-06 13:31 Message: Stefan, Rajarshi, thanx for the review! ---------------------------------------------------------------------- Comment By: Stefan Kuhn (shk3) Date: 2009-02-06 13:13 Message: Looks good. Not tested the 64bit stuff working. ---------------------------------------------------------------------- Comment By: Rajarshi Guha (rajarshi) Date: 2009-01-20 22:16 Message: Looks good - it's be useful to include a note in the docs regarding 64 bit Linux (I don't have a 64 bit OS anymore to test on) ---------------------------------------------------------------------- Comment By: Egon Willighagen (egonw) Date: 2009-01-20 20:18 Message: OS/X works, 32 bit and 64 bit. Linux 64bit has been reported not to work from within Bioclipse. ---------------------------------------------------------------------- Comment By: Egon Willighagen (egonw) Date: 2008-12-10 20:19 Message: Excellent point! We need 64bit and OS/X Nightly servers :) Ummm... what we also need, is a Nightly meta server, which keeps track of different instances of nightly's... and maybe put (comparable) test results in a table... which would make it easier to compare platform dependencies, as well as an easy way to autocompare (comparable) branches... say a patch branch for cdk1.2.x/... not so much thinking about comparing trunk with cdk1.2.x/ here... ---------------------------------------------------------------------- Comment By: Rajarshi Guha (rajarshi) Date: 2008-12-10 16:31 Message: Patch looks fine to me. Has this been tested on OS X or Linux 64 bit? ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=2405759&group_id=20024 |