Re: [Orgmode] Re: Patch for ob-sql.el SQL output

2010-12-21 Thread Charles Sebold
Sébastien: What do you think would be a good solution for the blank between the output of a table and the row count if someone didn't run SET NOCOUNT ON? I mean, should that be another hline, or no blank line, or what? If somebody's trying to programatically use this table, it will mess up their

Re: [Orgmode] Re: Patch for ob-sql.el SQL output

2010-12-21 Thread Eric Schulte
Hi Charles, This looks great, I've just applied this most recent patch. Thanks -- Eric Charles Sebold writes: > OK, another change: added a fix for the blank line at the end. > > diff --git a/lisp/ob-sql.el b/lisp/ob-sql.el > index 5bb123d..2ff85d9 100644 > --- a/lisp/ob-sql.el > +++ b/lisp/ob

Re: [Orgmode] Re: Patch for ob-sql.el SQL output

2010-12-21 Thread Charles Sebold
OK, another change: added a fix for the blank line at the end. diff --git a/lisp/ob-sql.el b/lisp/ob-sql.el index 5bb123d..2ff85d9 100644 --- a/lisp/ob-sql.el +++ b/lisp/ob-sql.el @@ -65,6 +65,7 @@ This function is called by `org-babel-execute-src-block'." (in-file (org-babel-temp-file "

Re: [Orgmode] Re: Patch for ob-sql.el SQL output

2010-12-21 Thread Charles Sebold
I feel silly. I was just testing with one output column. Change the $ in the regexp to "[^-]" or just reapply patch as follows: diff --git a/lisp/ob-sql.el b/lisp/ob-sql.el index 5bb123d..32b7bf0 100644 --- a/lisp/ob-sql.el +++ b/lisp/ob-sql.el @@ -65,6 +65,7 @@ This function is called by `org-b