-
-
Notifications
You must be signed in to change notification settings - Fork 375
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Could not look up qualified variable python::install::valid_versions
#126
Comments
I just hit this as well. The python class would have to inherit from python::install in order for this to work. What a PR be accepted to impliment the base class inherients from foo::params pattern? |
You define a variable as "::python::install::valid_version" https://github.com/stankevich/puppet-python/blob/master/manifests/install.pp#L18 The puppet compiler warns about it: Could not look up qualified variable '::python::install::valid_versions' |
This module definitely need a python::params class to mitigate this problem (fixing the typo in :python::install::valid_version is not fixing the problem) |
names are different: |
Even when correcting the typo for variable ::python::install::valid_versions you still can't access the variable ::python::install::valid_versions because the class ::python::install has not been evaluated yet. Class ::python needs to inherits from python::install, but then you run into the same type of problem with the python::install class which needs class ::python to be evaluated first as well... In order to mitigate these circular issues the proper way is to to create a ::python::params class and have class python inherits python::params Warning: Scope(Class[Python]): Could not look up qualified variable '::python::install::valid_versions'; class ::python::install has not been evaluated |
python::install::valid_versions
Scope(Class[Python]): Could not look up qualified variable '::python::install::valid_versions'; class ::python::install has not been evaluated
Definition:
https://github.com/stankevich/puppet-python/blob/master/manifests/install.pp#L18
Including:
https://github.com/stankevich/puppet-python/blob/master/manifests/init.pp#L73
The text was updated successfully, but these errors were encountered: