planetfederal / qgis-reports-plugin

A QGIS plugin to collect system information for debug purposes
Other
2 stars 4 forks source link

python error running on macOS #3

Closed gioman closed 7 years ago

gioman commented 7 years ago

tested on b1 installer and macOS 10.12.5

Traceback (most recent call last):
  File "/Library/Boundless/Desktop/1.1/Cellar/qgis2-bdesk/2.18.10/QGIS.app/Contents/MacOS/../Resources/python/plugins/reportingtool/plugin.py", line 107, in run
    filePath = self.lastReport()
  File "/Library/Boundless/Desktop/1.1/Cellar/qgis2-bdesk/2.18.10/QGIS.app/Contents/MacOS/../Resources/python/plugins/reportingtool/plugin.py", line 123, in lastReport
    return reports[-1][1]
IndexError: list index out of range
gioman commented 7 years ago

@alexbruy tried the latest commit upgrading from dev repo

IndexError: list index out of range 
Traceback (most recent call last):
  File "/Users/gmanghi/.qgis2/python/plugins/reportingtool/plugin.py", line 112, in run
    report, filePath = createReport()
  File "/Users/gmanghi/.qgis2/python/plugins/reportingtool/ext-libs/qgissysinfo/createreport.py", line 49, in createReport
    report = qgissysinfo.info_as_text()
  File "/Users/gmanghi/.qgis2/python/plugins/reportingtool/ext-libs/qgissysinfo/__init__.py", line 31, in info_as_text
    return _as_text(info())
  File "/Users/gmanghi/.qgis2/python/plugins/reportingtool/ext-libs/qgissysinfo/__init__.py", line 8, in info
    _info = systeminfo.allSystemInfo()
  File "/Users/gmanghi/.qgis2/python/plugins/reportingtool/ext-libs/qgissysinfo/systeminfo.py", line 62, in allSystemInfo
    info = systemInfo()
  File "/Users/gmanghi/.qgis2/python/plugins/reportingtool/ext-libs/qgissysinfo/systeminfo.py", line 81, in systemInfo
    "Processor": _cpuInfo(),
  File "/Users/gmanghi/.qgis2/python/plugins/reportingtool/ext-libs/qgissysinfo/systeminfo.py", line 173, in _cpuInfo
    info = subprocess.check_output("sysctl -n machdep.cpu.brand_string", shell=True, universal_newlines=True).split(":")[1].strip()
IndexError: list index out of range
alexbruy commented 7 years ago

Last error caused by changes in sysctl output. Should be fixed now, please retest

gioman commented 7 years ago

@alexbruy @luipir I guess this can be closed? This has been re-tested on macOS?

luipir commented 7 years ago

correct