<div dir="ltr"><div dir="ltr"><div dir="ltr"><div>Hi Mojca,</div><div>Thanks for the detailed reply.</div><div><br></div><div>Changes can be seen for this port: <a href="http://frozen-falls-98471.herokuapp.com/ports/qt5-qtlocation/" target="_blank">http://frozen-falls-98471.herokuapp.com/ports/qt5-qtlocation/ </a><br></div><div dir="ltr"><br></div><div dir="ltr">On Wed, Mar 20, 2019 at 6:40 AM Mojca Miklavec <<a href="mailto:mojca@macports.org" target="_blank">mojca@macports.org</a>> wrote:<br></div><div class="gmail_quote"><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">This is super useful. But I would probably link directly to the<br>
Portfile rather than the directory. Most ports don't have any patches,<br>
but if they do, one can easily browse one level higher once on the<br>
GitHub website.<br>
I would probably make those link (in particular the homepage link)<br>
open in a new window.<br></blockquote><div> </div><div>Thanks, I have made both the changes. </div><div><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">- The entries are not unique as they should be. You seem to have two<br>
entries for the same build (26315) for example. </blockquote><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">
- Sorting for 10.13 should be in reverse order (newest builds on top)<br></blockquote><div> </div><div>Fixed both.</div><div> </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">
- I'm more interested in duration than end time. (Not sure if it's<br>
more useful to have start or stop time, but one is sufficient. The<br>
other one would be duration of the build.)<br></blockquote><div> </div><div>I have removed 'Stop Time' and added 'Time Elapsed'</div><div> </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">The missing table (no urgency) would then be more similar to this one:<br>10.13 || 10.14<br>
OK [link to 52248] || OK [link to 26315]<br></blockquote><div> </div><div>Implemented this.</div><div><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">- In BuildHistory the port_name should hold a foreign key to the port<br>
id rather than just holding a string with port's name (I guess that's<br>
many-to-one relationship in Django?).<br></blockquote><div> </div><div>Yes, but right now I can achieve this only when I have all the ports in my aws database.</div><div> </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">- a useful addition would be information about commit's shasum which<br>
triggered this change (but that might be tricky to extract in a proper<br>
way)<br></blockquote><div>Thanks, I shall give it a try.</div><div><br></div><div>I have also changed the script (parse build history) to detect new builds, by comparing the last build id in the database with that on the buildbot. It then receives only the new builds from the buildbot. I am not sure how efficient this method be, or even if this is the right way of doing it. Now either we can run this script at some definite interval or modify buildbot to instruct when the script would run.</div><div><br></div><div>Thank You</div></div></div></div></div>