Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
    Martín Fernández
    @pazos
    fine for me
    NiLuJe
    @NiLuJe
    Yeah, fractional sleep is an optional busybox feature, while usleep is an "all or nothing" applet
    So that'd make the most sense, compatibility-wise ;)
    Martín Fernández
    @pazos
    ok, since it is called recursively I would go for usleep 500000 2>/dev/null || sleep 0.5
    NiLuJe
    @NiLuJe
    Yup
    NiLuJe
    @NiLuJe
    @pazos: Well, I'm kind of amazed I somehow managed to remember that, but that was indeed FW 2.x...
    [root@kindle root]# usleep 500000
    [root@kindle root]# sleep 0.5
    sleep: invalid number '0.5'
    \o/
    Frans de Jonge
    @Frenzie
    I was expecting that on Kobo tbh, but I tried it and was surprised to see that sleep 0.5 Just Worked™. (Isn't it one of those GNU extensions or whatever?)
    NiLuJe
    @NiLuJe
    It's a coreutils behavior that's optional in bb, yeah
    And usleep is bb only, funnily enough
    NiLuJe
    @NiLuJe
    Mwahahah, 8bpp on Kobo, I CAN PAN REALLY FAST GUYS§§!
    Frans de Jonge
    @Frenzie
    @NiLuJe Awesome! :-D
    Frans de Jonge
    @Frenzie
    @NiLuJe Shall I merge the gesture thingy tonight or wait until tomorrow/Tuesday?
    NiLuJe
    @NiLuJe
    My reply in the issue was meant to serve as a +1, yeah, go ahead ;).
    NiLuJe
    @NiLuJe
    @Frenzie : I sneaked in a crash report in https://github.com/koreader/koreader/pull/4607#issuecomment-464839072 in case it falls through the cracks ;)
    NiLuJe
    @NiLuJe
    I think that was on the nightly, but I might already have been using a local -56 build by then
    Frans de Jonge
    @Frenzie
    @NiLuJe Should be fixed by this one already I believe? koreader/koreader#4612
    NiLuJe
    @NiLuJe
    Quite possibly, haven't reproduced it yet on -56
    NiLuJe
    @NiLuJe
    Definitely like the idea, but can this work with symlinks instead?
    I usually see this sort of stuff handled with a symlink in oldschool http/ftp software mirrors
    Frans de Jonge
    @Frenzie
    Sure, I was talking more about the frontend code. ;)
    NiLuJe
    @NiLuJe
    That'd basically save you the parsing stage in the frontend, if stuff follows symlinks properly ;p
    Frans de Jonge
    @Frenzie
    Oh, that way. Hm, that wouldn't quite work
    It'd need to be a redirect, not a symlink
    otherwise you're downloading a file named "latest-nightly"
    or in any case it'd need HTTP header shenanigans :P
    NiLuJe
    @NiLuJe
    Does the frontend actually care about the filename? (IIRC, zsync stuffs ends up in updated.tar or something)
    Frans de Jonge
    @Frenzie
    No, but I sure do! ;)
    NiLuJe
    @NiLuJe
    Getting the version from it maybe, if that bypasses the zsync file
    Frans de Jonge
    @Frenzie
    Hm, perhaps I phrased the PR a bit confusingly?
    It doesn't affect zsync
    NiLuJe
    @NiLuJe
    No, just thinking aloud ;)
    Frans de Jonge
    @Frenzie
    The parsing in front is about getting the version though, btw
    NiLuJe
    @NiLuJe
    Yeah, that was my train of thought too ^^
    So, yeha, no, looks good. What happened to the isAndroid branch for the tar stuff?
    (there was a -C .. in some platforms and not others, IIRC)
    Frans de Jonge
    @Frenzie
    Oh yeah, I'll add a link to the rationale, which was mostly hidden here: https://github.com/koreader/koreader/issues/4599#issuecomment-464364860
    wait, wrong link
    NiLuJe
    @NiLuJe
    ^^
    NiLuJe
    @NiLuJe
    Oh, right, no need to create a local tarball if it's no longer using zsync, of course ;)
    Frans de Jonge
    @Frenzie

    The whole zsync idea is pretty cool but I just don't really see it working on Android, even setting aside that the OTA update would still have to be fixed. I got rid of the worst error mostly as a side effect, but it's still not working.

    But think of it:

    1. User downloads OTA update.
    2. Restart to update…
    3. Update says KOReader APK too old, download a new one…
    4. User: :grimacing: :grimacing: :grimacing:
    (not to mention a thing to manually have to update)
    NiLuJe
    @NiLuJe
    Yeah, tat definitely can't integrate properly on Android
    Frans de Jonge
    @Frenzie
    @NiLuJe Do you have any comments on the dummy in koreader/koreader-base#826 ?
    NiLuJe
    @NiLuJe
    XSCreenSaver should have a pretty stable API, so, yeah, that sounds sensible.
    I see Fedora is still on the helm of doing weird stuff to Linux :D