Closed
Bug 437429
Opened 16 years ago
Closed 16 years ago
change pushlog hg extension to use new sqlite backend
Categories
(Release Engineering :: General, defect, P2)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: ted, Assigned: ted)
References
Details
Attachments
(1 file)
5.67 KB,
patch
|
benjamin
:
review+
|
Details | Diff | Splinter Review |
I made the pushlog use a sqlite store in bug 437397, this bug is for making the feed extension use that backend. I'm also going to add a HTML view of the pushlog. Patch in a bit.
Assignee | ||
Updated•16 years ago
|
Assignee: nobody → ted.mielczarek
Assignee | ||
Updated•16 years ago
|
Status: NEW → ASSIGNED
Comment 1•16 years ago
|
||
Triaged to p2, after confirming with Ted in irc that he's well along working on it!
Priority: -- → P2
Comment 2•16 years ago
|
||
Work in here or likely based upon this will be required to make the buildbot hg poller list files in changes. Other things I'd like to see is a parameter to specify for the poller what it's last change is, or when that was, and for the feed to return the pushes since then, or in that time frame. I wonder if timeframe is something that we'd need for the tinderbox blame column, too.
Comment 3•16 years ago
|
||
Listing files in the buildbot hg poller was not a known goal... why do we need to do that? Parameters for "changes since time X" or "changes since revision X" should be relatively simple.
Assignee | ||
Comment 4•16 years ago
|
||
Listing files is trivially easy, and makes the HTML pushlog view more like bonsai, which I think is the point.
Comment 5•16 years ago
|
||
I need the files to make an educated decision on what to actually build when it comes down to l10n. Not sure yet how we're going to do that in the hg world at all. No idea why getting files wasn't on the initial goals for the feed, I didn't even know that there were initial goals. I did comment over on the buildbot trac bugs that lack of files is sad and should be added.
Assignee | ||
Comment 6•16 years ago
|
||
Ok, this changes the pushlog feed to use the sqlite backend, and also adds a pushloghtml command, which presents the pushlog similarly to the built-in mercurial changeset log. It's kind of hacky, since it's probably duplicating a lot of stuff from the Mercurial templates. I dunno if we can re-use them or not, it's probably worth looking into. I'll file some enhancements as followups, I'm kinda tired of working on this right now.
Attachment #323884 -
Flags: review?(benjamin)
Assignee | ||
Comment 7•16 years ago
|
||
I'm not 100% sure how all my links/etc will work when serving this from apache, I've only tested this with "hg serve". I don't actually know how to setup the Apache bits. bsmedberg: do you have such a setup that you could test this in?
Updated•16 years ago
|
Attachment #323884 -
Flags: review?(benjamin) → review+
Assignee | ||
Comment 8•16 years ago
|
||
Pushed. Since the HTML interface is new, we can deal with fixes in followups.
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•