apache / apache/cordova-node-xcode

Allow comma in buildSettings

未关闭
#86 1 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
主要语言
JavaScript
星标
217
派生
102
平均合并
2 天 3 小时
30 天内合并 PR
6

描述

when i try to set "TARGETED_DEVICE_FAMILY", it failed.
buildSettingsObj.TARGETED_DEVICE_FAMILY = '1,2'

```
While preparing Cordova project for platform iOS:
SyntaxError: Expected ".", "/*", ";", or [0-9] but "," found.
at peg$buildStructuredError (/Users/monsterstep/.meteor/packages/meteor-tool/.1.9.0.1eehdch.kfff++os.osx.x86_64+web.browser+web.browser.legacy+web.cordova/mt-os.osx.x86_64/dev_bundle/lib/node_modules/cordova-lib/node_modules/xcode/lib/parser/pbxproj.js:412:12)
at Object.peg$parse [as parse] (/Users/monsterstep/.meteor/packages/meteor-tool/.1.9.0.1eehdch.kfff++os.osx.x86_64+web.browser+web.browser.legacy+web.cordova/mt-os.osx.x86_64/dev_bundle/lib/node_modules/cordova-lib/node_modules/xcode/lib/parser/pbxproj.js:1886:11)
```

the xcodeproj has value like this

![image](https://user-images.githubusercontent.com/1147704/74629486-49e4df80-5193-11ea-9c87-73ffdf6929c2.png)

![image](https://user-images.githubusercontent.com/1147704/74628318-63385c80-5190-11ea-8b4c-5c3dc751e6a1.png)

https://github.com/apache/cordova-node-xcode/issues/57

贡献指南

打开贡献指南

调研方向

从错误中引用的 pbxproj.js 解析器开始,跟踪 buildSettingsObj.TARGETED_DEVICE_FAMILY 的解析或序列化方式。使用值 '1,2' 重现该失败,然后验证 Xcode 项目值能够被接受,且不会出现报告的逗号语法错误。

由索引模型根据 Issue 内容生成。

评估

技术栈
ios, javascript, nodejs
领域
build-system, mobile
Issue 类型
缺陷
难度
3/5
预计耗时
1-2 天
活跃度
停滞
描述清晰度
基本清楚
新手友好度
35/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。