edgecase / dieter

Asset pipeline ring middleware
134 stars 22 forks source link

make lein-dieter-precompile work with lein2 #31

Closed pelle closed 11 years ago

pelle commented 11 years ago

Just a few small changes needed for it to work

pbiggar commented 11 years ago

Does it still work with lein1?

bmabey commented 11 years ago

I tried this patch and I got the following error with lein 2.0.0-preview 10:

Exception in thread "main" java.io.FileNotFoundException: Could not locate leiningen/dieter_precompile/runtime__init.class or leiningen/dieter_precompile/runtime.clj on classpath:
bmabey commented 11 years ago

I found out why the plugin no longer works... with lein2 the plugin is no longer on the CLASSPATH. Midje ran into the same problem and this is how they solved it: https://github.com/marick/lein-midje/blob/master/src/leiningen/midje.clj#L92-L97

si14 commented 11 years ago

Any updates with this? I would really like to use Dieter as a plugin, but this issue is a showstopper.

pelle commented 11 years ago

Sorry I'm not currently using dieter so I haven't had time to look more into this.

pbiggar commented 11 years ago

This might have been fixed by https://github.com/edgecase/dieter/pull/35

pbiggar commented 11 years ago

This was definitely fixed in version 0.3.0