<div dir="ltr"><div dir="ltr">On Wed, Apr 15, 2020 at 2:14 PM Alexander Traud <<a href="mailto:pabstraud@compuserve.com">pabstraud@compuserve.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">When a contributor changes the file<br>
   <a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a>, or<br>
   menuselect/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a>, or<br>
   third-party/pjproject/configure.m4, or<br>
   third-party/jansson/configure.m4,<br>
the Asterisk Team forced the contributor to execute "./bootstrap.sh" via code review.<br></blockquote><div><br></div><div>To elaborate on why this is done - it's because in the beginning it made sense to include this as part of the review and process of contributing a configure change. With the configure script rarely receiving changes it hasn't been a problem. For cases where it has been then people have stepped in and helped out people with the changes.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
This created hotspots in the file "./configure" because different variants of the creation scripts exist on various platforms. For years, this was just [1] and [2].<br>
<br>
Since about two years, PKG_CHECK_MODULES has a different white space and a different log message. The whole drama can be seen in commit <<a href="https://github.com/asterisk/asterisk/commit/b7fbb9c41f3c1a071c594c68ff0dcc763291d451" rel="noreferrer" target="_blank">https://github.com/asterisk/asterisk/commit/b7fbb9c41f3c1a071c594c68ff0dcc763291d451</a>> normally a change of just two lines.<br>
<br>
Those hotspots spread, since the year 2018, "menuselect/configure" is touched as well, although its ".ac" file did not change: <<a href="https://github.com/asterisk/asterisk/commits/master/menuselect/configure" rel="noreferrer" target="_blank">https://github.com/asterisk/asterisk/commits/master/menuselect/configure</a>><br>
<br>
I tested how much time it costs to avoid those hotspots and created a clean change: 45 minutes. If there are merge-conflicts, add again 45 minutes for each conflict. So, that is not feasible.<br>
<br>
[1] 2014 Jul, first commit 4c21f69: #define LARGE_OFF_T (countless)<br>
[2] 2016 Jun, first commit 2cad58c: runstatedir (15 times)<br></blockquote><div> </div><div>I'm not sure exactly what you are proposing or what the core of the issue is. I understand that on different platforms and systems the resulting configure can differ, and even amongst regular contributors this can be the case. Is your issue that this makes cherry picking from the Gerrit interface difficult since you have to do it manually?</div></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div style="font-family:tahoma,sans-serif"><font color="#073763">Joshua C. Colp</font></div><div style="font-family:tahoma,sans-serif"><font color="#073763">Asterisk Technical Lead</font></div><div style="font-family:tahoma,sans-serif"><font color="#073763">Sangoma Technologies</font></div><div style="font-family:tahoma,sans-serif"><font color="#073763">Check us out at <a href="http://www.sangoma.com" target="_blank">www.sangoma.com</a> and <a href="http://www.asterisk.org" target="_blank">www.asterisk.org</a></font><br></div></div></div></div></div></div></div></div></div></div></div>