----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3202/#review10837 -----------------------------------------------------------
/trunk/build_tools/install_subst <https://reviewboard.asterisk.org/r/3202/#comment20429> If we have '-d' we need to shift. /trunk/build_tools/install_subst <https://reviewboard.asterisk.org/r/3202/#comment20425> This is missing __ASTERISK_ETC_DIR__ - Corey Farrell On Feb. 9, 2014, 2:24 p.m., Tzafrir Cohen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3202/ > ----------------------------------------------------------- > > (Updated Feb. 9, 2014, 2:24 p.m.) > > > 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 > >
-- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev