Including full build URL in the email

This commit is contained in:
Sergio Durigan Junior 2015-01-23 14:34:06 -05:00
parent 97dc582e04
commit 2fc71f839d

View file

@ -136,6 +136,10 @@ send to the gdb-testers mailing list."""
text += "Buildslave:\n" text += "Buildslave:\n"
text += "\t%s\n" % build.getSlavename () text += "\t%s\n" % build.getSlavename ()
# Including the link for the full build
text += "Full Build URL:\n"
text += "\t<%s>\n" % master_status.getURLForThing (build)
# Commits that were tested. Usually we should be dealing with # Commits that were tested. Usually we should be dealing with
# only one commit # only one commit
text += "Commit(s) tested:\n" text += "Commit(s) tested:\n"
@ -186,8 +190,8 @@ send to the gdb-testers mailing list."""
# We do not want to break here, because if this step # We do not want to break here, because if this step
# fails the test will continue. # fails the test will continue.
text += "*** Failed to make TAGS ***\n" text += "*** Failed to make TAGS ***\n"
text += "Log URL: %s/steps/%s/logs/%s\n\n" % (master_status.getURLForThing (build), text += "Log URL: <%s/steps/%s/logs/%s>\n\n" % (master_status.getURLForThing (build),
quote (n), quote (log.getName ())) quote (n), quote (log.getName ()))
continue continue
elif n == 'regressions' and log.getName () == 'regressions': elif n == 'regressions' and log.getName () == 'regressions':
text += "*** Regressions found ***\n" text += "*** Regressions found ***\n"