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

Ports: Update freeciv to version 3.1.1 #23882

Merged
merged 2 commits into from
Jun 2, 2024

Conversation

kennethmyhra
Copy link
Member

@kennethmyhra kennethmyhra commented Apr 7, 2024

  • Updates freeciv to version 3.1.1.
  • Sets '--enable-fcdb=no'. The default has been changed from 'no' in previous versions to 'sqlite3' in this version.
  • Formats the package script according to our ruleset.

@github-actions github-actions bot added the 👀 pr-needs-review PR needs review from a maintainer or community member label Apr 7, 2024
Ports/freeciv/package.sh Outdated Show resolved Hide resolved
Copy link
Member

@timschumi timschumi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Currently does not build, at least on my machine:

[freeciv/build] ld: /serenity/Toolchain/../Build/x86_64/Root/usr/lib/crt0.o: undefined reference to symbol '__begin_atexit_locking'
[freeciv/build] ld: /serenity/Toolchain/../Build/x86_64/Root/usr/lib/libc.so: error adding symbols: DSO missing from command line

Copy link

stale bot commented May 1, 2024

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions!

@stale stale bot added the stale label May 1, 2024
@kennethmyhra kennethmyhra removed the stale label May 2, 2024
@kennethmyhra kennethmyhra marked this pull request as draft May 2, 2024 08:28
@github-actions github-actions bot removed the 👀 pr-needs-review PR needs review from a maintainer or community member label May 2, 2024
This formats freeciv's package script according to our ruleset.
Sets '--enable-fcdb=no'. The default has been changed from 'no' in
previous versions to 'sqlite3' in this version.
@kennethmyhra kennethmyhra changed the title Ports: Update freeciv to version 3.1.0 Ports: Update freeciv to version 3.1.1 Jun 2, 2024
@kennethmyhra kennethmyhra marked this pull request as ready for review June 2, 2024 09:16
@github-actions github-actions bot added the 👀 pr-needs-review PR needs review from a maintainer or community member label Jun 2, 2024
@kennethmyhra
Copy link
Member Author

Figured out why this was failing: The default value for the option --enable-fcdb has changed from 'no' in previous versions to 'sqlite3' in this version.

Still not sure why it worked on some machines, like my laptop, but I guess that can be a future task for figuring out how to enable sqlite3 for this port.

@timschumi timschumi merged commit de883d6 into SerenityOS:master Jun 2, 2024
14 of 15 checks passed
@github-actions github-actions bot removed the 👀 pr-needs-review PR needs review from a maintainer or community member label Jun 2, 2024
@kennethmyhra kennethmyhra deleted the update-freeciv branch June 10, 2024 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants