algorithmswithgo.com icon indicating copy to clipboard operation
algorithmswithgo.com copied to clipboard

Moves hyphen as last arg.

Open mdeltour opened this issue 5 years ago • 5 comments

Hyphen in [...] returns an error : "grep: Invalid range end" In my opinion in this case, hyphen needs to be last or first.

mdeltour avatar Jan 22 '20 13:01 mdeltour

What OS are you on, or what is different that is causing this to not work for you but to work for me? Trying to figure out what, if any, notes I should add to the vids.

joncalhoun avatar Jan 22 '20 13:01 joncalhoun

I found this document that gives more information. https://lists.gnu.org/archive/html/bug-coreutils/2015-02/msg00033.html

For your information:

Distributor ID:	Ubuntu
Description:	Ubuntu 16.04.6 LTS
Release:	16.04
Codename:	xenial

mdeltour avatar Jan 22 '20 14:01 mdeltour

Facing same issue here

➜  module01 git:(master) ✗ go_test --run=TestSum -v
grep: Invalid range end

System info:

Distributor ID:	elementary
Description:	elementary OS 5.1 Hera
Release:	5.1
Codename:	hera

Surendrajat avatar Feb 08 '20 11:02 Surendrajat

I need to redo the video, or at least add a note to it, before I can commit this. I'm letting you know here so you are aware that I'm not ignoring this, but I don't want the code here to not match the videos.

joncalhoun avatar Feb 10 '20 14:02 joncalhoun

It is too trivial to spot in the video, and nobody's gonna type this long script off that video either. Everybody's gonna copy from README anyway so updating that should suffice, IMO.

Surendrajat avatar Feb 10 '20 15:02 Surendrajat