[asterisk-dev] [Code Review] appdocsxml install structure for asterisk-addons

[Russell Bryant]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.digium.com/r/26/#review79
-----------------------------------------------------------



/trunk/Makefile
<http://reviewboard.digium.com/r/26/#comment138>

    Hm.  I don't think we use Makefile targets like this anywhere else to conditionally install files.
    
    Just rename the target to "install-documentation" or something, and list the new target as ".PHONY" at the bottom of the Makefile.


- Russell


On 2008-11-05 16:04:54, Eliel Sardañons wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.digium.com/r/26/
> -----------------------------------------------------------
> 
> (Updated 2008-11-05 16:04:54)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> Structure of the make process for appdocsxml in asterisk-addons.
> 
> 
> Diffs
> -----
> 
>   /trunk/Makefile 684 
>   /trunk/apps/app_saycountpl.c 684 
>   /trunk/build_tools/get_documentation PRE-CREATION 
>   /trunk/configure 684 
>   /trunk/configure.ac 684 
>   /trunk/makeopts.in 684 
> 
> Diff: http://reviewboard.digium.com/r/26/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Eliel
> 
>