remove problematic ahead-of-time compilation

[sibyl229]

Hi @a8wright @mgrbyte @azurebrd I think avoiding Ahead-of-time compilation might be a fix for the mount problem #44, along with the problem that the server doesn't always update when rest/fields/gene.clj is modified.

Please run lein clean before starting the server for the first time with this change. This hopefully remove the product of previous compilation. It seems to work for me consistently. Please let me know whether this solution works for you guys. Thanks!

I don't have a full explanation for the fix. I don't understand the details about lein's compilation process myself. But based on the two issues #44 and the reloading problem, it appears that we get different compilation results based on timestamps of the source file. So the experiment has been to simplify compilation settings in the project.clj file, in particular avoid things that might cause a "stateful" compilation.

[adamjohnwright]

Thanks @sibyl229 I am going to try this out in the morning

[mgrbyte]

@sibyl229 nice find!
More detail on the aot compilation process here.

[mgrbyte]

Think we still need to specify skip-aot for the :main namespace, e.g:

:main ^:skip-aot  datomic-to-catalyst.get-handler

Additionally, I think we want to preserve aot for uberjar builds.
This can be done by adding:

:uberjar {:aot :all}

to the :profiles map.

[adamjohnwright]

@mgrbyte this makes sense to me. Our configuration was consistently working with the aot. It was just the runtime that was having troubles.

[mgrbyte]

@a8wright yep. I've submitted a PR upstream to the mount repo that should fix the underlying issue (as per ref above)

[sibyl229]

Thanks @mgrbyte @a8wright for catching the uberjar issue! The
clojure {:uberjar {:aot :all}} option you set should be sufficient to resolve the uberjar issue (without requiring :aot [datomic-rest-api.get-handler], based on project.clj doc on :aot.

It appears that the :skip-aot doesn't take effect for me, and I had to remove the :aot [datomic-rest-api.get-handler] in d95b31a to avoid the mount issue. project.clj doc on :main hinted about this @a8wright and I tested creating the uberjar after the change, and it appears to work. Just letting you know here about the change. Please let me know if this breaks anything.