[debbugs-tracker] bug#32055: closed ([PATCH 0/2] Add sqlcrush.)

[GNU bug Tracking System]

Your message dated Wed, 22 Aug 2018 00:53:56 +0200
with message-id <address@hidden>
and subject line Re: [PATCH 0/2] Add sqlcrush.
has caused the debbugs.gnu.org bug report #32055,
regarding [PATCH 0/2] Add sqlcrush.
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
32055: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=32055
GNU Bug Tracking System
Contact address@hidden with problems
> --- Begin Message ---
>
>
>
> Subject: 
>
> [PATCH 0/2] Add sqlcrush.
>
>
>
>
> Date: 
>
> Wed, 04 Jul 2018 23:09:25 +0200
>
>
>
>
> User-agent: 
>
> mu4e 1.0; emacs 26.1
>
>
>
>
> Guix,
>
> Basically: ttyMyAdmin.
>
> Tested only by poking around in an SQLite database (my reason for 
> packaging this).
>
> Kind regards,
>
> T G-R
>
>
>
> --- End Message ---
> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: [PATCH 0/2] Add sqlcrush.
>
>
>
>
> Date: 
>
> Wed, 22 Aug 2018 00:53:56 +0200
>
>
>
>
> Ludo',
>
> This got buried after the holdays. Thanks for the review!
>
> Ludovic Courtès wrote:
> > Tobias Geerinckx-Rice <me <at> tobias.gr> skribis:
>
> [...]
>
> > > +      (propagated-inputs
> > > +       `(("python-cryptography" ,python-cryptography)
> > > +         ("python-psycopg2" ,python-psycopg2)
> > > +         ("python-pymysql" ,python-pymysql)
> > > +         ("python-sqlalchemy" ,python-sqlalchemy)))
>
> [...]
>
> > If it’s an application rather than a library, then perhaps we 
> > don’t need
> > to propagate these?
>
> Agreed.
>
> > > +    (synopsis "Pure-Python MySQL Driver")
> >
> > Nitpick: "driver" (lowercase).
> >
> > Otherwise LGTM, thanks!
>
> Pushed with the suggested changes as 311f06c et al.
>
> Kind regards,
>
> T G-R
>
>
> --- End Message ---