Skip to content
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

metadata.json dependencies leads to errors on the commandline #19

Closed
stefanlasiewski opened this issue Dec 23, 2015 · 1 comment · Fixed by #21
Closed

metadata.json dependencies leads to errors on the commandline #19

stefanlasiewski opened this issue Dec 23, 2015 · 1 comment · Fixed by #21

Comments

@stefanlasiewski
Copy link

This module is returning errors for our puppet module commands, running Puppet 3.8.2 , even through the files are there:

$ puppet module list >/dev/null
Warning: Missing dependency 'puppetlabs-concat':
  'derdanne-nfs' (v0.0.7) requires 'puppetlabs-concat' (>= 1.1.2)
Warning: Missing dependency 'puppetlabs-stdlib':
  'derdanne-nfs' (v0.0.7) requires 'puppetlabs-stdlib' (>= 4.5.0)

$ puppet module list
??? puppetlabs-concat (v1.2.3)
??? puppetlabs-inifile (v1.4.2)
??? puppetlabs-stdlib (v4.10.0)
$

If I modify nfs/metadata.json and and replace the hyphens with slashes, the error goes away:

Old:

  "dependencies": [
    {"name":"puppetlabs-stdlib","version_requirement":">= 4.5.0"},
    {"name":"puppetlabs-concat","version_requirement":">= 1.1.2"}
  ]   

New:

  "dependencies": [
    {"name":"puppetlabs/stdlib","version_requirement":">= 4.5.0"},
    {"name":"puppetlabs/concat","version_requirement":">= 1.1.2"}
  ]
$ puppet module list >/dev/null
# Nothing here. All is good.
$

The example at https://docs.puppetlabs.com/puppet/latest/reference/modules_metadata.html#specifying-dependencies uses slash characters and not hyphens. I believe that slash characters are preferred, but I'm not 100% positive.

@derdanne
Copy link
Owner

I will make the changes in the next version, thanks!

derdanne added a commit that referenced this issue Jan 13, 2016
update client mount spec tests
fix #19
update gentoo default $client_nfsv4_fstype to reflect syntax update in net-fs/nfs-utils
@derdanne derdanne mentioned this issue Jan 13, 2016
derdanne added a commit that referenced this issue Jan 13, 2016
update client mount spec tests
fix #19
update gentoo default $client_nfsv4_fstype to reflect syntax update in net-fs/nfs-utils
version bump
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants