From 57ea480c24ff914dfa6cbc5a309020b21fa4af39 Mon Sep 17 00:00:00 2001 From: Jelmer Vernooij Date: Tue, 13 Dec 2005 21:42:03 +0000 Subject: [PATCH] r12219: Use some more common functions --- source/build/smb_build/makefile.pm | 54 ++++++++++++------------------ source/build/smb_build/output.pm | 8 ++--- source/main.mk | 4 +-- source/torture/config.mk | 2 +- 4 files changed, 29 insertions(+), 39 deletions(-) diff --git a/source/build/smb_build/makefile.pm b/source/build/smb_build/makefile.pm index 5d7a31379d0..d5ac6de6f9b 100644 --- a/source/build/smb_build/makefile.pm +++ b/source/build/smb_build/makefile.pm @@ -99,14 +99,14 @@ SHELL=$self->{config}->{SHELL} PERL=$self->{config}->{PERL} +CPP=$self->{config}->{CPP} +CPPFLAGS=$self->{config}->{CPPFLAGS} + CC=$self->{config}->{CC} -CFLAGS=-I\$(srcdir)/include -I\$(srcdir) -I\$(srcdir)/lib -D_SAMBA_BUILD_ -DHAVE_CONFIG_H $self->{config}->{CFLAGS} $self->{config}->{CPPFLAGS} +CFLAGS=-I\$(srcdir)/include -I\$(srcdir) -I\$(srcdir)/lib -D_SAMBA_BUILD_ -DHAVE_CONFIG_H $self->{config}->{CFLAGS} \$(CPPFLAGS) PICFLAG=$self->{config}->{PICFLAG} HOSTCC=$self->{config}->{HOSTCC} -CPP=$self->{config}->{CPP} -CPPFLAGS=$self->{config}->{CPPFLAGS} - DEVEL_LDFLAGS=-Wl,-rpath,bin/ LD=$self->{config}->{LD} LDFLAGS=$self->{config}->{LDFLAGS} -Lbin/$devld @@ -226,23 +226,13 @@ sub _prepare_list($$$) $self->output("$ctx->{TYPE}\_$ctx->{NAME}_$var =$tmplist\n"); } -sub _prepare_obj_list($$$) -{ - my ($self,$var,$ctx) = @_; - - my $tmplist = array2oneperline($ctx->{OBJ_LIST}); - return if ($tmplist eq ""); - - $self->output("$var\_$ctx->{NAME}_OBJS =$tmplist\n"); -} - sub SharedLibrary($$) { my ($self,$ctx) = @_; push (@{$self->{shared_libs}}, "bin/$ctx->{LIBRARY_NAME}"); - $self->_prepare_obj_list($ctx->{TYPE}, $ctx); + $self->_prepare_list($ctx, "OBJ_LIST"); $self->_prepare_list($ctx, "CFLAGS"); $self->_prepare_list($ctx, "DEPEND_LIST"); $self->_prepare_list($ctx, "LINK_LIST"); @@ -251,11 +241,11 @@ sub SharedLibrary($$) $self->output(<< "__EOD__" # -$ctx->{TARGET}: \$(LIBRARY_$ctx->{NAME}_DEPEND_LIST) \$(LIBRARY_$ctx->{NAME}_OBJS) bin/.dummy +$ctx->{TARGET}: \$($ctx->{TYPE}_$ctx->{NAME}_DEPEND_LIST) \$($ctx->{TYPE}_$ctx->{NAME}_OBJ_LIST) bin/.dummy \@echo Linking \$\@ \@\$(SHLD) \$(SHLD_FLAGS) -o \$\@ \\ - \$(LIBRARY_$ctx->{NAME}_LINK_FLAGS) \\ - \$(LIBRARY_$ctx->{NAME}_LINK_LIST) + \$($ctx->{TYPE}_$ctx->{NAME}_LINK_FLAGS) \\ + \$($ctx->{TYPE}_$ctx->{NAME}_LINK_LIST) __EOD__ ); @@ -283,14 +273,14 @@ sub MergedObj($$) return unless $ctx->{TARGET}; - $self->_prepare_obj_list($ctx->{TYPE}, $ctx); + $self->_prepare_list($ctx, "OBJ_LIST"); $self->_prepare_list($ctx, "CFLAGS"); $self->_prepare_list($ctx, "DEPEND_LIST"); - $self->output("$ctx->{TARGET}: \$($ctx->{TYPE}_$ctx->{NAME}_OBJS)\n"); + $self->output("$ctx->{TARGET}: \$($ctx->{TYPE}_$ctx->{NAME}_OBJ_LIST)\n"); $self->output("\t\@echo \"Pre-Linking $ctx->{TYPE} $ctx->{NAME}\"\n"); - $self->output("\t@\$(LD) -r \$($ctx->{TYPE}_$ctx->{NAME}_OBJS) -o $ctx->{TARGET}\n"); + $self->output("\t@\$(LD) -r \$($ctx->{TYPE}_$ctx->{NAME}_OBJ_LIST) -o $ctx->{TARGET}\n"); $self->output("\n"); } @@ -300,11 +290,11 @@ sub ObjList($$) return unless $ctx->{TARGET}; - $self->_prepare_obj_list($ctx->{TYPE}, $ctx); + $self->_prepare_list($ctx, "OBJ_LIST"); $self->_prepare_list($ctx, "CFLAGS"); $self->_prepare_list($ctx, "DEPEND_LIST"); $self->output("$ctx->{TARGET}: "); - $self->output("\$($ctx->{TYPE}_$ctx->{NAME}_DEPEND_LIST) \$($ctx->{TYPE}_$ctx->{NAME}_OBJS)\n"); + $self->output("\$($ctx->{TYPE}_$ctx->{NAME}_DEPEND_LIST) \$($ctx->{TYPE}_$ctx->{NAME}_OBJ_LIST)\n"); $self->output("\t\@touch $ctx->{TARGET}\n"); } @@ -314,7 +304,7 @@ sub StaticLibrary($$) push (@{$self->{static_libs}}, $ctx->{OUTPUT}); - $self->_prepare_obj_list($ctx->{TYPE}, $ctx); + $self->_prepare_list($ctx, "OBJ_LIST"); $self->_prepare_list($ctx, "CFLAGS"); $self->_prepare_list($ctx, "DEPEND_LIST"); @@ -323,10 +313,10 @@ sub StaticLibrary($$) $self->output(<< "__EOD__" # -$ctx->{TARGET}: \$(LIBRARY_$ctx->{NAME}_DEPEND_LIST) \$(LIBRARY_$ctx->{NAME}_OBJS) bin/.dummy +$ctx->{TARGET}: \$($ctx->{TYPE}_$ctx->{NAME}_DEPEND_LIST) \$($ctx->{TYPE}_$ctx->{NAME}_OBJ_LIST) bin/.dummy \@echo Linking \$@ \@\$(STLD) \$(STLD_FLAGS) \$@ \\ - \$(LIBRARY_$ctx->{NAME}_LINK_LIST) + \$($ctx->{TYPE}_$ctx->{NAME}_LINK_LIST) library_$ctx->{NAME}: basics $ctx->{TARGET} @@ -354,7 +344,7 @@ sub Binary($$) push (@{$self->{bin_progs}}, $ctx->{TARGET}); } - $self->_prepare_obj_list($ctx->{TYPE}, $ctx); + $self->_prepare_list($ctx, "OBJ_LIST"); $self->_prepare_list($ctx, "CFLAGS"); $self->_prepare_list($ctx, "DEPEND_LIST"); $self->_prepare_list($ctx, "LINK_LIST"); @@ -363,11 +353,11 @@ sub Binary($$) $self->output(<< "__EOD__" # # -bin/$ctx->{BINARY}: bin/.dummy \$(BINARY_$ctx->{NAME}_DEPEND_LIST) \$(BINARY_$ctx->{NAME}_OBJS) +bin/$ctx->{BINARY}: bin/.dummy \$($ctx->{TYPE}_$ctx->{NAME}_DEPEND_LIST) \$($ctx->{TYPE}_$ctx->{NAME}_OBJ_LIST) \@echo Linking \$\@ \@\$(CC) \$(LDFLAGS) -o \$\@ \\ - \$\(BINARY_$ctx->{NAME}_LINK_LIST) \\ - \$\(BINARY_$ctx->{NAME}_LINK_FLAGS) + \$\($ctx->{TYPE}_$ctx->{NAME}_LINK_LIST) \\ + \$\($ctx->{TYPE}_$ctx->{NAME}_LINK_FLAGS) binary_$ctx->{BINARY}: basics bin/$ctx->{BINARY} __EOD__ @@ -412,7 +402,7 @@ sub ProtoHeader($$) { my ($self,$ctx) = @_; - $self->_prepare_obj_list($ctx->{TYPE}, $ctx); + $self->_prepare_list($ctx, "OBJ_LIST"); } sub write($$) @@ -432,7 +422,7 @@ sub write($$) if ($self->{developer}) { $self->output(<<__EOD__ -#-include \$(_ALL_OBJS_OBJS:.o=.d) +#-include \$(_ALL_OBJS_OBJ_LIST:.o=.d) IDL_FILES = \$(wildcard librpc/idl/*.idl) \$(patsubst librpc/idl/%.idl,librpc/gen_ndr/ndr_%.c,\$(IDL_FILES)) \\ \$(patsubst librpc/idl/%.idl,librpc/gen_ndr/ndr_\%_c.c,\$(IDL_FILES)) \\ diff --git a/source/build/smb_build/output.pm b/source/build/smb_build/output.pm index 92b2f05c62f..b7a9653b2b4 100644 --- a/source/build/smb_build/output.pm +++ b/source/build/smb_build/output.pm @@ -36,7 +36,7 @@ sub generate_objlist($) my $subsys = shift; $subsys->{TARGET} = "bin/.$subsys->{TYPE}_$subsys->{NAME}"; - $subsys->{OUTPUT} = "\$($subsys->{TYPE}_$subsys->{NAME}_OBJS)"; + $subsys->{OUTPUT} = "\$($subsys->{TYPE}_$subsys->{NAME}_OBJ_LIST)"; } sub generate_shared_library($) @@ -45,7 +45,7 @@ sub generate_shared_library($) my $link_name; @{$lib->{DEPEND_LIST}} = (); - @{$lib->{LINK_LIST}} = ("\$($lib->{TYPE}_$lib->{NAME}\_OBJS)"); + @{$lib->{LINK_LIST}} = ("\$($lib->{TYPE}_$lib->{NAME}\_OBJ_LIST)"); $link_name = $lib->{NAME}; $link_name =~ s/^LIB//; @@ -73,7 +73,7 @@ sub generate_static_library($) $link_name =~ s/^LIB//; $lib->{LIBRARY_NAME} = "lib".lc($link_name).".a"; - @{$lib->{LINK_LIST}} = ("\$($lib->{TYPE}_$lib->{NAME}\_OBJS)"); + @{$lib->{LINK_LIST}} = ("\$($lib->{TYPE}_$lib->{NAME}\_OBJ_LIST)"); @{$lib->{LINK_FLAGS}} = (); $lib->{TARGET} = "bin/$lib->{LIBRARY_NAME}"; @@ -85,7 +85,7 @@ sub generate_binary($) my $bin = shift; @{$bin->{DEPEND_LIST}} = (); - @{$bin->{LINK_LIST}} = ("\$($bin->{TYPE}_$bin->{NAME}\_OBJS)"); + @{$bin->{LINK_LIST}} = ("\$($bin->{TYPE}_$bin->{NAME}\_OBJ_LIST)"); @{$bin->{LINK_FLAGS}} = (); $bin->{TARGET} = $bin->{OUTPUT} = "bin/$bin->{NAME}"; diff --git a/source/main.mk b/source/main.mk index 06a762ecd95..1f8838a4952 100644 --- a/source/main.mk +++ b/source/main.mk @@ -166,11 +166,11 @@ include/config.h: @echo "You need to rerun ./autogen.sh and ./configure" @/bin/false -include/proto.h: $(PROTO_PROTO_OBJS:.o=.c) +include/proto.h: $(PROTO_PROTO_OBJ_LIST:.o=.c) @-rm -f include/includes.h.gch @$(SHELL) script/mkproto.sh "$(PERL)" \ -h _PROTO_H_ include/proto.h \ - $(PROTO_PROTO_OBJS) + $(PROTO_PROTO_OBJ_LIST) @touch include/proto.h proto: include/proto.h diff --git a/source/torture/config.mk b/source/torture/config.mk index 2c7e697b56e..41d17073b7e 100644 --- a/source/torture/config.mk +++ b/source/torture/config.mk @@ -281,6 +281,6 @@ gcov: CFLAGS="$(CFLAGS) $(GCOV_FLAGS)" \ LD_FLAGS="$(LD_FLAGS) $(GCOV_FLAGS)" \ LIBS="$(LIBS) $(GCOV_LIBS)" - for I in $(sort $(dir $(_ALL_OBJS_OBJS))); \ + for I in $(sort $(dir $(_ALL_OBJS_OBJ_LIST))); \ do $(GCOV) -p -o $$I $$I/*.c; \ done -- 2.34.1