[
https://issues.apache.org/jira/browse/CB-12813?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16255432#comment-16255432
]
ASF GitHub Bot commented on CB-12813:
-------------------------------------
janpio commented on a change in pull request #21: CB-12813 ios Fix target
chooser to use full target name
URL: https://github.com/apache/cordova-paramedic/pull/21#discussion_r151432524
##########
File path: lib/ParamedicTargetChooser.js
##########
@@ -128,12 +128,10 @@ ParamedicTargetChooser.prototype.chooseTargetForWindows
= function (emulator, ta
ParamedicTargetChooser.prototype.chooseTargetForIOS = function (emulator,
target) {
logger.info('cordova-paramedic: Choosing Target for iOS');
- var simulatorModelId = util.getSimulatorModelId(this.cli, target);
- var split = simulatorModelId.split(', ');
- var device = split[0].trim();
- var simId = util.getSimulatorId(simulatorModelId);
+ var deviceOSVersion = util.getSimulatorModelId(this.cli, target);
Review comment:
variable name is strange.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
> Cordova Paramedic is not supporting full target name with OS version
> --------------------------------------------------------------------
>
> Key: CB-12813
> URL: https://issues.apache.org/jira/browse/CB-12813
> Project: Apache Cordova
> Issue Type: Bug
> Components: cordova-paramedic
> Affects Versions: Master
> Reporter: Tomasz Subik
>
> Cordova paramedic is not supporting full target name with OS version. When
> providing --target "'iPhone-6, 9.3'" it is simply running first iPhone-6
> emulator not the exact one I want.
> Logs:
> cordova-paramedic: Choosing Target for iOS
> running:
> cordova run --list --emulator --no-telemetry --no-update-notifier | grep
> 'iPhone-6, 9.3' | tail -n1
> running:
> instruments -s devices | grep ^iPhone
> cordova-paramedic: Setting required permissions.
> cordova-paramedic: running command cordova run ios --no-telemetry
> --no-update-notifier --target iPhone-6 --emulator
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
