lachrimae/lix
0
0
Fork 0
forked from lix-project/lix
Code Pull requests Activity

Fix library ordering in Makefiles

Browse source 
The existing ordering linked `libutil` before `libstore`, which causes
link failures when building statically. This is due to `libstore` using
functions from `libutil`, and the fact that symbol resolution works
"forward" - i.e. if you pass `-lfoo -lbar -lbaz`, any symbols that
`libbar` uses from `libbaz` will be resolved, but symbols from `libfoo`
will not since it comes first in the command line.

All this to say: this commit reorders the libraries which fixes the link
errors.
This commit is contained in:
Andrew Dunham 2018-04-21 21:10:52 -07:00
parent 040acdcee2
commit 3a918014b2
3 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/build-remote/local.mk
View file

@ -4,6 +4,6 @@ build-remote_DIR := $(d)
build-remote_INSTALL_DIR := $(libexecdir)/nix
build-remote_LIBS = libmain libutil libformat libstore
build-remote_LIBS = libmain libformat libstore libutil
build-remote_SOURCES := $(d)/build-remote.cc

2
src/nix-channel/local.mk
View file

@ -2,6 +2,6 @@ programs += nix-channel
nix-channel_DIR := $(d)
nix-channel_LIBS = libmain libutil libformat libstore
nix-channel_LIBS = libmain libformat libstore libutil
nix-channel_SOURCES := $(d)/nix-channel.cc

2
src/nix-copy-closure/local.mk
View file

@ -2,6 +2,6 @@ programs += nix-copy-closure
nix-copy-closure_DIR := $(d)
nix-copy-closure_LIBS = libmain libutil libformat libstore
nix-copy-closure_LIBS = libmain libformat libstore libutil
nix-copy-closure_SOURCES := $(d)/nix-copy-closure.cc