After ports recompilation for 6.0 (upgrade from 5-STABLE) I noticed that
quakeforge start and plays the demo, but completely ignores any keyboard input.
Besides that, it reports that plugin /usr/local/lib/quakeforge/console_client.so
cannot be loaded.
Investigation had shown that the plugin is used to drive the game menu, and the
reason it cannot be loaded is that the main executable of the game does not export
the symbol Key_Progs_Init.
Symbol is absent since build links final exe from some static libraries, and
.o with Key_Progs_Init is (for some reasons) not referenced by exe.
As a workaround, I propose to add an explicit reference for the symbol.
Proper fix would be something like -Wl,--whole-archive switch for final link,
but this does not work due to libtool only allows to put switches at the start
of the command line. And cc links libgcc.a twice, that leads to duplicate
@@ -69,6 +69,9 @@
fcntl (0, F_SETFL, fcntl (0, F_GETFL, 0) & ~O_NONBLOCK);
+extern void Key_Progs_Init (progs_t *pr);
+void (*x)() = Key_Progs_Init;
main (int c, const char *v)
Maintainer of games/quakeforge,
Please note that PR ports/89065 has just been submitted.
If it contains a patch for an upgrade, an enhancement or a bug fix
you agree on, reply to this email stating that you approve the patch
and a committer will take care of it.
The full text of the PR can be found at:
Awaiting maintainers feedback
please have a look at ports/86175 first and commit it. I transfered
maintainership of quake(1)forge to Alejandro Pulver
<email@example.com>, as he is way more active than me, with regard to
PS: I can confirm the bug, did something happen with gcc or libtool in
between the 5.4 and 6.0 timeframe?
On Thu, 17 Nov 2005 20:30:11 UT
Edwin Groothuis <edwin@FreeBSD.org> wrote:
> Maintainer of games/quakeforge,
> Please note that PR ports/89065 has just been submitted.
> If it contains a patch for an upgrade, an enhancement or a bug fix
> you agree on, reply to this email stating that you approve the patch
> and a committer will take care of it.
> The full text of the PR can be found at:
> Edwin Groothuis
I agree with the patch.
Patch commited, thanks!