Patch adds a command that opens an SQL CLI.

CommentFileSizeAuthor
#1 drush_extras.diff1.43 KBdrumm
drush_extras.diff1.43 KBdrumm

Comments

drumm’s picture

drumm
he/him
Location NY, US
commented
StatusFileSize
new drush_extras.diff1.43 KB

(Fix corrupted characters.)

moshe weitzman’s picture

moshe weitzman commented

Love it. I'll wait for some comments and then commit.

moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Fixed

Committed and backported to 6. Thanks.

moshe weitzman’s picture

moshe weitzman commented

BTW, is that $pipes param needed? It is not defined anywhere nor does proc_close() appear to recognize it in the docs.

drumm’s picture

drumm
he/him
Location NY, US
commented

$pipes is for proc_open() and is required. If we wanted to do anything other than pass-through it would be useful.

Like working with files, normally you do

$process = proc_open(...);
// Do stuff with $process
proc_close($process);

Since we don't do anything special, I shortened the whole construct to one line.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.