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

Provide fixes to the build system #37

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

pbassa
Copy link

@pbassa pbassa commented Jul 19, 2016

  • Use @LIBS@ from configure in Makefile.in (this helps statically linking fcgiwrap), and
  • link with libsystemd rather than libsystemd-daemon, as needed by latest versions of systemd.

…nually.

fcgiwrap uses libfcgi, and libfcgi uses libm.  This is an issue when
using static libs, because -lm does not get properly added to the link
flags in this case.  A solution is to provide LIBS=-lm to configure
when needed.  However, fcgiwrap does not use Automake and its
Makefile.in ignores LIBS from configure.  Fix that.

Signed-off-by: Thomas Claveirole <[email protected]>
Newer versions of systemd do not provide libsystemd-daemon anymore.
Therefore, use libsystemd instead.

Signed-off-by: Thomas Claveirole <[email protected]>
@Lekensteyn
Copy link
Contributor

LGTM. FYI, systemd 209 moved away from the separate libs.

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