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

platform superspawn exists version check is unfortunate

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 3.4.0
    • None
    • cordova-cli, cordova-windows
    • None

    Description

      The code before CB-5181 / b675ee35d5b92f5b997a834d4e6575e1edc14ef3 didn't really look before it leapt, which meant that a platform with only a cordova/version.bat and no cordova/version would magically work.

      While I'm not sure of any Windows only platforms, it doesn't seem unreasonable.

      A quick check confirms that in fact windows8 doesn't have this file.

      https://git-wip-us.apache.org/repos/asf/cordova-windows.git @ ca106cfd91efa74b3556d70b6a936e77d85ec169

      cordova-windows $ find windows8 -name 'versi*'
      windows8/template/cordova/version.bat

      Attachments

        Issue Links

          relates to

          Improvement - An improvement or enhancement to an existing feature or task. CB-5181 consolidate ALL child process exec and spawn commands CLIs

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Closed

          Activity

            People

              agrieve Andrew Grieve
              jsoref Josh Soref
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: