Skip to content

Kobo: Unbreak automagic_sysfs on the Libra 2#9998

Merged
NiLuJe merged 2 commits into
koreader:masterfrom
NiLuJe:master
Jan 6, 2023
Merged

Kobo: Unbreak automagic_sysfs on the Libra 2#9998
NiLuJe merged 2 commits into
koreader:masterfrom
NiLuJe:master

Conversation

@NiLuJe

@NiLuJe NiLuJe commented Jan 6, 2023

Copy link
Copy Markdown
Member

Regression since #9902, because I'd forgotten that the default Kobo block does set those...

Reported by @ptrm on Gitter, thanks!


This change is Reviewable

Regression since koreader#9902, because I'd forgotten that the default Kobo
block *does* set those...

Reported by @ptrm on Gitter, thanks!
@NiLuJe

NiLuJe commented Jan 6, 2023

Copy link
Copy Markdown
Member Author

Wait, that doesn't work, forgot that this is a real class with inheritance. Argh.

We have defaults for most of those in the Kobo superclass, so these can
never really be nil.
@NiLuJe

NiLuJe commented Jan 6, 2023

Copy link
Copy Markdown
Member Author

There, learned my lesson: don't try to be cute ;p.

@NiLuJe NiLuJe marked this pull request as ready for review January 6, 2023 01:21
@Frenzie Frenzie added this to the 2023.01 milestone Jan 6, 2023

@Frenzie Frenzie left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks okay, fingers crossed and all.

@ptrm

ptrm commented Jan 6, 2023

Copy link
Copy Markdown
Contributor

Checked the new frontend/device/kobo/device.lua with v2022.11-77-gb3a7d32d_2023-01-05 on Libra 2, and Clara 2e just in case, and it works, thanks :)
Libra 2 battery level

As a side note, which is probably something different, Libra 2 in USBMS mode was showing battery level correctly all the time, while Clara 2e shows it's at 0% (though 2e wasn't affected by the kobo/device.lua bug):
2e usbms

@NiLuJe

NiLuJe commented Jan 6, 2023

Copy link
Copy Markdown
Member Author

@ptrm: Yup, I'd forgotten the Clara 2E bits in the USBMS tool ;o).

@NiLuJe NiLuJe merged commit 5c13057 into koreader:master Jan 6, 2023
0xstillb pushed a commit to 0xstillb/koreader-thai that referenced this pull request May 9, 2026
Regression since koreader#9902, because I'd forgotten that the default Kobo block *does* set those...
Reported by @ptrm on Gitter, thanks!

Take two, essentially revert 5974425
We have defaults for most of those in the Kobo superclass, so these can never really be `nil`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants