News:

Want to praise Simutrans?
Your feedback is important for us ;D.

GitHub Actions CI builds

Started by ceeac, August 16, 2020, 08:05:26 AM

Previous topic - Next topic

0 Members and 1 Guest are viewing this topic.

ceeac

Following the discussion in this thread I made CI builds for Linux and macOS using GitHub Actions.
Please note that while everything compiles successfully, the generated zip packages probably do not work (yet). So for now this is only useful to check whether a commit compiles.
I also added a check which runs cleanup_code.sh to check for trailing whitespace.
The corresponding WIP pull request is here: #252.

freddyhayward

I tested the Linux SDL2 build and it works fine after copying the Pak128.Britain-Ex folder into the directory.

prissi

IN principle one can also make windows build, provided one patches the vcx... files to include the needed libraries. Or one can use the msys2 system. Both were so much more complicated than crosscompiling on my server, so I abanded it. (Even more since the current SDL2 from MSYS2 does not allow static linking, while a self-built one has no such problems ...)

jamespetts

I have merged the pull request, but I am now receiving regular e-mail from Github telling me that the builds have failed:


Run failed for master (f043bf6)

Repository: jamespetts/simutrans-extended
Workflow: CI
Duration: 4 minutes and 20.0 seconds
Finished: 2020-08-16 11:36:31 UTC

View results
Jobs:

    Code Formatting succeeded (0 annotations)
    Linux (Allegro, single-threaded) succeeded (0 annotations)
    Linux (headless) failed (1 annotation)
    Linux (SDL2, multi-threaded, freetype) succeeded (0 annotations)
    Linux (SDL1) succeeded (0 annotations)
    Linux (OpenGL) succeeded (0 annotations)
    Linux Makeobj succeeded (0 annotations)
    Linux (SDL1 + SDL_mixer) succeeded (0 annotations)
    Linux (Debug SDL2, multi-threaded) succeeded (0 annotations)
    macOS Makeobj succeeded (0 annotations)
    Linux Nettool succeeded (0 annotations)
    macOS Nettool succeeded (0 annotations)
    macOS (SDL2) failed (1 annotation)


You are receiving this because this workflow ran on your branch.
Manage your GitHub Actions notifications here.


I am not familiar with CI setups, I am afraid, so I am not sure how to interpret this.
Download Simutrans-Extended.

Want to help with development? See here for things to do for coding, and here for information on how to make graphics/objects.

Follow Simutrans-Extended on Facebook.

freddyhayward

Quote from: jamespetts on August 16, 2020, 12:37:20 PM
I have merged the pull request, but I am now receiving regular e-mail from Github telling me that the builds have failed:


Run failed for master (f043bf6)

Repository: jamespetts/simutrans-extended
Workflow: CI
Duration: 4 minutes and 20.0 seconds
Finished: 2020-08-16 11:36:31 UTC

View results
Jobs:

    Code Formatting succeeded (0 annotations)
    Linux (Allegro, single-threaded) succeeded (0 annotations)
    Linux (headless) failed (1 annotation)
    Linux (SDL2, multi-threaded, freetype) succeeded (0 annotations)
    Linux (SDL1) succeeded (0 annotations)
    Linux (OpenGL) succeeded (0 annotations)
    Linux Makeobj succeeded (0 annotations)
    Linux (SDL1 + SDL_mixer) succeeded (0 annotations)
    Linux (Debug SDL2, multi-threaded) succeeded (0 annotations)
    macOS Makeobj succeeded (0 annotations)
    Linux Nettool succeeded (0 annotations)
    macOS Nettool succeeded (0 annotations)
    macOS (SDL2) failed (1 annotation)


You are receiving this because this workflow ran on your branch.
Manage your GitHub Actions notifications here.


I am not familiar with CI setups, I am afraid, so I am not sure how to interpret this.
There's more detailed information on the github site:
build/default/gui/components/gui_chart.o: In function `gui_chart_t::draw(scr_coord)':
gui_chart.cc:(.text+0xc4c): undefined reference to `display_filled_roundbox_clip(short, short, short, short, unsigned short, bool)'
gui_chart.cc:(.text+0x11b3): undefined reference to `display_filled_roundbox_clip(short, short, short, short, unsigned short, bool)'

jamespetts

It is not clear whether this is caused by something specific in the CI setup or whether this is a new general problem with the code introduced by an unknown one of the changes integrated since the last successful build last night.
Download Simutrans-Extended.

Want to help with development? See here for things to do for coding, and here for information on how to make graphics/objects.

Follow Simutrans-Extended on Facebook.

Mariculous

It's not CI specific.
I got the same issue when compiling the headless binary on my system.

jamespetts

Quote from: Freahk on August 16, 2020, 01:15:16 PMdisplay_filled_roundbox_clip

Thank you for letting me know. I have pushed what I believe to be a fix; I should be grateful if you could re-test and confirm. Thank you.
Download Simutrans-Extended.

Want to help with development? See here for things to do for coding, and here for information on how to make graphics/objects.

Follow Simutrans-Extended on Facebook.

freddyhayward

Quote from: jamespetts on August 16, 2020, 01:24:49 PM
Thank you for letting me know. I have pushed what I believe to be a fix; I should be grateful if you could re-test and confirm. Thank you.
Thankfully, this is no longer necessary. The CI is working as intended!