[asterisk-dev] [svn-commits] kpfleming: branch 1.4 r93180 - in /branches/1.4: ./ apps/ build_tools/ cdr/ c...

[Russell Bryant]

SVN commits to the Digium repositories wrote:
> Author: kpfleming
> Date: Mon Dec 17 00:44:51 2007
> New Revision: 93180
> 
> URL: http://svn.digium.com/view/asterisk?view=rev&rev=93180
> Log:
> In http://lists.digium.com/pipermail/asterisk-dev/2007-December/031145.html,
> rizzo brought up some issues related to the way that the metadata required
> for menuselect and the rest of the build system is extracted from the source
> files. Since I had a few hours to kill on an airplane today, I decided to
> improve this situation... so now the system caches the extracted metadata
> and uses it to build the menuselect 'tree' as much as it can. The result
> of this is that when a single source file is changed, only the metadata for
> that file needs to be extracted again, and the rest is used from the cache
> files. I also reduced the number of forked processes required to do the
> metadata extraction; it was actually possible to do most of what we needed
> in the Makefiles themselves without using any shell scripts at all! On my
> laptop, these changes resulted in an 80% decrease in the time required
> for the 'menuselect.makeopts' automatic check to occur after editing a single
> source file.

Nice job!

> While doing this work I also cleaned up a few minor things in the Makefiles,
> adding a check for 'awk' to the configure script and changed all remaining
> places we use 'grep' or 'awk' to use the ones found by the configure script,
> and changed the 'prep_tarball' script to build the menuselect metadata so
> that tarballs of Asterisk will include it and won't require the user to
> wait while it is extracted after unpacking.

I added that same change to the prep_tarball script at one point.  However,
there is one catch.  The current menuselect-tree target lists configure as a
dependency.  I'm not sure that it still needs to be, though ...

So, either we need to make configure not a dependency, or the prep_tarball
script has to be further modified to run configure, generate and copy off the
menuselect-tree, run a distclean, and the copy the menuselect-tree back in.

-- 
Russell Bryant
Senior Software Engineer
Open Source Team Lead
Digium, Inc.