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

Update makefile.linux #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update makefile.linux #12

wants to merge 1 commit into from

Conversation

lilith
Copy link

@lilith lilith commented Dec 29, 2013

GCC broke compatibility by changing the default linking behavior to --as-needed. We have to switch it back or math breaks.

cc -g3 -Wall -Wconversion -ansi -pedantic   -c -o pptest.o pptest.c
pptest.c: In function 'LUAF_boxboolean':
pptest.c:41:22: warning: conversion to 'char' from 'int' may alter its value [-Wconversion]
cc -rdynamic -o pptest pptest.o -lm -ldl -llua
/usr/local/lib/liblua.a(lvm.o): In function `Arith':
lvm.c:(.text+0x485): undefined reference to `pow'
lvm.c:(.text+0x4b4): undefined reference to `floor'
/usr/local/lib/liblua.a(lvm.o): In function `luaV_execute':
lvm.c:(.text+0x22f1): undefined reference to `floor'
lvm.c:(.text+0x240e): undefined reference to `pow'

like so.

GCC broke compatibility by changing the default linking behavior to --as-needed. We have to switch it back or math breaks.
@hoelzro
Copy link
Owner

hoelzro commented Dec 29, 2013

Hi Nathanael,

I am no longer maintainer of the Pluto project. All fixes should be directed to Stefan Reich.

@lilith
Copy link
Author

lilith commented Jan 5, 2014

I wasn't able to find his fork of the repo. Could you share the URL?

@hoelzro
Copy link
Owner

hoelzro commented Jan 5, 2014

I don't know if he even has a GitHub account, and his site is down. You can try searching the Lua mailing list to find his e-mail address...

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

Successfully merging this pull request may close these issues.

2 participants