schrepfler / puppet-jdk_oracle

Puppet module to install JDK from oracle using wget
Other
1 stars 7 forks source link

package wget undefined even though its required #1

Closed viper233 closed 10 years ago

viper233 commented 10 years ago

The package wget is required though not defined anywhere.

Does this module require other modules? I'm running this module alone.

Error message.

Error: Failed to apply catalog: Could not find dependency Package[wget] for Exec[get_jdk_installer] at /etc/puppet/modules/jdk_oracle/manifests/init.pp:47

schrepfler commented 10 years ago

Hi Viper, wget is definitely a required package. Still not too versed in puppet, I think if I specifically add a package wget on the class it should resolve it.

schrepfler commented 10 years ago

eg. package { 'wget': ensure => present, }

viper233 commented 10 years ago

Thanks for the prompt reply. I tried that and then it asked for the Yum class.. I'll talk to my colleagues who are a bit more familiar with puppet and see what I can come up with.

schrepfler commented 10 years ago

We can probably declare that as well although I think puppet should be able to map a generic package onto any package manager (anyhow, this module is rpm specific, probably we should declare rpm explicitly as well).

Cinderhaze commented 10 years ago

I forked the original repo and submitted a pull request with the change you suggested above.

https://github.com/Cinderhaze/puppet-jdk_oracle

alitheg commented 10 years ago

That makes sense - I've been using this module from another which already pulls in wget so didn't see the issue.

schrepfler commented 10 years ago

fixed.