Apache Cordova
  1. Apache Cordova
  2. CB-4456

Remove support of plugins-plist elements in plugman

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: 3.6.0
    • Component/s: Plugman
    • Labels:
      None

      Activity

      Hide
      Mark Koudritsky added a comment -

      Just looked at it.
      I did remove the support back in february and left only a deprecation warning. Some plugins-plist related code was left in the tests - removed it now.

      Show
      Mark Koudritsky added a comment - Just looked at it. I did remove the support back in february and left only a deprecation warning. Some plugins-plist related code was left in the tests - removed it now.
      Hide
      ASF subversion and git services added a comment -

      Commit 3e468db64c480a446884849586d7686b1bc2e6bf in cordova-lib's branch refs/heads/master from Mark Koudritsky
      [ https://git-wip-us.apache.org/repos/asf?p=cordova-lib.git;h=3e468db ]

      CB-4456: Remove plugins-plist bits from the tests

      Support for plugins-plist was removed as of 3.4 in Feb 2014. It was replaced
      with a deprecation warning. This change is for removal of plugins-plist
      related bits in the tests.

      Show
      ASF subversion and git services added a comment - Commit 3e468db64c480a446884849586d7686b1bc2e6bf in cordova-lib's branch refs/heads/master from Mark Koudritsky [ https://git-wip-us.apache.org/repos/asf?p=cordova-lib.git;h=3e468db ] CB-4456 : Remove plugins-plist bits from the tests Support for plugins-plist was removed as of 3.4 in Feb 2014. It was replaced with a deprecation warning. This change is for removal of plugins-plist related bits in the tests.
      Hide
      Mark Koudritsky added a comment -

      But it should be pretty easy. I can remove it today or tomorrow.

      Show
      Mark Koudritsky added a comment - But it should be pretty easy. I can remove it today or tomorrow.
      Hide
      Mark Koudritsky added a comment -

      No, left it in, under a deprecation warning.

      Show
      Mark Koudritsky added a comment - No, left it in, under a deprecation warning.
      Hide
      Shazron Abdullah added a comment -

      Mark Koudritsky Have you removed this? Per the Deprecation policy it should have been removed in 3.4.0
      https://wiki.apache.org/cordova/DeprecationPolicy

      Show
      Shazron Abdullah added a comment - Mark Koudritsky Have you removed this? Per the Deprecation policy it should have been removed in 3.4.0 https://wiki.apache.org/cordova/DeprecationPolicy
      Hide
      Mark Koudritsky added a comment - - edited

      As 3.4 is already in the works, is it ok if I remove support for plugins-plist in config-changes.js ?
      I'm refactoring that file right now, would be glad to just remove anything that has to deal with plugins-plist.

      Show
      Mark Koudritsky added a comment - - edited As 3.4 is already in the works, is it ok if I remove support for plugins-plist in config-changes.js ? I'm refactoring that file right now, would be glad to just remove anything that has to deal with plugins-plist.

        People

        • Assignee:
          Braden Shepherdson
          Reporter:
          Filip Maj
        • Votes:
          0 Vote for this issue
          Watchers:
          4 Start watching this issue

          Dates

          • Created:
            Updated:

            Development