Last Comment Bug 764207 - Tryserver pushes should output TBPL link for the push
: Tryserver pushes should output TBPL link for the push
Status: RESOLVED FIXED
:
Product: Developer Services
Classification: Other
Component: Mercurial: hg.mozilla.org (show other bugs)
: other
: All All
: -- enhancement
: ---
Assigned To: Graeme McCutcheon [:graememcc]
:
:
Mentors:
Depends on: 751867
Blocks:
  Show dependency treegraph
 
Reported: 2012-06-12 16:15 PDT by Justin Dolske [:Dolske]
Modified: 2014-10-02 07:01 PDT (History)
6 users (show)
See Also:
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
v1 (989 bytes, patch)
2012-06-13 07:41 PDT, Graeme McCutcheon [:graememcc]
ted: review+
Details | Diff | Splinter Review

Description Justin Dolske [:Dolske] 2012-06-12 16:15:48 PDT
+++ This bug was initially created as a clone of Bug #663585 +++

"It'd be handy if, when pushing, the hg changeset URL were printed, since often you want to paste that URL into a bug."

It would also be REALLY handy if a Tryserver push spat out a TBPL link for the push, since on occasion people will look at that URL to see results. :)

Presumably all the info should be at-hand...

$ hg log -l 1
changeset:   96464:1b3e7e9f46a5

TBPL link: https://tbpl.mozilla.org/?tree=Try&rev=1b3e7e9f46a5


I suppose there's no reason to limit it exclusively to Try, but that would be the most useful to me.
Comment 1 Dirkjan Ochtman (:djc) 2012-06-13 00:28:20 PDT
Instead of having a push print out the changeset URL, have it post the changeset URL to the bug mentioned in the commit message?
Comment 2 Cameron McCormack (:heycam) 2012-06-13 00:29:47 PDT
There are plenty of times I push to try without it being something I want reported in the bug, so I'd rather it be just printed out after pushing.
Comment 3 Dirkjan Ochtman (:djc) 2012-06-13 01:43:51 PDT
Sure. You could also have it distinguish between pushing to try and elsewhere.
Comment 4 Graeme McCutcheon [:graememcc] 2012-06-13 07:41:46 PDT
Created attachment 632690 [details] [diff] [review]
v1

Taking the simple approach here.

I guess I'd be slightly cautious about talking to something external in the hook. I believe the hook for pulse.mozilla.org caused a few intermittent tree closures before being disabled.
Comment 5 Ted Mielczarek [:ted.mielczarek] 2012-06-13 09:49:53 PDT
Comment on attachment 632690 [details] [diff] [review]
v1

Review of attachment 632690 [details] [diff] [review]:
-----------------------------------------------------------------

Fine with me. In the future, feel free to push changes here without review if you'd like. It's your code. :)
Comment 6 Graeme McCutcheon [:graememcc] 2012-06-13 09:54:20 PDT
http://hg.mozilla.org/hgcustom/hghooks/rev/38218ddfda34
Comment 7 Justin Dolske [:Dolske] 2012-06-17 00:44:06 PDT
<3

Note You need to log in before you can comment on or make changes to this bug.