puppet-nodejs icon indicating copy to clipboard operation
puppet-nodejs copied to clipboard

installing from git

Open onzyone opened this issue 10 years ago • 7 comments

when installing a module from git

  nodejs::npm { 'node-wimoto':
    ensure  => 'present',
    package => 'node-wimoto',
    source  => 'onzyone/node-wimoto',
    target  => '/opt/packages',
  }

source ends up in the grep as source is no longer registry

  if $source != 'registry' {
    $install_check_package_string = $source
    $package_string = $source

this fails and re installs the module over and over again as it is looking for

Debug: Exec[npm_install_node-wimoto](provider=posix): Executing check '/usr/bin/npm ls --long --parseable | grep "/opt/packages/node_modules/onzyone/node-wimoto"'
Debug: Executing: '/usr/bin/npm ls --long --parseable | grep "/opt/packages/node_modules/onzyone/node-wimoto"'

and not

grep "/opt/packages/node_modules/node-wimoto"'

onzyone avatar Sep 21 '15 02:09 onzyone

@onzyone do you think you can provide a patch for this?

igalic avatar Sep 21 '15 16:09 igalic

Yup I am looking into it and will give one in the next few days.

On Mon, Sep 21, 2015 at 12:12 PM, Igor Galić [email protected] wrote:

@onzyone https://github.com/onzyone do you think you can provide a patch for this?

— Reply to this email directly or view it on GitHub https://github.com/puppet-community/puppet-nodejs/issues/166#issuecomment-142030701 .

onzyone avatar Sep 21 '15 17:09 onzyone

have a look at this and run a few tests: https://github.com/onzyone/puppet-nodejs/tree/onzyone-patch-1 and I will do merge

onzyone avatar Sep 21 '15 23:09 onzyone

uhm… @onzyone if you create the pull request against this repo, then travis can take care of running tests for you :)

igalic avatar Sep 22 '15 10:09 igalic

What's the status of this?

jyaworski avatar Feb 29 '16 00:02 jyaworski

¯\(o_°)/¯

igalic avatar Mar 09 '16 14:03 igalic

If I recall correctly I think the bug is that the module assumes package rather than namespace/package for source

juniorsysadmin avatar Jun 02 '16 12:06 juniorsysadmin