[asterisk-dev] Re: [svn-commits] bweschke: branch 1.2 r58008 - /branches/1.2/channels/h323/Makefile

[Tzafrir Cohen]

On Tue, Mar 06, 2007 at 03:17:16PM -0000, svn-commits at lists.digium.com wrote:
> Author: bweschke
> Date: Tue Mar  6 09:17:16 2007
> New Revision: 58008
> 
> URL: http://svn.digium.com/view/asterisk?view=rev&rev=58008
> Log:
>  Cleanup the Makefile so that we only attempt to include a file when 
> we're building and not 'clean'ing so 'make clean' completes successfully. 
>
> chan_h323 maintainer: please check to make sure I haven't broken your 
> build target. From: jsmith in #asterisk-dev

Actually you have:

> 
> 
> Modified:
>     branches/1.2/channels/h323/Makefile
> 
> Modified: branches/1.2/channels/h323/Makefile
> URL: http://svn.digium.com/view/asterisk/branches/1.2/channels/h323/Makefile?view=diff&rev=58008&r1=58007&r2=58008
> ==============================================================================
> --- branches/1.2/channels/h323/Makefile (original)
> +++ branches/1.2/channels/h323/Makefile Tue Mar  6 09:17:16 2007
> @@ -21,7 +21,8 @@
>  OPENH323DIR=$(HOME)/openh323
>  endif
>  
> -include $(OPENH323DIR)/openh323u.mak

I figure that the diff you wanted is actually:
-include $(OPENH323DIR)/openh323u.mak
+-include $(OPENH323DIR)/openh323u.mak

so it will 

> +clean:
> +	rm -f *.so *.o .depend
>  
>  $(SOURCES)::	$(SOURCES:.cxx=.cpp)
>  	ln -f $< $@
> @@ -30,6 +31,7 @@
>  	touch $(SOURCES)
>  
>  libchanh323.a:	$(OBJS)
> +	include $(OPENH323DIR)/openh323u.mak

This will try to run the shell command "include ..." as part of the
target libchanh323.a .

The include keyword only works out of targets.

>  	ar crv $@ $(OBJS)
>  
>  Makefile.ast:	FORCE
> 

-- 
               Tzafrir Cohen       
icq#16849755                    jabber:tzafrir at jabber.org
+972-50-7952406           mailto:tzafrir.cohen at xorcom.com       
http://www.xorcom.com  iax:guest at local.xorcom.com/tzafrir