From 2fc71f839d9b8afd40aa7175c65bf85a10ac0193 Mon Sep 17 00:00:00 2001 From: Sergio Durigan Junior Date: Fri, 23 Jan 2015 14:34:06 -0500 Subject: [PATCH] Including full build URL in the email --- master.cfg | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/master.cfg b/master.cfg index d8db036..d6d645d 100644 --- a/master.cfg +++ b/master.cfg @@ -136,6 +136,10 @@ send to the gdb-testers mailing list.""" text += "Buildslave:\n" 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 # only one commit 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 # fails the test will continue. text += "*** Failed to make TAGS ***\n" - text += "Log URL: %s/steps/%s/logs/%s\n\n" % (master_status.getURLForThing (build), - quote (n), quote (log.getName ())) + text += "Log URL: <%s/steps/%s/logs/%s>\n\n" % (master_status.getURLForThing (build), + quote (n), quote (log.getName ())) continue elif n == 'regressions' and log.getName () == 'regressions': text += "*** Regressions found ***\n"