search

cljsjs / boot-cljsjs

Helper tasks to aid the packaging of JS dependencies for Clojurescript projects
66 stars 22 forks source link

Add support to externs free libraries #23

Closed jeluard closed 9 years ago

jeluard commented 9 years ago

Some libraries do not need externs to properly work. A canonical example is a polyfill library whose associated externs are already in closure compiler.

Let me know if this is desirable. I have a patch for this.

martinklepsch commented 9 years ago

Good point. What kind of patch do you have in mind?

jeluard commented 9 years ago

Simply removing the check on externs and not including it in deps.cljs. It appears to work fine. Am I missing something?

martinklepsch commented 9 years ago

Maybe we should introduce a flag --no-externs so we it's a little more explicit and we still keep the validation helpers for the common case of packaging libs with extern files?

jeluard commented 9 years ago

Sounds good I'll do that.