123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164 |
- From 0485ff3e7f62a331aa4f1e8ec4c2fdcee400e324 Mon Sep 17 00:00:00 2001
- From: Mike Frysinger <vapier@gentoo.org>
- Date: Sun, 16 Nov 2008 09:10:31 -0500
- Subject: [PATCH] build system fixes
- This touches up the homebrewed build system to work much better "out of the
- box" for people. Specifically:
- - allow toolchain vars to be set via environment
- - CC / BUILD_CC / AR / RANLIB
- - CFLAGS / CPPFLAGS / LDFLAGS
- - split CPPFLAGS out of CFLAGS
- - break -fPIC out of global CFLAGS and only use where needed
- - use LDLIBS for libraries, not LDFLAGS
- Signed-off-by: Mike Frysinger <vapier@gentoo.org>
- ---
- Make.Rules | 26 ++++++++++++++------------
- libcap/Makefile | 11 ++++++-----
- pam_cap/Makefile | 8 +++++---
- progs/Makefile | 2 +-
- 4 files changed, 26 insertions(+), 21 deletions(-)
- diff --git a/Make.Rules b/Make.Rules
- index 18b7cf7..5364c46 100644
- --- a/Make.Rules
- +++ b/Make.Rules
- @@ -45,27 +45,29 @@ MINOR=24
-
- # Compilation specifics
-
- -KERNEL_HEADERS := $(topdir)/libcap/include/uapi
- -IPATH += -fPIC -I$(KERNEL_HEADERS) -I$(topdir)/libcap/include
- -
- -CC := gcc
- -CFLAGS := -O2 -D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64
- -BUILD_CC := $(CC)
- -BUILD_CFLAGS := $(CFLAGS) $(IPATH)
- -AR := ar
- -RANLIB := ranlib
- +CC ?= gcc
- +CFLAGS ?= -O2
- +BUILD_CC ?= $(CC)
- +BUILD_CFLAGS ?= $(CFLAGS)
- +AR ?= ar
- +RANLIB ?= ranlib
- DEBUG = -g #-DDEBUG
- WARNINGS=-Wall -Wwrite-strings \
- -Wpointer-arith -Wcast-qual -Wcast-align \
- -Wstrict-prototypes -Wmissing-prototypes \
- -Wnested-externs -Winline -Wshadow
- LD=$(CC) -Wl,-x -shared
- -LDFLAGS := #-g
- +LDFLAGS ?= #-g
-
- -SYSTEM_HEADERS = /usr/include
- +KERNEL_HEADERS = $(topdir)/libcap/include/uapi
- +LIBCAP_CPPFLAGS = -I$(KERNEL_HEADERS) -I$(topdir)/libcap/include
- +LIBCAP_CPPFLAGS += -D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64
- +CPPFLAGS += $(LIBCAP_CPPFLAGS)
- +BUILD_CPPFLAGS += $(LIBCAP_CPPFLAGS)
- INCS=$(topdir)/libcap/include/sys/capability.h
- LDFLAGS += -L$(topdir)/libcap
- -CFLAGS += -Dlinux $(WARNINGS) $(DEBUG)
- +CPPFLAGS += -Dlinux
- +CFLAGS += $(WARNINGS) $(DEBUG)
- PAM_CAP := $(shell if [ -f /usr/include/security/pam_modules.h ]; then echo yes ; else echo no ; fi)
- INDENT := $(shell if [ -n "$$(which indent 2>/dev/null)" ]; then echo "| indent -kr" ; fi)
- DYNAMIC := $(shell if [ ! -d "$(topdir)/.git" ]; then echo yes; fi)
- diff --git a/libcap/Makefile b/libcap/Makefile
- index 0ccd2e7..6b446ea 100644
- --- a/libcap/Makefile
- +++ b/libcap/Makefile
- @@ -16,7 +16,7 @@ FILES=cap_alloc cap_proc cap_extint cap_flag cap_text
- # no support).
- ifeq ($(LIBATTR),yes)
- FILES += cap_file
- -LDFLAGS += -lattr
- +LDLIBS += -lattr
- DEPS = -lattr
- endif
-
- @@ -25,6 +25,7 @@ OBJS=$(addsuffix .o, $(FILES))
- MAJLIBNAME=$(LIBNAME).$(VERSION)
- MINLIBNAME=$(MAJLIBNAME).$(MINOR)
- GPERF_OUTPUT = _caps_output.gperf
- +CFLAGS += -fPIC
-
- all: $(MINLIBNAME) $(STALIBNAME) libcap.pc
-
- @@ -43,7 +44,7 @@ libcap.pc: libcap.pc.in
- $< >$@
-
- _makenames: _makenames.c cap_names.list.h
- - $(BUILD_CC) $(BUILD_CFLAGS) $< -o $@
- + $(BUILD_CC) $(BUILD_CFLAGS) $(BUILD_CPPFLAGS) $< -o $@
-
- cap_names.h: _makenames
- ./_makenames > cap_names.h
- @@ -60,15 +61,15 @@ $(STALIBNAME): $(OBJS)
- $(RANLIB) $@
-
- $(MINLIBNAME): $(OBJS)
- - $(LD) $(CFLAGS) $(LDFLAGS) -Wl,-soname,$(MAJLIBNAME) -o $@ $^
- + $(LD) $(CFLAGS) $(LDFLAGS) -Wl,-soname,$(MAJLIBNAME) -o $@ $^ $(LDLIBS)
- ln -sf $(MINLIBNAME) $(MAJLIBNAME)
- ln -sf $(MAJLIBNAME) $(LIBNAME)
-
- %.o: %.c $(INCLS)
- - $(CC) $(CFLAGS) $(IPATH) -c $< -o $@
- + $(CC) $(CFLAGS) $(CPPFLAGS) -c $< -o $@
-
- cap_text.o: cap_text.c $(USE_GPERF_OUTPUT) $(INCLS)
- - $(CC) $(CFLAGS) $(IPATH) $(INCLUDE_GPERF_OUTPUT) -c $< -o $@
- + $(CC) $(CFLAGS) $(CPPFLAGS) $(INCLUDE_GPERF_OUTPUT) -c $< -o $@
-
- install: all
- mkdir -p -m 0755 $(FAKEROOT)$(INCDIR)/sys
- diff --git a/pam_cap/Makefile b/pam_cap/Makefile
- index cc32fb6..6f07b6b 100644
- --- a/pam_cap/Makefile
- +++ b/pam_cap/Makefile
- @@ -9,6 +9,8 @@ include ../Make.Rules
- # written (and you know why it fails), email me and explain why. Thanks!
- LDLIBS += -L../libcap -lcap
-
- +CFLAGS += -fPIC
- +
- all: pam_cap.so
- $(MAKE) testcompile
-
- @@ -17,13 +19,13 @@ install: all
- install -m 0755 pam_cap.so $(FAKEROOT)$(LIBDIR)/security
-
- pam_cap.so: pam_cap.o
- - $(LD) $(LDFLAGS) -o pam_cap.so $< $(LDLIBS)
- + $(LD) $(CFLAGS) $(LDFLAGS) -o pam_cap.so $< $(LDLIBS)
-
- pam_cap.o: pam_cap.c
- - $(CC) $(CFLAGS) $(IPATH) -c $< -o $@
- + $(CC) $(CFLAGS) $(CPPFLAGS) -c $< -o $@
-
- testcompile: test.c pam_cap.o
- - $(CC) $(CFLAGS) $(LDFLAGS) -o $@ $+ -lpam -ldl $(LDLIBS)
- + $(CC) $(CFLAGS) $(CPPFLAGS) $(LDFLAGS) -o $@ $+ -lpam -ldl $(LDLIBS)
-
- clean:
- rm -f *.o *.so testcompile *~
- diff --git a/progs/Makefile b/progs/Makefile
- index 778149e..8f4e2e6 100644
- --- a/progs/Makefile
- +++ b/progs/Makefile
- @@ -22,7 +22,7 @@ $(BUILD): %: %.o
- $(CC) $(CFLAGS) $(LDFLAGS) -o $@ $< $(LDLIBS)
-
- %.o: %.c $(INCS)
- - $(CC) $(IPATH) $(CFLAGS) -c $< -o $@
- + $(CC) $(CFLAGS) $(CPPFLAGS) -c $< -o $@
-
- install: all
- mkdir -p -m 0755 $(FAKEROOT)$(SBINDIR)
- --
- 2.4.1
|