nordtheme / dircolors

An arctic, north-bluish clean and elegant dircolors theme.
https://www.nordtheme.com/ports/dircolors
MIT License
523 stars 104 forks source link

readme instructions are incorrect #15

Closed jantari closed 3 years ago

jantari commented 4 years ago

In the README.MD,

the following command is suggested:

test -r "~/.dir_colors" && eval $(dircolors ~/.dir_colors)

but it won't work ( tested on Debian 10, bash 5.0-4 ). image

The quotes need to be ommited and the command should be:

test -r ~/.dir_colors && eval $(dircolors ~/.dir_colors)

The problem is explained here: https://serverfault.com/a/417255

arcticicestudio commented 4 years ago

Hi @jantari :wave:, thanks for your contribution :+1:

Well recognized, variable/parameter expansion are always are always a great pitfall (even though the screenshot of the command in the README is correct) 😄 Would you like to submit a PR o fix this in this repository as well as for the port project documentations?

ghost commented 3 years ago

19 is open to address this.