<p>Corey Farrell <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://gerrit.asterisk.org/9806">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Build System: Improve ccache matching for different menuselect options.<br><br>Changing any Menuselect option in the `Compiler Flags` section causes a<br>full rebuild of the Asterisk source tree.  Every enabled option causes<br>a #define to be added to buildopts.h, thus breaking ccache caching for<br>every source file that includes "asterisk.h".  In most cases each option<br>only applies to one or two files.  Now we only define those options for<br>the specific sources which use them, this causes much better cache<br>matching when working with multiple builds.  For example testing code<br>with an without MALLOC_DEBUG will now use just over half the ccache<br>size, only main/astmm.o will have two builds cached instead of every<br>file.<br><br>Reorder main/Makefile so _ASTCFLAGS set on specific object files are all<br>together, sorted by filename.  Stop adding -DMALLOC_DEBUG to CFLAGS of<br>bundled pjproject, this define is no longer used by any header so only<br>serves to break cache.<br><br>The only code change is a slight adjustment to how main/astmm.c is<br>initialized.  Initialization functions always exist so main/asterisk.c<br>can call them unconditionally.  Additionally rename the astmm<br>initialization functions so they are not exported.<br><br>Change-Id: Ie2085237a964f6e1e6fff55ed046e2afff83c027<br>---<br>M Makefile.rules<br>M build_tools/make_buildopts_h<br>M channels/Makefile<br>M include/asterisk/_private.h<br>M include/asterisk/astmm.h<br>M main/Makefile<br>M main/asterisk.c<br>M main/astmm.c<br>M tests/Makefile<br>M third-party/pjproject/Makefile<br>M utils/Makefile<br>11 files changed, 75 insertions(+), 52 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/06/9806/2</pre><p>To view, visit <a href="https://gerrit.asterisk.org/9806">change 9806</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/9806"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: Ie2085237a964f6e1e6fff55ed046e2afff83c027 </div>
<div style="display:none"> Gerrit-Change-Number: 9806 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>