Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unexpected Exit on Grep. #56

Open
nyounker opened this issue Mar 4, 2016 · 6 comments
Open

Unexpected Exit on Grep. #56

nyounker opened this issue Mar 4, 2016 · 6 comments

Comments

@nyounker
Copy link

nyounker commented Mar 4, 2016

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' *
@dthree
Copy link
Owner

dthree commented Mar 4, 2016

Thanks.

@i-am-steve-oh
Copy link
Collaborator

I'll try to resolve this issue.

@i-am-steve-oh i-am-steve-oh self-assigned this Mar 9, 2016
@nyounker
Copy link
Author

nyounker commented Apr 7, 2016

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.

@nyounker
Copy link
Author

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.

@i-am-steve-oh i-am-steve-oh removed their assignment Apr 21, 2016
@i-am-steve-oh
Copy link
Collaborator

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

@dthree
Copy link
Owner

dthree commented Apr 21, 2016

@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!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants