Latest Posts

Topic: Updated flatpak available

tothxa
Avatar
Joined: 2021-03-24, 12:44
Posts: 212
Ranking
Widelands-Forum-Junkie
Posted at: 2022-09-06, 10:44

@Nordfriese

Now that we can (hopefully) bump the runtime version, I'm wondering whether I should keep building the flatpaks for v1.1 development and later the release with clang 14 instead of GCC 12.1. It seems to be somewhat faster on the buildbot and the executable is about 1MB smaller. (clang-14 ~10.5MB vs. GCC-12.1 ~11.5MB vs. older GCC ~12.3MB) Normally I'd stick to GCC, as it is the default compiler for the runtime and clang is an extension, but AIUI GCC 12.1 was practically a beta release that is now superseded by 12.2.

I'd drop the extension for v1.2 though if the next runtime version has a stable default compiler.

Is Widelands supposed to be neutral regarding the compiler or is either one "more official" than the other?


@hessenfarmer

Can we bump the commit for the second round of the tournament to allow testing the flatpak with the 22.08 runtime? (of course I will not update it until the first round is over)


Top Quote
hessenfarmer
Avatar
Joined: 2014-12-11, 23:16
Posts: 2280
Ranking
One Elder of Players
Location: Bavaria
Posted at: 2022-09-06, 11:24

tothxa wrote:

@hessenfarmer

Can we bump the commit for the second round of the tournament to allow testing the flatpak with the 22.08 runtime? (of course I will not update it until the first round is over)

I wanted to avoid having different versions for the tournament as far as possible. So only if a bug arises that makes playing impossible or very annoying then we should change the version for the tournament.

Edit: However the release candidate is scheduled for end of october so if the RC is out before the last round we willö use the rc for this round.

Edited: 2022-09-06, 11:25

Top Quote
Nordfriese
Avatar
Joined: 2017-01-17, 18:07
Posts: 1660
Ranking
One Elder of Players
Location: 0x55555d3a34c0
Posted at: 2022-09-06, 17:08

tothxa wrote:

@Nordfriese

Now that we can (hopefully) bump the runtime version, I'm wondering whether I should keep building the flatpaks for v1.1 development and later the release with clang 14 instead of GCC 12.1. It seems to be somewhat faster on the buildbot and the executable is about 1MB smaller. (clang-14 ~10.5MB vs. GCC-12.1 ~11.5MB vs. older GCC ~12.3MB) Normally I'd stick to GCC, as it is the default compiler for the runtime and clang is an extension, but AIUI GCC 12.1 was practically a beta release that is now superseded by 12.2.

I'd drop the extension for v1.2 though if the next runtime version has a stable default compiler.

Is Widelands supposed to be neutral regarding the compiler or is either one "more official" than the other?

My opinion here is that although we should target as many platforms and compilers as possible, better compilers deserve more prominent treatment. Adding a rule to the CMakeLists that tells the user "Your compiler is buggy" is not exactly neutral, is it face-smile.png

So +1 for using Clang.

@hessenfarmer

Can we bump the commit for the second round of the tournament to allow testing the flatpak with the 22.08 runtime? (of course I will not update it until the first round is over)

I actually wanted to ask the same thing because I'd prefer #5545 to get some more testing. If there is a new bug there, it would be highly platform- and libraries-versions-specific, but it will be much easier to discover than to fix.


Top Quote
tothxa
Avatar
Joined: 2021-03-24, 12:44
Posts: 212
Ranking
Widelands-Forum-Junkie
Posted at: Yesterday 09:15

I think I'll update the beta flatpak to current master using the 22.08 runtime and clang anyway, because they need testing before the RC. Even if the tournament stays on the previous version I believe there's not much risk in it:

  1. current master should be compatible (sync-safe) with the tournament version
  2. I don't think any of the participants actually uses the flatpak (please let me know if I'm wrong)

I'll wait for objecions until @hessenfarmer can give his opinion.

edit:

meanwhile, the test build is available as:

flatpak install --user https://dl.flathub.org/build-repo/110250/org.widelands.Widelands.flatpakref
Edited: Yesterday 09:21

Top Quote