search

foam-framework / foam2

FOAM: Feature-Oriented Active Modeller, Version 2
Apache License 2.0
73 stars 63 forks source link

foam.lookup() should work for LIBs #196

Open mdittmer opened 7 years ago

mdittmer commented 7 years ago

If the package paths is to not pollute the global namespace (as discussed in #195), then LIBs, like CLASSes must be accessible via some API. The most natural fit seems to be making foam.lookup() work on LIBs, just as it does on CLASSes.

bshepherdson commented 7 years ago

I'm not clear on the bug here. Is it that libs are not put onto the namespace? Or is it that they are, but that's wrong?

If I read this and #195 correctly, you seem to be proposing or expecting that (a) no globals but foam will be created automatically, and (b) that therefore libs in other packages need a way to be accessed.

I counter that (a) is wrong, and therefore (b) is no problem: access your libs with their globals.

mdittmer commented 7 years ago

Changing this to a feature request. I would argue that FOAM should not force developers to use a style that places their models in the global namespace. The lookup interface seems like a natural way to facilitate this.