Skip to content

Draft: Barry/2021 11 26/requires auto set package attr

Barry Smith requested to merge barry/2021-11-26/requires-auto-set-package-attr into main

What do you think of having framework.requires automatically setting the attribute instead of manually always setting it?

Drawback is the name capitalization changes some (self.setCompilers for example)

Advantage is names will always be consistent everywhere, less boilerplate code

By adding a new final optional argument we could also automatically set odeps and deps

framework.require('config.packages.BlasLapack',self) (defaults to adding self.blaslapack to self.deps)

and

framework.require('config.packages.cuda',self,Optional=true) (instead adds self.cuda to self.odeps)

Merge request reports