Apache Cordova
  1. Apache Cordova
  2. CB-1298

bin/create - if you move the created project, the CordovaLib sub-project reference breaks

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: 2.1.0
    • Fix Version/s: 2.1.0
    • Component/s: iOS
    • Labels:
      None

      Description

      1. Create a project using bin/create
      2. Move the created project to another folder
      3. The CordovaLib reference is invalid

      Fix:
      Right now the sub-project path is relative to the group, we need to set it to an absolute path.

        Issue Links

          Activity

          Hide
          Andrew Grieve added a comment -

          We could add this as an option to create, but my thinking was that relative paths would be preferable because they work better with source control.

          e.g. you have your own repo and you check in your project as well as it's dependencies (cordova). Absolute paths wouldn't be desirable in this case.

          Show
          Andrew Grieve added a comment - We could add this as an option to create, but my thinking was that relative paths would be preferable because they work better with source control. e.g. you have your own repo and you check in your project as well as it's dependencies (cordova). Absolute paths wouldn't be desirable in this case.
          Hide
          Shazron Abdullah added a comment -

          Didn't think about the source control scenario. Let's keep the default (relative to group), and add an option for absolute path in any case (since that is analogous to the previous CordovaLib variable)

          Show
          Shazron Abdullah added a comment - Didn't think about the source control scenario. Let's keep the default (relative to group), and add an option for absolute path in any case (since that is analogous to the previous CordovaLib variable)
          Hide
          Shazron Abdullah added a comment -

          Won't fix - works as expected. I will add an "improvement" issue for 2.2.0 for the absolute path option, low prio.

          Show
          Shazron Abdullah added a comment - Won't fix - works as expected. I will add an "improvement" issue for 2.2.0 for the absolute path option, low prio.

            People

            • Assignee:
              Andrew Grieve
              Reporter:
              Shazron Abdullah
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development