<div dir="ltr">The builds from the previous commit were fine, and completed successfully. If the builders are smart enough to detect that nothing was changed in the Portfile and are able to skip trying to recompile the package, then I suppose it would be fine to simply leave the build requests in the queue.<div><br clear="all"><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div>-- </div><div>Jason Liu</div></div></div></div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Dec 6, 2021 at 2:40 PM Chris Jones <<a href="mailto:jonesc@hep.phy.cam.ac.uk">jonesc@hep.phy.cam.ac.uk</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div dir="auto"><div dir="ltr"></div><div dir="ltr"><br></div><div dir="ltr">Normally making a commit that does not change the revision or version will not trigger a rebuild, assuming the previous port revision built OK on a given platform. If this is the case then again the builds do not need to be cancelled. If the previous builds in fact did not build, then yes another rebuild will be attempted, so if nothing has changed to fix the previous build failure it will fail again. If this is the case then its best not avoid making such commits until you are able to do so at the same times as fixing the build failures. If this cannot be done the the port should use the known_fail pattern for the OSes known to fail, to avoid needless failing rebuilds.</div><div dir="ltr"><br></div><div dir="ltr">Chris</div><div dir="ltr"><br><blockquote type="cite">On 6 Dec 2021, at 7:35 pm, Jason Liu <<a href="mailto:jasonliu@umich.edu" target="_blank">jasonliu@umich.edu</a>> wrote:<br><br></blockquote></div><blockquote type="cite"><div dir="ltr"><div dir="ltr">Neither the revision nor the version numbers were changed in the Portfile (in fact, no changes were made to the Portfile at all), only the new file got added to the files/ folder. But builds got queued up on the buildbot anyway after the PR got merged.<div><br></div><div>Adding the file to the port (PR <a href="https://github.com/macports/macports-ports/pull/12976" target="_blank">#12976</a>) was only in preparation for the PR that actually includes bugfixes (PR <a href="https://github.com/macports/macports-ports/pull/13266" target="_blank">#13266</a>), which obviously will change what is installed, and do indeed need builds to be rerun. I'm trying to get the queued builds for PR #12976 removed from the builders.<br><div><br clear="all"><div><div dir="ltr"><div dir="ltr"><div>-- </div><div>Jason Liu</div></div></div></div><br></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Dec 6, 2021 at 2:02 PM Chris Jones <<a href="mailto:jonesc@hep.phy.cam.ac.uk" target="_blank">jonesc@hep.phy.cam.ac.uk</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div dir="auto"><div dir="ltr"></div><div dir="ltr">Hi,</div><div dir="ltr"><br></div><div dir="ltr">If the revision and version numbers have changed, because what is installed on disc has changed, then the builds need to be rerun, otherwise you are forcing the long builds on users, which is worse than tying up the buildbots for a while.</div><div dir="ltr"><br></div><div dir="ltr">Cheers Chris</div><div dir="ltr"><br><blockquote type="cite">On 6 Dec 2021, at 6:10 pm, Jason Liu <<a href="mailto:jasonliu@umich.edu" target="_blank">jasonliu@umich.edu</a>> wrote:<br><br></blockquote></div><blockquote type="cite"><div dir="ltr"><div dir="ltr">Hi all,<div><br></div><div>Would it be possible to remove the queued builds for my PR that was recently merged on the <a href="https://build.macports.org/builders/ports-10.13_x86_64-watcher" target="_blank">10.13</a> and <a href="https://build.macports.org/builders/ports-12_x86_64-watcher" target="_blank">12_x86_64</a> watchers? That PR only added an auxiliary file, and really shouldn't be re-built. Since there's such a long queue on the 10.13 and 12 builders, I'm thinking that cancelling a fairly length build that isn't needed would help clear up the backlog a bit faster. If there's an easy way to cancel the build on all of the builders (not just 10.13 and 12), that would probably be even better. On the other hand, if cancelling the build would be a hassle, then don't worry about it.</div><div><br></div><div>The commit is identified by the hash</div><div><br></div><div>14e7b489f630...<br>(Jason Liu <<a href="mailto:jasonliu--@users.noreply.github.com" target="_blank">jasonliu--@users.noreply.github.com</a>>)</div><div><br clear="all"><div><div dir="ltr"><div dir="ltr"><div>-- </div><div>Jason Liu</div></div></div></div></div></div>
</div></blockquote></div></blockquote></div>
</div></blockquote></div></blockquote></div>