apache / apache/cordova-node-xcode

Allow comma in buildSettings

オープン
#86 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
JavaScript
スター
217
フォーク
102
平均マージ
2日 3時間
マージ済み PR(30日)
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 を短くまとめたダイジェスト。