1748 replies to this topic

[Persian Prince]

This would fix it: https://github.com/O...rvicemp3/pull/7

 

It's not just "Update PluginComponent.py" as it's the title, see the info under it: "Remove ancient ".pyc""

[littlesat]

Omg..... indeed pp avoid it! Sorry I did not notice it. And indeed pp can make better descriptions... e.g Oe-a colleagues have tons of it and afterwards when something does not work it is extreme difficult to find the cullpit when the comment is just tweak, fix... etc...

Edited by littlesat, 8 July 2018 - 10:19.

[Persian Prince]

The fault is mine, sorry.

 

Regarding commit info, I follow github so few words in the title and the rest in extended description, isn't it right?

[littlesat]

And now I’m lost... but it looks like Erik wants te remove the previous merge... add a fix for the old white spaces and then make a new commit with the final ‘work’...

[Erik Slagter]

I did a revert from github, but apparently it's made into a PR as well, instead of a proper revert. I am not sure what the status is now. Next friday I can investigate.

[WanWizard]

Github does everything via a PR.

[Erik Slagter]

Lesson learned. For low level corrective actions, use command line as usual.

[Persian Prince]

https://github.com/O...e-core/pull/507

[littlesat]

Merged.... 

[Persian Prince]

https://github.com/O...igma2/pull/1658

Attached Files

•  screenshot_19700101013113.jpg   66.7KB   5 downloads

Edited by Persian Prince, 28 July 2018 - 08:46.

[littlesat]

Cannot be accepted... sorry..
There is an if box x then do y in the code
A three line read file... the wierdness openatv does anywhere... making things extra complicated...
There is a missing else that makes the code more readable...
When other boxes do not have a proc chipset just read that one else do what we did -or- tell zgemma to change the procs...

Edited by littlesat, 28 July 2018 - 13:03.

[Persian Prince]

https://github.com/O...igma2/pull/1659

 

I thinl it's better now

[littlesat]

And when you read chipset alternaly later when the previous stuff had no result... then you fix it for other (futer) boxes and you avoid a systeminfo.

Edited by littlesat, 29 July 2018 - 08:46.

[rantanplan]

https://github.com/O...2f8845ffc0d400c

 

After building the image, there is a reboot loop with the Python problem.

Does not seem to hit every box, but maybe someone wants to and looks at it.

[littlesat]

Can you post the crashlog here.... As far I can see the code you point to should not be related...

[rantanplan]

I've written the commit.

Is not an 'official' but Homebuild Image.
However, it is a python error that could possibly hit other official buid as well.

That's why I asked for it.

Lines 285 and 287 in AvSwitch.py ​​are output as crash.

[littlesat]

Repeat post crashlog... I don’t see what is wrong there...

Edited by littlesat, 31 July 2018 - 19:59.

[rantanplan]

no crashlog, reboot loop

 

log with telnet

openpli homebuild spark7162

spark7162 login: root
root@spark7162:~# init 4
root@spark7162:~# enigma2
PYTHONPATH: /usr/lib/enigma2/python
DVB_API_VERSION 5 DVB_API_VERSION_MINOR 10
ENIGMA_DEBUG_LVL=3
[MAIN] executing main
Traceback (most recent call last):
  File "/usr/lib/enigma2/python/mytest.py", line 545, in <module>
    Components.AVSwitch.InitAVSwitch()
  File "/usr/lib/enigma2/python/Components/AVSwitch.py", line 287, in InitAVSwitch
    config.av.hdmi_audio_source.addNotifier(setHDMIAudioSource)
  File "/usr/lib/enigma2/python/Components/config.py", line 131, in addNotifier
    notifier(self)
  File "/usr/lib/enigma2/python/Components/AVSwitch.py", line 285, in setHDMIAudioSource
    open(SystemInfo["HDMIAudioSource"], "w").write(configElement.value)
TypeError: coercing to Unicode: need string or buffer, bool found
root@spark7162:~#

As I said, yes, is not an official image,
but maybe the mistake is so relevant.

[littlesat]

I think this is more related to a mistake in the not official image...

[littlesat]

Uuuh you are right fileexists in systeminfo should be filecheck....