Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

add in sugar for escaping #4

Closed
yocontra opened this issue Oct 22, 2015 · 2 comments
Closed

add in sugar for escaping #4

yocontra opened this issue Oct 22, 2015 · 2 comments

Comments

@yocontra
Copy link
Contributor

Example where this currently breaks:

<preference name="ios-XCBuildConfiguration-LD_RUNPATH_SEARCH_PATHS" value="@executable_path/Frameworks" />

needs to be changed to

<preference name="ios-XCBuildConfiguration-LD_RUNPATH_SEARCH_PATHS" value="&quot;@executable_path/Frameworks&quot;" />

or else it corrupts the pbxproj file
@yocontra
Copy link
Contributor Author

Actually, just tried it: If you try to escape it in the XML cordova just logs "undefined" and crashes. Right now there is no workaround :(

yocontra pushed a commit to yocontra/cordova-custom-config that referenced this issue Oct 22, 2015
Merge pull request #1 from contra/patch-1
6a1452a 
escape values from preferences (closes dpa99c#4)
@dpa99c
Copy link
Owner

dpa99c commented Oct 31, 2015

Hi Eric, Thanks for raising these issues. I'm not ignoring them, just really busy right now. Will try to make some time to investigate/fix sometime this week.

@dpa99c dpa99c closed this as completed in a316a9b Nov 3, 2015
dpa99c added a commit that referenced this issue Nov 3, 2015
@dpa99c
Merge pull request #5 from contra/patch-1
00391ef 
escape values from preferences (closes #4)
@dpa99c dpa99c reopened this Nov 16, 2015
@dpa99c dpa99c closed this as completed Nov 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@yocontra @dpa99c