[ https://issues.apache.org/jira/browse/CB-5012?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13789744#comment-13789744 ]
ASF subversion and git services commented on CB-5012: ----------------------------------------------------- Commit 64ce537ddabd659d36d6dfc8b8fd4cdfbd8de5ac in branch refs/heads/master from [~goya] [ https://git-wip-us.apache.org/repos/asf?p=cordova-plugman.git;h=64ce537 ] [CB-5012]: No whitespace in empty plist string nodes. > Crash occurs because plugman adds extra information in iOS Info.plist when > installing a plugin > ---------------------------------------------------------------------------------------------- > > Key: CB-5012 > URL: https://issues.apache.org/jira/browse/CB-5012 > Project: Apache Cordova > Issue Type: Bug > Components: Plugman > Affects Versions: Master > Reporter: Shazron Abdullah > Assignee: Brett Rudd > Priority: Critical > Fix For: 3.2.0 > > Attachments: KeyboardTest-Info.after-plugman.plist, > KeyboardTest-Info.original.plist > > > See attached .original.plist and .after-plugman.plist. > The plugin.xml used is: > https://github.com/apache/cordova-labs/blob/86fb3afef5aaaba8e82e5bf0719b74b4a07582e0/statusbar/plugin.xml > (note the value it tries to add in the iOS Info.plist). > What plugman does is, it adds extra info (newlines, spaces) as the string > value for the keys NSMainNibFile and NSMainNibFile~ipad in Info.plist. > They had no values before. At runtime, iOS will search for these "values" and > not finding the .nib files to load, then crash. > This is critical for plugins that need to modify the Info.plist, like the > Facebook PhoneGap plugin. There was a pull request for this, but I don't know > why it was closed exactly: https://github.com/apache/cordova-plugman/pull/21 > Expected: > plugman should not modify plist keys that have empty string values. > Actual: > plugman adds a newline, 6 spaces, and another newline for keys that have > empty string values. -- This message was sent by Atlassian JIRA (v6.1#6144)