Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build error on armhf architecture #8962

Closed
zoltantamasvajda opened this issue Apr 6, 2021 · 3 comments
Closed

Build error on armhf architecture #8962

zoltantamasvajda opened this issue Apr 6, 2021 · 3 comments

Comments

@zoltantamasvajda
Copy link

@zoltantamasvajda zoltantamasvajda commented Apr 6, 2021

Hello there lovely people of OpenTTD contributors!

I'm the maintainer of the openttd-server snap package which I've created for my own purposes but seems to be gaining a bit of popularity, which is nice.

What is not nice is that I seem to be having an issue with building the package for version 1.11.0 on armhf architecture.

Apparently there are at least 5 people out there who'd like to run a dedicated openttd server using snaps on armhf (Raspberry Pi?) and I'd hate to let them down.

I've attached the full cmake logs from the build pipelines but the relevant part seems to be this:

`GenerateIndustries()':
<artificial>:(.text+0x9fc8): undefined reference to `Pool<Industry, unsigned short, 64u, 64000u, (PoolType)1, false, true>::MAX_SIZE'
/usr/bin/ld: /tmp/openttd.LStJUE.ltrans19.ltrans.o: in function `GenerateTowns(TownLayout)':
<artificial>:(.text+0x8d7c): undefined reference to `Pool<Town, unsigned short, 64u, 64000u, (PoolType)1, false, true>::MAX_SIZE'

Any help would be appreciated. Thanks!

cmake.log

@LordAro
Copy link
Member

@LordAro LordAro commented Apr 6, 2021

Already fixed in #8924

@TrueBrain
Copy link
Member

@TrueBrain TrueBrain commented Apr 6, 2021

Good news! We fixed this issue in master :)

4d50165

It will be in our next release. Till that time, please patch the code yourself if you can. A new release won't be long btw.

@LordAro LordAro closed this Apr 6, 2021
@zoltantamasvajda
Copy link
Author

@zoltantamasvajda zoltantamasvajda commented Apr 6, 2021

Lovely! Thanks you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants