Re: patch suggestion mysql-sql.lisp
Kevin Rosenberg <kevin-HJRc7zDS/[email protected]>
| Newsgroups | gmane.lisp.clsql.general |
|---|---|
| Message-ID | <[email protected]> |
On Aug 23, 2009, at 12:13 AM, Michael Pheasant wrote: > Database-list ignores a supplied database name and instead uses > "mysql". It is possible that a mysql userid has permissions to issue > "show databases" but no permission to connect to db "mysql". > database-list gives an error in this scenario. > > The following patch overcomes this by using the user-supplied db name > or else "mysql" if nil supplied. That seems like a helpful patch. Thanks, I'll include it in the next CLSQL release. > I'm new to this list; what is the procedure for suggesting patches? Posting a unified diff to this list is the best way. An attachment is a little bit better than an inlined diff. Additions to the test suite to demonstrate the improved behavior is always appreciated. Kevin _______________________________________________ CLSQL mailing list [email protected] http://lists.b9.com/cgi-bin/mailman/listinfo/clsql