<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Tue, 25 Aug 2015 at 09:10 R. David Murray <<a href="mailto:rdmurray@bitdance.com">rdmurray@bitdance.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, 25 Aug 2015 15:59:23 -0000, Brett Cannon <<a href="mailto:brett@python.org" target="_blank">brett@python.org</a>> wrote:<br>
> On Mon, 24 Aug 2015 at 23:19 Nick Coghlan <<a href="mailto:ncoghlan@gmail.com" target="_blank">ncoghlan@gmail.com</a>> wrote:<br>
><br>
> > On 25 August 2015 at 05:52, Gregory P. Smith <<a href="mailto:greg@krypto.org" target="_blank">greg@krypto.org</a>> wrote:<br>
> > > What we tested and decided to use on our own builds after benchmarking at<br>
> > > work was to build with:<br>
> > ><br>
> > > make profile-opt PROFILE_TASK="-m test.regrtest -w -uall,-audio -x<br>
> > test_gdb<br>
> > > test_multiprocessing"<br>
> > ><br>
> > > In general if a test is unreliable or takes an extremely long time,<br>
> > exclude<br>
> > > it for your sanity.  (i'd also kick out test_subprocess on 2.7; we<br>
> > replaced<br>
> > > subprocess with subprocess32 in our build so that wasn't an issue)<br>
> ><br>
> > Having the "production ready" make target be "make profile-opt"<br>
> > doesn't strike me as the most intuitive thing in the world.<br>
> ><br>
> > I agree we want the "./configure && make" sequence to be oriented<br>
> > towards local development builds rather than highly optimised<br>
> > production ones, so perhaps we could provide a "make production"<br>
> > target that enables PGO with an appropriate training set from<br>
> > regrtest, and also complains if "--with-pydebug" is configured?<br>
> ><br>
><br>
> That's an interesting idea for a make target. It might help get the<br>
> visibility of PGO builds higher as well.<br>
<br>
If we did want to make PGO the default, having a 'make develop' target<br>
would also be an option.  We already have a precedent for that in the<br>
'setup.py develop' command.<br></blockquote><div><br></div><div>With a `make develop` target we also can make sure not only that --with-pydebug is used but that the installation target is /tmp so that new contributors don't accidentally install a debug build. </div></div></div>