dthree / cash

Cross-platform Linux commands in ES6

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Unexpected Exit on Grep.

nyounker opened this issue · comments

Initially I attempted a "grep" to search for a string inside a file using the -irl flags and found that the -L (lowercase) was not available but then tried the -n flag instead

Command executed

grep -irn "test" *

cash exits to windows prompt.

verification of error.

root:C/Users/Public/Documents$ root:C/Users/Public/Documents$ mkdir test1, test2, test3

root:C/Users/Public/Documents$ ls
desktop.ini My Music  My Pictures  My Videos  test1,  test2,  test3

root:C/Users/Public/Documents$ grep -irn "test" *
readline.js:925
            throw err;
            ^

SyntaxError: Invalid regular expression: /(*)/: Nothing to repeat
    at RegExp (native)
    at new RegExp (C:\Users\$user\AppData\Roaming\npm\node_modules\cash-global\node_modules\cash\node_modules\vorpal\node_modules\babel-polyfill\node_mo
dules\core-js\modules\es6.regexp.constructor.js:26:11)
    at CommandInstance.exec (C:\Users\$user\AppData\Roaming\npm\node_modules\cash-global\node_modules\cash\node_modules\vorpal-grep\dist\grep.js:23:19)
    at CommandInstance.<anonymous> (C:\Users\$user\AppData\Roaming\npm\node_modules\cash-global\node_modules\cash\node_modules\vorpal-grep\dist\grep.js:
97:15)
    at EventEmitter.session.execCommandSet (C:\Users\$user\AppData\Roaming\npm\node_modules\cash-global\node_modules\cash\node_modules\vorpal\dist\sessi
on.js:476:20)
    at EventEmitter.vorpal._exec (C:\Users\$user\AppData\Roaming\npm\node_modules\cash-global\node_modules\cash\node_modules\vorpal\dist\vorpal.js:970:1
8)
    at EventEmitter.vorpal._execQueueItem (C:\Users\$user\AppData\Roaming\npm\node_modules\cash-global\node_modules\cash\node_modules\vorpal\dist\vorpal
.js:761:17)
    at EventEmitter.vorpal._queueHandler (C:\Users\$user\AppData\Roaming\npm\node_modules\cash-global\node_modules\cash\node_modules\vorpal\dist\vorpal.
js:745:10)
    at EventEmitter.vorpal.exec (C:\Users\$user\AppData\Roaming\npm\node_modules\cash-global\node_modules\cash\node_modules\vorpal\dist\vorpal.js:692:10
)
    at C:\Users\$user\AppData\Roaming\npm\node_modules\cash-global\node_modules\cash\node_modules\vorpal\dist\vorpal.js:635:10
readline.js:925
            throw err;
            ^

SyntaxError: Invalid regular expression: /(*)/: Nothing to repeat
    at RegExp (native)
    at new RegExp (C:\Users\$user\AppData\Roaming\npm\node_modules\cash-global\node_modules\cash\node_modules\vorpal\node_modules\babel-polyfill\node_mo
dules\core-js\modules\es6.regexp.constructor.js:26:11)
    at CommandInstance.exec (C:\Users\$user\AppData\Roaming\npm\node_modules\cash-global\node_modules\cash\node_modules\vorpal-grep\dist\grep.js:23:19)
    at CommandInstance.<anonymous> (C:\Users\$user\AppData\Roaming\npm\node_modules\cash-global\node_modules\cash\node_modules\vorpal-grep\dist\grep.js:
97:15)
    at EventEmitter.session.execCommandSet (C:\Users\$user\AppData\Roaming\npm\node_modules\cash-global\node_modules\cash\node_modules\vorpal\dist\sessi
on.js:476:20)
    at EventEmitter.vorpal._exec (C:\Users\$user\AppData\Roaming\npm\node_modules\cash-global\node_modules\cash\node_modules\vorpal\dist\vorpal.js:970:1
8)
    at EventEmitter.vorpal._execQueueItem (C:\Users\$user\AppData\Roaming\npm\node_modules\cash-global\node_modules\cash\node_modules\vorpal\dist\vorpal
.js:761:17)
    at EventEmitter.vorpal._queueHandler (C:\Users\$user\AppData\Roaming\npm\node_modules\cash-global\node_modules\cash\node_modules\vorpal\dist\vorpal.
js:745:10)
    at EventEmitter.vorpal.exec (C:\Users\$user\AppData\Roaming\npm\node_modules\cash-global\node_modules\cash\node_modules\vorpal\dist\vorpal.js:692:10
)
    at C:\Users\$user\AppData\Roaming\npm\node_modules\cash-global\node_modules\cash\node_modules\vorpal\dist\vorpal.js:635:10

I don't know if this output will help and I'm not sure if there is an issue with an thirdparty .js, however I did test this exit with the following commands just to verify as well.. same result of exiting.

grep -irn "test*" *
grep -irn 'test*' *
grep -irn 'test' *
commented

Thanks.

I'll try to resolve this issue.

Wow. I didn't mean to make a spectacle of that error. I really apologize and thank you for legien stepping up.. after a month later I realized when m$ saw your project they got scared. You put some sense into the world of m$ . You and everybody participating. so they released bash/ or if you are from windows \ mixed with ubuntu and unity. ... There is a lot more that needs to be done with this project, only because everybody noticed the hard cut that m$ tried to apply. someone is scared about this project and that means WE should keep on working on it.

I would recommend some additional help as this issue is still unresolved and I think it is an important issue to solve no matter the statement of releasing bash for ms, as is no one has solved this issue. seems like a power play to me to deviate from the good work that is going on here.

The whole project was not updated for a month. I had no time recently. So, in you want to resolve this issue: Go ahead.

commented

@nyounker thanks for your kind words :)

@Legien sorry about that! I've been super busy in life recently and haven't had any time for OSS :(

Hoping that will change soon!