search

samoht / assemblage

A collection of tools to manage the configuration of OCaml projects
54 stars 4 forks source link

Lazy ocamlfind in makefile #95

Closed dbuenzli closed 10 years ago

dbuenzli commented 10 years ago

Do not merge. This is a (useless) proof of concept only.

Having been running in circles the whole day between #93, #88 and what I suggested in #92, it seems in fact hard to boil down to per package ocamlfind calls (which I guess @samoht realized long time ago). But I fell on that hack to memoize values in makefiles. I just tried a proof of concept implementation which would optimize calls to ocamlfind per ordered set of packages. But on my examples it doesn't seem to make any difference since what we had in e767148.

samoht commented 10 years ago

Not sure I understand what the lazy invocation bits really improve. In Makefiles, declaring a variable is already lazy.

dbuenzli commented 10 years ago

Iiiiiik 1) this was not for merging 2) the improvement here was that the same ocamlfind invocation found in several variable definition would be computed only once.

samoht commented 10 years ago

ack. reverted