Re: [aur-dev] [PATCH 1/1] add SQL_DEBUG variable and database logging

2011-04-27 Thread Lukas Fleischer
On Tue, Apr 26, 2011 at 11:52:11AM -0700, elij wrote:
 add a hook to db_query to log all sql queries when SQL_DEBUG is set
 Additionally, provide better logging for sql error situations (provide
 backtrace as well as error message).
 ---
  web/lib/aur.inc  |   12 
  web/lib/config.inc.proto |4 
  2 files changed, 16 insertions(+), 0 deletions(-)
 

Pushed that (after fixing up stuff). I also added some note on merging
config file changes to UPGRADING.


Re: [aur-dev] [PATCH 1/1] add SQL_DEBUG variable and database logging

2011-04-27 Thread elij
On Wed, Apr 27, 2011 at 9:34 AM, Lukas Fleischer
archli...@cryptocrack.de wrote:
 On Tue, Apr 26, 2011 at 11:52:11AM -0700, elij wrote:
 add a hook to db_query to log all sql queries when SQL_DEBUG is set
 Additionally, provide better logging for sql error situations (provide
 backtrace as well as error message).
 ---
  web/lib/aur.inc          |   12 
  web/lib/config.inc.proto |    4 
  2 files changed, 16 insertions(+), 0 deletions(-)


 Pushed that (after fixing up stuff). I also added some note on merging
 config file changes to UPGRADING.

Just curious, but what did you 'fixed up'?


Re: [aur-dev] [PATCH 1/1] add SQL_DEBUG variable and database logging

2011-04-27 Thread elij
On Wed, Apr 27, 2011 at 1:57 PM, elij elij...@gmail.com wrote:
 On Wed, Apr 27, 2011 at 9:34 AM, Lukas Fleischer
 archli...@cryptocrack.de wrote:
 On Tue, Apr 26, 2011 at 11:52:11AM -0700, elij wrote:
 add a hook to db_query to log all sql queries when SQL_DEBUG is set
 Additionally, provide better logging for sql error situations (provide
 backtrace as well as error message).
 ---
  web/lib/aur.inc          |   12 
  web/lib/config.inc.proto |    4 
  2 files changed, 16 insertions(+), 0 deletions(-)


 Pushed that (after fixing up stuff). I also added some note on merging
 config file changes to UPGRADING.

 Just curious, but what did you 'fixed up'?

oh. nevermind. I see you just squashed the previous 'oops forgot to
remove this quote mark' fixes.
:)