koreader / android-luajit-launcher

Android NativeActivity based launcher for LuaJIT, implementing the main loop within Lua land via FFI
MIT License
130 stars 83 forks source link

tolino page 2 #455

Closed pazos closed 7 months ago

pazos commented 8 months ago

Coming from #11264, KOReader works just fine with the android-arm apk. The drivers that are working are "Tolino Ntx" and "Tolino Root" for lights and "Freescale/NTX" for e-ink. The driver test also states Device already supported, EPD: freescale, Lights: tolino. The only two minor concerns I have are

  1. Both working light drivers have a warmth slider but the device doesn't support warmth.
  2. KOReader is unable to enable/disable the frontlight, it can only change the brightness. Toggling the light has to be done outside of KOReader by tapping the android status bar.
test.log ``` Device info: Manufacturer: rakuten kobo inc. Brand: rakutenkobo Model: tolino Device: ntx_6sl Product: ntx_6sl Hardware: e60qv0 Platform: imx6 Test logs: ```

The device uses Android 4.4.2 with kernel version 3.0.35-gb5779db-dirty

Originally posted by @RubixDev in https://github.com/koreader/koreader/issues/8482#issuecomment-1868903843

pazos commented 8 months ago
  1. Both working light drivers have a warmth slider but the device doesn't support warmth.

Feel free to submit a patch with a new controller. Just copy the relevant class with a different name, set hasWarmth to false and remove warmth functions.

  1. KOReader is unable to enable/disable the frontlight, it can only change the brightness. Toggling the light has to be done outside of KOReader by tapping the android status bar.

TolinoRoot controller requires root to toggle frontlight. That's not something fixable. Feel free to play with TolinoNTX controller, see if that works and then change the assigned driver in LightsFactory.kt.

RubixDev commented 8 months ago

Feel free to submit a patch with a new controller. Just copy the relevant class with a different name, set hasWarmth to false and remove warmth functions.

I can try that, I would just like some guidance on how I can test my changes locally.

TolinoRoot controller requires root to toggle frontlight. That's not something fixable. Feel free to play with TolinoNTX controller, see if that works and then change the assigned driver in LightsFactory.kt.

Ah, TolinoNTX is able to toggle the frontlight. I just didn't realise the root driver stayed selected after exiting the compat test screen.

pazos commented 8 months ago

See https://github.com/koreader/koreader/blob/master/doc/Building.md and https://github.com/koreader/koreader/blob/master/doc/Building_targets.md.

You need to build an APK since you're playing with compiled bytes.

Once you're done setting your environment a simple ./kodev release android should be enough to produce an APK. You need to sign it (1st example of https://github.com/patrickfav/uber-apk-signer)

mergen3107 commented 8 months ago

You need to sign it

Oh, I've never done this. Yet, I was able to install any built debug apks. Is this correct, or should I start signing them too?

pazos commented 8 months ago

Debug builds are good just for debugging. They weight a lot, assets extraction is a lot slower and the program is marginally slower as well.

The only thing you get for free with a debug build is a signature. You can get the same signature with the method above (without the requirement of creating your own key) and have a release build, without nonsense.

mergen3107 commented 8 months ago

I see, thanks! Turns out, I haven't released anything non-debug yet :D