if 'createdb' fails, the error message should include the dbi error

RESOLVED FIXED in Bugzilla 2.18

Status

()

RESOLVED FIXED
14 years ago
6 years ago

People

(Reporter: glob, Assigned: glob)

Tracking

2.18
Bugzilla 2.18
Bug Flags:
approval +
approval2.18 +

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

14 years ago
if 'createdb' fails, the error message should include the dbi error.
(Assignee)

Comment 1

14 years ago
Created attachment 161846 [details] [diff] [review]
include $dbh->errstr in createdb error
(Assignee)

Updated

14 years ago
Attachment #161846 - Flags: review?

Comment 2

14 years ago
Comment on attachment 161846 [details] [diff] [review]
include $dbh->errstr in createdb error

Sure.
Attachment #161846 - Flags: review? → review+
(Assignee)

Updated

14 years ago
Flags: approval?
Comment on attachment 161846 [details] [diff] [review]
include $dbh->errstr in createdb error

'patch' won't apply this, says it's malformed.	The header on the first section
appears to have an inaccurate line count.
Attachment #161846 - Flags: review-
Flags: approval?
Hardware: PC → All
(Assignee)

Comment 4

14 years ago
Created attachment 162560 [details] [diff] [review]
include $dbh->errstr in createdb error v2
Attachment #161846 - Attachment is obsolete: true
(Assignee)

Comment 5

14 years ago
Comment on attachment 162560 [details] [diff] [review]
include $dbh->errstr in createdb error v2

carrying forward r=kiko
Attachment #162560 - Flags: review+
(Assignee)

Updated

14 years ago
Status: NEW → ASSIGNED
Flags: approval?
Let's put this in 2.18, too.  Useful debugging info.
Flags: approval?
Flags: approval2.18+
Flags: approval+
Target Milestone: --- → Bugzilla 2.18

Comment 7

14 years ago
Checking in checksetup.pl;
/cvsroot/mozilla/webtools/bugzilla/checksetup.pl,v  <--  checksetup.pl
new revision: 1.310; previous revision: 1.309
done

Checking in checksetup.pl;
/cvsroot/mozilla/webtools/bugzilla/checksetup.pl,v  <--  checksetup.pl
new revision: 1.289.2.10; previous revision: 1.289.2.9
done
Status: ASSIGNED → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.