Topic: Github action workflow for android nightly build

[Yona-TYT]

--

[Dwachs]

@prissi, additional patch, that ties in-game logging to Android logcat. Android redirects stdout and stderr to /dev/null, and introduces different logging functions for logcat, The patch below applicable to r10049 makes use of them. For those who want to capture the logs, add the command line parameters -debug x at the beginning (SDL config), and capture logs via adb logcat -s com.simutrans.

This patch could use cbuffer_t to get rid of fixed size char buffers. Then it could be included.

[Dwachs]

submitted the fluidsynth patch in r10084.

[krosk]

@Dwachs
As instructed, the attached patch (applicable to r10084) has switched to cbuffer_t.

[Yona-TYT]

I am very happy with this project!

There is something very important that I think we are overlooking.

Zoom in and out with two fingers, and it is that inplementing this in android would greatly improve the playability on touch screens. :C


It would be a shame if we don't have this in the first official version released for android.

What do you think of this friends? .

[Dwachs]

submitted the logging-patch in r10085.

[prissi]

The zoom gesture is only supported with SDL2 ... And then it was not working. I submitted something that makes SDL2 useful with finger on my touchscreen under window in r10087/10088

[krosk]

@Yona-TYT
Libsdl android has a handy feature which is able to map pinch gestures to a command.
In this case, I mapped the pinch gestures to emulate mouse wheel, which achieves map zoom in/out via pinch.
However this is not an optimal fix; for example mouse wheel also controls list scrolling, and pinching a list will make it scroll up/down; definitely an incorrect behavior.
Nonetheless, I will submit the patch later since the pros are bigger than the cons.

@prissi
Good news; I make it a goal to move the android port to SDL 2, so it will benefit from native touch support with SDL2 from other platforms.

[prissi]

Since windows also trnslates pinch to scroll wheel events, also SDL2 will behave the same ... i.e. scrolling in lists with pinch.

[Yona-TYT]

 

The zoom gesture is only supported with SDL2 ... And then it was not working. I submitted something that makes SDL2 useful with finger on my touchscreen under window in r10087/10088

Thank you very much, I will try as soon as it is available. 
Edit.
Ah wait! ...   This is just SDL2, and the android version is not ported yet, therefore it is not possible for me to test.     

@prissi Good news; I make it a goal to move the android port to SDL 2, so it will benefit from native touch support with SDL2 from other platforms.

This is great! I'm happy to hear this!.   

[krosk]

Patch applicable to r10089.

This patch configures libsdl-android to map pinching touch events to mouse wheel (they were previously mapped to keyboard + and keyboard -); the cfg file change is on the libsdl-android repo fork.

[prissi]

Thanks, submitted. in r10092.

[Yona-TYT]

For some reason it no longer works on my device, it closes right after stop loading.

[krosk]

Patch applicable to r10103.

The github actions pipeline is again silently passing upon failure to download the soundfont (occasional 502 Bad Gateway reported by wget, https://github.com/aburch/simutrans/runs/3583126278?check_suite_focus=true#step:14:25170), which broke the apk reported by @Yona-TYT.
Previous patch was not exhaustive enough, so this one will retry for any reason except success and ensure build failure upon any error.


Additionally, as a bonus background work, the libsdl-android now downloads prebuilt openssl binaries, which should reduce building time by 10 minutes.

[prissi]

Commited, thank you

[Yona-TYT]

Well the above problem seems to be fixed.

Now I have a new problem, this time with the resolution of the screen, before it looked better on my device, now it does not allow me to do anything.   
 Look at the image as it looks:

[krosk]

@Yona-TYT
- What is the model of your device? This will let me know about your device native resolution and screen size for reference, to check if code behavior is expected.
- On the SDL prompt at the start on top of the screen there is a button "Change device configuration". Click on it, then on "Command line parameters, one argument per line":
  * if you remove the line "-autodpi", it will switch to the native resolution with the correct aspect ratio.
  * If the above is too small, you can customize resolution by typing "-screensize" in one line, and "800x600" in the next line; change the numbers for the resolution you want to have. Let me know also which resolution looks nice.

[Yona-TYT]

- What is the model of your device?

Alcatel one 5033E.

[Yona-TYT]

screen: 5" 18:9

[krosk]

@Yona-TYT
I had a look; your device native resolution is 960x480.
So removing the -autodpi argument in the command line options should be fine for you, even without setting a screensize. The resolution is low enough to display a big enough UI.

[prissi]

480 is a little bit too low to display most dialogues. So maybe oversize resolution to 1200x600

[Yona-TYT]

I'm not sure if "-screensize" is taking effect, as I can't tell the difference.

Code: [Select]

-screensize 1200x600

[krosk]

@Yona-TYT,
you need to write it on two separate lines:

-screensize
1200x600

Try it, but I think it won't work because it is above your smartphone resolution...