lighthouse icon indicating copy to clipboard operation
lighthouse copied to clipboard

Create a release for the tablet (x86)

Open cgfrost opened this issue 9 years ago • 10 comments
trafficstars

A few people are getting tablets now. If it's straight forward to do a x86 release in harbor would be great.

cgfrost avatar Mar 01 '16 11:03 cgfrost

I was wondering that as well, so I tried a quick build in the SDK. Some variable had to be changed to a number in a .cpp to get it compiling (unknown function call and argument somewhere), but then lighthouse ran on the jolla tablet. It's not entirely nice, though: It shows no battery info and insists on the cpu temperature being extremely high – wrong by a factor of 1000?

jgibbon avatar Mar 18 '16 23:03 jgibbon

Hmm that's a bit of an odd result. I expected it to either work ok or not work at all (with GUI problems of course).

Are you able to ssh into your tablet? How confident are you with that? All the info is read from the /proc folder.

2016-03-18 17:53 GMT-06:00 jgibbon [email protected]:

I was wondering that as well, so I tried a quick build in the SDK. Some variable had to be changed to a number in a .cpp to get it compiling (unknown function call and argument somewhere), but then lighthouse ran on the jolla tablet. It's not entirely nice, though: It shows no battery info and insists on the cpu temperature being extremely high – wrong by a factor of 1000?

— You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub https://github.com/almindor/lighthouse/issues/11#issuecomment-198586427

Feel the power of Opensource. Feel the power of Free Pascal.

almindor avatar Mar 19 '16 16:03 almindor

Yes, ssh is available and used daily, linux user.
I'd be glad to check some stuff in /proc and report back if you want me to… could you tell me what you like me to "cat" or run? edit: screenshots edit2: what I changed to get it to compile was: I changed a line in monitor.cpp to fTicksPerSecond = 1;//sysconf(_SC_CLK_TCK); because… I don't know C++ and just guessed. ;)

jgibbon avatar Mar 19 '16 23:03 jgibbon

The modifications for the tablet to be made are rather small: For the various battery checks: replace the path /sys/class/power_supply/battery/ to /sys/class/power_supply/dollar_cove_battery/

And the /sys/class/thermal/thermal_zone0/temp result needs to be divided by 1000.

BTW I could compile for the tablet out of the box (no code change needed).

a-dekker avatar Mar 27 '16 11:03 a-dekker

I've merged the harbour branch now to master. This will make things easier for contributors and hopefully a-dekker can re-do his pull requests for the tablet and we can fix this.

almindor avatar Jun 01 '16 17:06 almindor

third (and hopefully final) pull request is made.

a-dekker avatar Jun 04 '16 09:06 a-dekker

Just noticed the new version arrived in the Jolla store, including a i486 tablet version.

a-dekker avatar Jun 07 '16 21:06 a-dekker

Can someone please test latest master (build manually) and tell me if everything looks good now? I fixed some issues with sizing etc.

almindor avatar Aug 12 '16 01:08 almindor

Personally I think the cpu/mem bar in processes got too big, both on the Jolla 1 and tablet, losing a more compact overview. I rather have a size in between the old and new value, but that might be a personal taste.

a-dekker avatar Aug 13 '16 10:08 a-dekker

Yeah it's hard to find a good spot. I'll have to see if there's a DPI/realsize way of calculating correct point size for this.

The reason it got bigger is I switched to Theme.xxxfontsize due to people on Nexus phones complaining of the selection being too small for a finger.

almindor avatar Aug 13 '16 19:08 almindor