Uploaded image for project: 'Apache Cordova'
  1. Apache Cordova
  2. CB-12749

coho should not care about license dependencies for "check-license"

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Open
    • Major
    • Resolution: Unresolved
    • None
    • None
    • cordova-coho
    • None

    Description

      We only care about licenses of code we ship (only code in the repo, like bundled deps).
      I did a fresh repo clone of the tools, for example. coho repo-clone -r tools

      Right now it complains about node_module deps:

      coho check-license -r tools > license.txt
      Warning: dependency "configstore" probably not installed.
      please install dependencies from npm before running nlf
      Warning: dependency "cordova-common" probably not installed.
      please install dependencies from npm before running nlf
      Warning: dependency "cordova-lib" probably not installed.
      please install dependencies from npm before running nlf
      Warning: dependency "editor" probably not installed.
      please install dependencies from npm before running nlf
      Warning: dependency "insight" probably not installed.
      please install dependencies from npm before running nlf
      Warning: dependency "nopt" probably not installed.
      please install dependencies from npm before running nlf
      Warning: dependency "q" probably not installed.
      please install dependencies from npm before running nlf
      Warning: dependency "update-notifier" probably not installed.
      please install dependencies from npm before running nlf
      Warning: dependency "cordova-lib" probably not installed.
      please install dependencies from npm before running nlf
      Warning: dependency "nopt" probably not installed.
      please install dependencies from npm before running nlf
      Warning: dependency "q" probably not installed.
      please install dependencies from npm before running nlf
      Warning: dependency "cordova-app-hello-world" probably not installed.
      please install dependencies from npm before running nlf
      Warning: dependency "cordova-common" probably not installed.
      please install dependencies from npm before running nlf
      Warning: dependency "cordova-fetch" probably not installed.
      please install dependencies from npm before running nlf
      Warning: dependency "q" probably not installed.
      please install dependencies from npm before running nlf
      Warning: dependency "shelljs" probably not installed.
      please install dependencies from npm before running nlf
      Warning: dependency "valid-identifier" probably not installed.
      please install dependencies from npm before running nlf
      Warning: dependency "browserify" probably not installed.
      please install dependencies from npm before running nlf
      

      Attachments

        Activity

          People

            shazron Shazron Abdullah
            shazron Shazron Abdullah
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated: