[asterisk-dev] [Code Review] 3202: install_subst: helper script for installing with path substitution

Tzafrir Cohen reviewboard at asterisk.org
Sun Feb 9 13:24:04 CST 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3202/
-----------------------------------------------------------

Review request for Asterisk Developers.


Repository: Asterisk


Description
-------

The Makefile repeats several time installing some scripts with paths (as denoted in __<special name>__) substituted with their actual values.

Substituting this, along with handling the errors correctly, is something that should not be repeated again and again in the makefile. This commit moves it to a single script:

./build_tools/install_subst [-d] <src> <dst>

It installs <src> to <dst> but first substitutes some variables in it. -d (data) installs it as a data file (-m644). Otherwise it is an executable (install's default permissions).

I want this in order to avoid having yet another copy of the above for the systemd unit file (review 3062).


Diffs
-----

  /trunk/build_tools/install_subst PRE-CREATION 
  /trunk/Makefile 407855 

Diff: https://reviewboard.asterisk.org/r/3202/diff/


Testing
-------


Thanks,

Tzafrir Cohen

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140209/54c626a4/attachment.html>


More information about the asterisk-dev mailing list