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

          Shazron Abdullah created issue -
          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.
          Shazron Abdullah made changes -
          Field Original Value New Value
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Won't Fix [ 2 ]
          Shazron Abdullah made changes -
          Link This issue is related to CB-1382 [ CB-1382 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Resolved Resolved
          6d 21h 51m 1 Shazron Abdullah 06/Sep/12 19:24

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development