pushlog contains 'None' at the end of the file causing HgPoller to barf

RESOLVED FIXED

Status

defect
--
critical
RESOLVED FIXED
11 years ago
5 years ago

People

(Reporter: bhearsum, Assigned: ted)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Getting tracebacks like this:
2008/07/16 03:38 PDT [-] Hg poll failed: [Failure instance: Traceback: <class 'xml.parsers.expat.ExpatError'>: junk after document element: line 141, column 16
        /tools/twisted-2.4.0/lib/python2.5/site-packages/twisted/internet/defer.py:107:maybeDeferred
        /tools/buildbotcustom/buildbotcustom/changes/hgpoller.py:194:poll
        /tools/twisted-2.4.0/lib/python2.5/site-packages/twisted/internet/defer.py:191:addCallback
        /tools/twisted-2.4.0/lib/python2.5/site-packages/twisted/internet/defer.py:182:addCallbacks
        --- <exception caught here> ---
        /tools/twisted-2.4.0/lib/python2.5/site-packages/twisted/internet/defer.py:307:_runCallbacks
        /tools/buildbotcustom/buildbotcustom/changes/hgpoller.py:244:_process_changes
        /tools/buildbotcustom/buildbotcustom/changes/hgpoller.py:225:_parse_changes
        /tools/python-2.5.1/lib/python2.5/xml/dom/minidom.py:1923:parseString
        /tools/python-2.5.1/lib/python2.5/xml/dom/expatbuilder.py:940:parseString
        /tools/python-2.5.1/lib/python2.5/xml/dom/expatbuilder.py:223:parseString


And here's the last line of pushlog:
 </entry></feed>None
*sigh* Sorry about that. I'll figure out what's going on here.
Assignee: nobody → ted.mielczarek
Stupid, I made a change when I switched to using a template for the html output, but it broke the feed output, since that's not using a template. Didn't notice it here, I must be running a different version of Mercurial than hg.mozilla.org.

Pushed:
http://hg.mozilla.org/users/bsmedberg_mozilla.com/hgpoller/index.cgi/rev/d4bf9acf1959
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Filed bug 445484 on getting this deployed.
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.