"!record <logfile>" as an external command line option (e.g. "--log <logfile>")

[beaumex]

First things first : thank you for maintaining that extraordinary tool ; I searched a lot to find this graal in a very heterogeneous db environment !

I mainly use (as of today) sqlline 1.3 as a ddl deployment tool but I still have to dance a little in order to wrap those committed scripts I have to deploy in order to get the "!record" statements in.

Provided the same functionality as a command line option, I could leave scripts unaltered
(In fact, I build an intermediary launcher as a sqlline commands script including !record and !run on the original ddl file)

Thanks

[julianhyde]

You're welcome! I'm very pleased that sqlline is useful to you.

A --log <logfile> command line option totally makes sense. I may get to it someday, but I would be grateful to receive e pull request.

[beaumex]

Hi Julian,

but I would be grateful to receive e pull request.

I'm not familiar to git but by what I've found about "pull request", this is some kind of invitation to contribute (?) I'm too lousy a developer for that but if I may formalize further a simple "request" (via pull request ?) Also, about contributing, is there a donor way to support ? Thanks Cheers, Xavier

…

On Wed, Sep 20, 2017 at 9:07 PM, Julian Hyde ***@***.***> wrote: You're welcome! I'm very pleased that sqlline is useful to you. A --log <logfile> command line option totally makes sense. I may get to it someday, but I would be grateful to receive e pull request. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#67 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AenZM05HW6fSpiq-DtEQIG7kzrNU0e8lks5skWKOgaJpZM4Pcjuv> .

--

_____________________________________________________________ Xavier BEAUME 30 rue d'Issy +33 (0) 6.60.06.60.26 92100 Boulogne - Billancourt xavier@beaume.org France http://www.beaume.org/beaumex

[snuyanzin]

Should the issue be closed or is there anything not implemented yet?
cc @julianhyde

[julianhyde]

Fixed in f62bb74, PR #92. Thanks @snuyanzin!