[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:30:53 +0100 |
Sorry. That was the wrong patch.
0001-lisp-ob-sql.el-allow-string-in-sql-connection-alist.patch
Description: Binary data
> On 20. Jan 2023, at 18:24, Andreas Gerler <baron@bundesbrandschatzamt.de>
> wrote:
>
> <0001-lisp-ob-eval.el-Display-error-fix.patch>
>
>> 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, 2023/01/20
- Re: [BUG] ob-sql sql-connection-alist,
Andreas Gerler <=
- 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
- Re: [BUG] ob-sql sql-connection-alist, Andreas Gerler, 2023/01/28