search

cloojure / tupelo

Tupelo: Clojure With A Spoonful of Honey
Eclipse Public License 1.0
510 stars 15 forks source link

Make DatomicPro dependency optional #3

Closed dlebrero closed 8 years ago

dlebrero commented 8 years ago

Right now tupelo.core depends on tupelo.schema which references the datomic.db.Db class which comes from the DatomicPro jar, which needs to be manually downloaded and installed.

Could the DatomicPro dependency be made optional?

cloojure commented 8 years ago

That is a good question! I have not played around with Datomic Free, which would be the other option. Obviously, either the Pro or the Free distro would be required for Tupelo Datomic to make sense.

Maybe I just need to refactor the tupelo.schema class. Let me look at that.

Alan

On Wed, Nov 25, 2015 at 11:48 AM, dlebrero notifications@github.com wrote:

Right now tupelo.core depends on tupelo.schema which references the datomic.db.Db class which comes from the DatomicPro jar, which needs to be manually downloaded and installed.

Could the DatomicPro dependency be made optional?

— Reply to this email directly or view it on GitHub https://github.com/cloojure/tupelo/issues/3.

cloojure commented 8 years ago

Hi - Would you like to try version 0.1.55? I removed the datomic dependency into a difference ns, so I think that will fix it. If not, please let me know.

Thanks, Alan

On Wed, Nov 25, 2015 at 11:52 AM, Alan Thompson cloojure@gmail.com wrote:

That is a good question! I have not played around with Datomic Free, which would be the other option. Obviously, either the Pro or the Free distro would be required for Tupelo Datomic to make sense.

Maybe I just need to refactor the tupelo.schema class. Let me look at that.

Alan

On Wed, Nov 25, 2015 at 11:48 AM, dlebrero notifications@github.com wrote:

Right now tupelo.core depends on tupelo.schema which references the datomic.db.Db class which comes from the DatomicPro jar, which needs to be manually downloaded and installed.

Could the DatomicPro dependency be made optional?

— Reply to this email directly or view it on GitHub https://github.com/cloojure/tupelo/issues/3.

dlebrero commented 8 years ago

Works fine now. Thanks a lot!