[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [BUG] ob-sql sql-connection-alist
From: |
Andreas Gerler |
Subject: |
Re: [BUG] ob-sql sql-connection-alist |
Date: |
Fri, 20 Jan 2023 18:24:41 +0100 |
0001-lisp-ob-eval.el-Display-error-fix.patch
Description: Binary data
> On 16. Jan 2023, at 11:25, Daniel Kraus <daniel@kraus.my> wrote:
>
> Hi!
>
> Andreas Gerler <baron@bundesbrandschatzamt.de> writes:
>
>> Last week I heard about using ob-sql with credentials stored in the variable
>> used by isql.
>> However I had to modify ob-sql to get it actually working.
>> Can somebody test the pach before I send in a commit?
>>
>> #+begin_src sql :engine mysql :dbconnection testdb
>> show tables;
>> #+end_src
>
> I actually use this feature daily.
> You have to quote the dbconnection. So this works currently:
>
>> #+begin_src sql :engine mysql :dbconnection 'testdb
>
> but I would agree that not needing the quote makes sense.
> And since `assoc-string` works with symbol and string (i.e. it's backwards
> compatible)
> I would install the patch if you send it.
>
>> I was considering writing another patch to map the sql-product to engine.
>> That way we could get rid of another parameter in the src block.
>> Opinions?
>
> I agree. Specifying :engine when it's already in the connection-alist is
> unnecessary.
>
> Thanks,
> Daniel
signature.asc
Description: Message signed with OpenPGP
- [BUG] ob-sql sql-connection-alist, Andreas Gerler, 2023/01/15
- Re: [BUG] ob-sql sql-connection-alist, Daniel Kraus, 2023/01/16
- Re: [BUG] ob-sql sql-connection-alist,
Andreas Gerler <=
- Re: [BUG] ob-sql sql-connection-alist, Andreas Gerler, 2023/01/20
- Re: [BUG] ob-sql sql-connection-alist, Daniel Kraus, 2023/01/20
- Re: [BUG] ob-sql sql-connection-alist, Andreas Gerler, 2023/01/20
- Re: [BUG] ob-sql sql-connection-alist, Ihor Radchenko, 2023/01/21
- Re: [BUG] ob-sql sql-connection-alist, Andreas Gerler, 2023/01/21
- Re: [BUG] ob-sql sql-connection-alist, Bastien Guerry, 2023/01/22
- Re: [BUG] ob-sql sql-connection-alist, Bastien Guerry, 2023/01/26
- Re: [BUG] ob-sql sql-connection-alist, Ihor Radchenko, 2023/01/27
- Re: [BUG] ob-sql sql-connection-alist, Ihor Radchenko, 2023/01/27
- Re: [BUG] ob-sql sql-connection-alist, Andreas Gerler, 2023/01/27