Author: jelmer
Date: 2005-12-13 21:42:03 +0000 (Tue, 13 Dec 2005)
New Revision: 12219

WebSVN: 
http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=12219

Log:
Use some more common functions

Modified:
   branches/SAMBA_4_0/source/build/smb_build/makefile.pm
   branches/SAMBA_4_0/source/build/smb_build/output.pm
   branches/SAMBA_4_0/source/main.mk
   branches/SAMBA_4_0/source/torture/config.mk


Changeset:
Modified: branches/SAMBA_4_0/source/build/smb_build/makefile.pm
===================================================================
--- branches/SAMBA_4_0/source/build/smb_build/makefile.pm       2005-12-13 
21:00:52 UTC (rev 12218)
+++ branches/SAMBA_4_0/source/build/smb_build/makefile.pm       2005-12-13 
21:42:03 UTC (rev 12219)
@@ -99,14 +99,14 @@
 
 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 @@
        $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 @@
        $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
        [EMAIL PROTECTED] Linking \$\@
        [EMAIL PROTECTED](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 @@
 
        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("[EMAIL PROTECTED] \"Pre-Linking $ctx->{TYPE} 
$ctx->{NAME}\"\n");
-       $self->output("[EMAIL PROTECTED](LD) -r 
\$($ctx->{TYPE}_$ctx->{NAME}_OBJS) -o $ctx->{TARGET}\n");
+       $self->output("[EMAIL PROTECTED](LD) -r 
\$($ctx->{TYPE}_$ctx->{NAME}_OBJ_LIST) -o $ctx->{TARGET}\n");
        $self->output("\n");
 }
 
@@ -300,11 +290,11 @@
 
        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("[EMAIL PROTECTED] $ctx->{TARGET}\n");
 }
 
@@ -314,7 +304,7 @@
 
        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 @@
 
        $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
        [EMAIL PROTECTED] Linking \$@
        [EMAIL PROTECTED](STLD) \$(STLD_FLAGS) \$@ \\
-               \$(LIBRARY_$ctx->{NAME}_LINK_LIST)
+               \$($ctx->{TYPE}_$ctx->{NAME}_LINK_LIST)
 
 library_$ctx->{NAME}: basics $ctx->{TARGET}
 
@@ -354,7 +344,7 @@
                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 @@
        $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)
        [EMAIL PROTECTED] Linking \$\@
        [EMAIL PROTECTED](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 @@
 {
        my ($self,$ctx) = @_;
 
-       $self->_prepare_obj_list($ctx->{TYPE}, $ctx);
+       $self->_prepare_list($ctx, "OBJ_LIST");
 }
 
 sub write($$)
@@ -432,7 +422,7 @@
 
        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)) \\

Modified: branches/SAMBA_4_0/source/build/smb_build/output.pm
===================================================================
--- branches/SAMBA_4_0/source/build/smb_build/output.pm 2005-12-13 21:00:52 UTC 
(rev 12218)
+++ branches/SAMBA_4_0/source/build/smb_build/output.pm 2005-12-13 21:42:03 UTC 
(rev 12219)
@@ -36,7 +36,7 @@
        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 @@
        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 @@
        $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 @@
        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}";

Modified: branches/SAMBA_4_0/source/main.mk
===================================================================
--- branches/SAMBA_4_0/source/main.mk   2005-12-13 21:00:52 UTC (rev 12218)
+++ branches/SAMBA_4_0/source/main.mk   2005-12-13 21:42:03 UTC (rev 12219)
@@ -166,11 +166,11 @@
        @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

Modified: branches/SAMBA_4_0/source/torture/config.mk
===================================================================
--- branches/SAMBA_4_0/source/torture/config.mk 2005-12-13 21:00:52 UTC (rev 
12218)
+++ branches/SAMBA_4_0/source/torture/config.mk 2005-12-13 21:42:03 UTC (rev 
12219)
@@ -281,6 +281,6 @@
                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

Reply via email to