Closed Bug 449377 Opened 17 years ago Closed 17 years ago

pushlog feed errors on empty repo

Categories

(Developer Services :: Mercurial: hg.mozilla.org, defect)

Product:
Developer Services
Component:
Mercurial: hg.mozilla.org
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: Pike, Assigned: Pike)

References

Details

We just set up the l10n repos, and pushlog feed is erroring on the empty repos because it can't find the database file. We should catch that error and return an empty list, IMHO.
I thought this used to work right. It's possible I broke it.
Assignee: nobody → ted.mielczarek
I fixed this while I was fixing bug 464018.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Found out that this is actually failing differently on hg.mo, because the webapp doesn't have write permission to the repo, so it fails in sqlite.connect.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Taking, Ted's sick and I have an idea on how to do it.
Assignee: ted.mielczarek → l10n
Status: REOPENED → ASSIGNED
http://hg.mozilla.org/users/bsmedberg_mozilla.com/hgpoller/rev/6db4ef76c839 has both a test and the fix. I did the test first, which broke, and then fixed it. Works locally on hg serve on a readonly repo, too. Back to FIXED. PS: I got a landing card from Ted on irc, used that.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago → 17 years ago
Resolution: --- → FIXED
Blocks: 466805
Filed bug 466805 to get this into production.
Verified, http://hg.mozilla.org/releases/l10n-mozilla-1.9.1/bn-IN/pushloghtml displays an empty pushlog right now.
Status: RESOLVED → VERIFIED
Product: mozilla.org → Release Engineering
Product: Release Engineering → Developer Services
You need to log in before you can comment on or make changes to this bug.