HostedRedmine.com has moved to the Planio platform. All logins and passwords remained the same. All users will be able to login and use Redmine just as before. Read more...
Feature #854615
Comment purpose of fc_config.h in binary distributions
0%
Description
We ship fc_config.h in the Windows releases. (gna patch #6544)
It could do with a comment explaining what it is for, in that context, in case someone thinks it is a configuration file that they can usefully edit.
(AH_TOP macro came in in 2000 , in Autoconf 2.50 or earlier. Our minimum version is 2.61.)
History
#1
Updated by Jacob Nevins over 1 year ago
- File m-fc-config-comment.patch m-fc-config-comment.patch added
- File 30-26-fc-config-comment.patch 30-26-fc-config-comment.patch added
- Status changed from In Progress to Resolved
(Also added to Meson-generated file for consistency, even though Meson build doesn't obviously install fc_config.h yet.)
#2
Updated by Marko Lindqvist over 1 year ago
Jacob Nevins wrote:
(Also added to Meson-generated file for consistency, even though Meson build doesn't obviously install fc_config.h yet.)
Should it also be added to freeciv_config.h of meson builds? Autotools version installs only fc_config.h as it's strictly a superset of freeciv_config.h (there's nothing in freeciv_config.h that's not in fc_config.h). Meson version of fc_config.h does not contain freeciv_config.h stuff, but #includes it. So presumably we are going to install both fc_config.h and freeciv_config.h of meson based builds in the future.
#3
Updated by Jacob Nevins over 1 year ago
Good point, done.
#4
Updated by Jacob Nevins over 1 year ago
- Status changed from Resolved to Closed
.