> On 16. Jan 2023, at 11:25, Daniel Kraus wrote: > > Hi! > > Andreas Gerler 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