Closed
Bug 842641
Opened 11 years ago
Closed 11 years ago
dxr.mozilla.org no longer shows hgblame links
Categories
(Webtools Graveyard :: DXR, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jcranmer, Assigned: jcranmer)
References
Details
Attachments
(1 file)
This is a known regression, caused by hglib not working properly on the buildbots.
Assignee | ||
Comment 1•11 years ago
|
||
Well, I can't f? without a patch, so I'll needinfo? instead. I've started work on a plugin called "omniglot" which is designed to replace the current hg plugin: https://github.com/jcranmer/dxr/commit/82728124691f8358e388ba6c2037fa581ce2a482 That commit should support links for hg.mozilla.org and github (my small repositories all seem to be github ones, go figure), as well as supporting cases where there are multiple repositories in a tree or where the tree itself is a subdir of the repository (e.g., js/src as a tree).
Flags: needinfo?(erik)
Comment 3•11 years ago
|
||
Review comments: * Typical style stuff: PEP 8 naming & import ordering * Consistency: sometimes you say VCS, sometimes Vcs * See also PEP 257 for docstrings. The design looks reasonable. We can refactor and nitpick once we get it working. You're going to make a lot of people very happy!
Flags: needinfo?(erik)
Assignee | ||
Comment 5•11 years ago
|
||
This changeset is a partial fix for bug 842641.
Attachment #789184 -
Flags: review?(erik)
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → Pidgeot18
Comment 6•11 years ago
|
||
Commit pushed to master at https://github.com/mozilla/dxr https://github.com/mozilla/dxr/commit/ca2150da02f5e2b23ee02d8493293aae53d170cf Add the omniglot plugin, which can provide links to blame for both hg and git. This changeset is a partial fix for bug 842641. --HG-- extra : rebase_source : d99e59def249fe23ff99bab49c783e6b791e3752
Assignee | ||
Comment 7•11 years ago
|
||
Looks like dxr.mozilla.org needed no config change to get the links appearing this morning, so I declare this fixed!
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•11 years ago
|
Attachment #789184 -
Flags: review?(erik)
Comment 8•11 years ago
|
||
Scope creep, but it would be extra nice if the panel with the blame links floated in the top right of the window so that one doesn't need to scroll back to the top of the page for them...
Comment 9•11 years ago
|
||
The blame links work with middle-click and right-click->"Open in new tab", but left-click doesn't cause navigation (some onclick handler attached here?).
Comment 10•11 years ago
|
||
Fixing the blame link in bug 920131.
Updated•3 years ago
|
Product: Webtools → Webtools Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•