dryark / stf_ios_support

Central repo to connect and document components/repos needed for IOS stf support
Other
153 stars 65 forks source link

can't use the latest commit to run #91

Closed chaooe closed 3 years ago

chaooe commented 3 years ago

i am using the lastest commit https://github.com/DeviceFarmer/stf_ios_support/commit/240875c31245b7064a17c23b2b91477aad091ca1 but failed to run as below 图片

INFO[0001] ideviceinfo call                              ops="[-u 00008030-001969593ADB802E -k ProductVersion]" type=ilib_getinfo_call
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x43bfa07]

goroutine 1 [running]:
main.NewRunningDev(0xc0002a2380, 0xc000283350, 0xc00029b3e0, 0xc0002834a0, 0xc000272025, 0x19, 0xc0002b40f8)
        /Users/alexzhao/Work/Archs/stf_ios_support/coordinator/coordinator.go:508 +0x307
main.event_loop(0xc0002a2380, 0xc00029b8f0, 0xe, 0xc0002961e0, 0xc0002804e0, 0xc00028e59c, 0x3, 0xc000296180, 0xc000283350, 0xc00029b3e0, ...)
        /Users/alexzhao/Work/Archs/stf_ios_support/coordinator/coordinator.go:619 +0x31bb
main.main()
        /Users/alexzhao/Work/Archs/stf_ios_support/coordinator/coordinator.go:422 +0x1511
E: 20-12-22 16:28:11 [52721]dangling 'REP' socket created at :336
E: 20-12-22 16:28:11 [52721]dangling 'PULL' socket created at :152
E: 20-12-22 16:28:11 [52721]dangling 'PUB' socket created at :28
chaooe commented 3 years ago

fixed . it need to add devices in th config.json file as below.

},
  "video": {
    "enabled": false,
    "use_vnc": false,
    "vnc_scale": 2,
    "vnc_password": "",
    "frame_rate": 5
  },
  "devices": [
  ],
  "install": {
    "root_path": "",
    "config_path": "./config.json",
    "set_working_dir": false
  },
nanoscopic commented 3 years ago

This commit also fixes this issue: https://github.com/DeviceFarmer/stf_ios_support/commit/5d5f3e7ec95f2e74e8c9a4f5df75785c3c6a6504

This was the intent to begin with; it should function even without devices being set in config. Thanks for making a ticket for this to quickly bring it to my attention.