oridb / mc

Myrddin Compiler
MIT License
387 stars 34 forks source link

Dynamically link if any deps use dynamic libs #166

Closed cbvi closed 6 years ago

cbvi commented 6 years ago

This lets a pure myrddin project use an installed lib that binds to C and be linked correctly.

I think addlib is the best place to determine this since it's walking the dependency tree anyway, but maybe there's a better way?

cbvi commented 6 years ago

How about this?

cbvi commented 6 years ago

Third time's the charm?

It's back in the myrtag, so not b.dynlink, because of multiple targets as you explained earlier.

oridb commented 6 years ago

Looks good to me.